[PATCH] D51446: [OpenMP][bugfix] Add missing macros for Power

Hahnfeld added a comment. Not needed anymore after the reverts in https://reviews.llvm.org/rC341115 and https://reviews.llvm.org/rC341118, right? Maybe we should revive the test to make sure we don't break this in the future? Repository: rC Clang https://reviews.llvm.org/D51446 _

[PATCH] D51446: [OpenMP][bugfix] Add missing macros for Power

gtbercea updated this revision to Diff 163377. gtbercea added a comment. Add test. Repository: rC Clang https://reviews.llvm.org/D51446 Files: lib/Frontend/InitPreprocessor.cpp test/Preprocessor/aux-triple.c Index: test/Preprocessor/aux-triple.c ==

[PATCH] D51446: [OpenMP][bugfix] Add missing macros for Power

kkwli0 added a comment. Need to update the test too? Repository: rC Clang https://reviews.llvm.org/D51446 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D51446: [OpenMP][bugfix] Add missing macros for Power

Hahnfeld requested changes to this revision. Hahnfeld added a comment. This revision now requires changes to proceed. Please also update the test. Comment at: lib/Frontend/InitPreprocessor.cpp:1115-1130 case llvm::Triple::ppc64: +if (AuxTI.getLongDoubleWidth() == 128) {

[PATCH] D51446: [OpenMP][bugfix] Add missing macros for Power

gtbercea created this revision. gtbercea added reviewers: ABataev, Hahnfeld, caomhin. Herald added subscribers: cfe-commits, guansong. Add missing macros when the auxiliary triple points to the PPC architecture. Repository: rC Clang https://reviews.llvm.org/D51446 Files: lib/Frontend/InitP