Project

General

Profile

Task #2699

Test POWER9

Added by Mark Abraham 6 months ago. Updated 17 days ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
core library
Target version:
Difficulty:
uncategorized
Close

Description

Mark and Szilard have access at BSC.

xlc 16.1 still cannot compiler c++11 in its c++11 mode, so I propose we ban that compiler so people don't waste time trying


Subtasks

Feature #2748: Power9 build system supportClosed
Task #2772: check performance on POWER9Rejected

History

#1 Updated by Erik Lindahl 6 months ago

I would rather suggest to keep it simple and not add more CMake code for corner cases - we can test for specific features, or simply accept that some broken compilers will result in things crashing.

For cases where there are silent bugs we might need specific code to ban compilers, but otherwise I would suggest that we avoid adding a ton of code checking specific compiler versions that we then need to keep testing and modifying every release!

#2 Updated by Mark Abraham 6 months ago

Erik Lindahl wrote:

I would rather suggest to keep it simple and not add more CMake code for corner cases - we can test for specific features, or simply accept that some broken compilers will result in things crashing.

OK, but you're the one who wants to stop checking in our CMake for c++11 features that don't work :-) e.g. adding

class temp
{
  int x[2] = {0, 0};
};

in the existing checks for c++11 compatibility of the compiler is likely enough to effectively ban the xlc with which I've noticed a compilation error for that kind of construct. IMO that is what we should do.

For cases where there are silent bugs we might need specific code to ban compilers, but otherwise I would suggest that we avoid adding a ton of code checking specific compiler versions that we then need to keep testing and modifying every release!

Sure, I'd much rather have a feature test than a specific disabling

#3 Updated by Szilárd Páll 6 months ago

#4 Updated by Paul Bauer 4 months ago

  • Target version changed from 2019 to 2020

retargeted to 2020

Also available in: Atom PDF