Project

General

Profile

Cleanup #1311

Cleanup #1064: Replace std::maps with a better implementation

Align XL-specific conditional compilation TRAM to relevant versions

Added by Phil Miller over 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
11/23/2016
Due date:
% Done:

0%


Description

We need testing of the XL compiler on Crest to see whether it references standard libraries that contain std::unordered_map and whether it has fixed the bug that prevented compiling TRAM under BGQ XLC. Then adjust the compilation flags accordingly.

History

#1 Updated by Phil Miller over 2 years ago

  • Target version changed from 6.8.0 to 6.8.1

#2 Updated by Sam White almost 2 years ago

If we're requiring C++11 support in 6.9.0 does this issue go away?

#3 Updated by Eric Bohm almost 2 years ago

  • Target version changed from 6.8.1 to 6.9.0

#4 Updated by Sam White over 1 year ago

  • Subject changed from Align XL-specific conditional compilation (TRAM, std::unordered_map) to relevant versions to Align XL-specific conditional compilation TRAM to relevant versions

TRAM needs to be tested on the XLC on Summit-dev now without the #if CMK_USING_XLC's in charmxi's generated code

Related:
https://charm.cs.illinois.edu/gerrit/#/c/3183/
https://charm.cs.illinois.edu/gerrit/#/c/3190/
https://charm.cs.illinois.edu/gerrit/#/c/3189/

#5 Updated by Ronak Buch over 1 year ago

  • Status changed from New to In Progress

#6 Updated by Ronak Buch over 1 year ago

I think this has already been fixed in https://charm.cs.illinois.edu/gerrit/#/c/3190/, but my latest changes fix improve the situation a bit: https://charm.cs.illinois.edu/gerrit/#/c/3603/, https://charm.cs.illinois.edu/gerrit/#/c/3604/, https://charm.cs.illinois.edu/gerrit/#/c/3605/.

One lingering question is if we should remove the {mpi, pami, pamilrts}-bluegeneq conv-mach.sh files (and the other relevant files). They currently define CMK_USING_XLC, which is what all of these new changes is trying to excise. I think this should be done since BG/Q is no longer a relevant target for new versions of Charm++. Any input?

#7 Updated by Ronak Buch over 1 year ago

  • Status changed from In Progress to Closed

Also available in: Atom PDF