ROL: Explicitly uses GlobalOrdinal=int with Tpetra, doesn't need to

[mhoemmen]

@trilinos/rol
Line 292 of rol/example/PDE-OPT/TOOLS/assembler.hpp (in Trilinos develop) explicitly calls Tpetra::createOneToOne<int,int>. This forces use of GlobalOrdinal = int. See #74 for why we don't want this.

The line is easy to fix: just do this:

myUniqueStateMap_ = Tpetra::createOneToOne (myOverlapStateMap_);

That is, let createOneToOne deduce its template parameters on its own. It can do so by looking at the input argument.

There are a few other places in the file that force this constraint, e.g., line 231:

Teuchos::Array<int> myGlobalIds;

(replace int with GO and define GO as Tpetra::Map<>::global_ordinal_type).

[dridzal]

@mhoemmen , do you need this fix pushed ASAP?

[mhoemmen]

@dridzal No, but thanks for asking! You would see faster build times by disabling support for GlobalOrdinal=int, especially with ETI turned on.

[github-actions]

This issue has had no activity for 365 days and is marked for closure. It will be closed after an additional 30 days of inactivity.
If you would like to keep this issue open please add a comment and remove the MARKED_FOR_CLOSURE label.
If this issue should be kept open even with no activity beyond the time limits you can add the label DO_NOT_AUTOCLOSE.

[github-actions]

This issue was closed due to inactivity for 395 days.