echristo accepted this revision.
echristo added a reviewer: echristo.
echristo added a comment.
Please do add tests before committing. Otherwise ok.
Thanks.
-eric
================
Comment at: lib/Basic/Targets.cpp:1239
@@ -1227,1 +1238,3 @@
+ if (defs & ArchDefinePwr9)
+ Builder.defineMacro("_ARCH_PWR9");
if (defs & ArchDefineA2)
----------------
nemanjai wrote:
> Come to think of it - I should add a test case for this macro. It will be on
> the next revision if one is required or in the actual commit otherwise.
Please add a test for everything you just added, yes. :)
Repository:
rL LLVM
http://reviews.llvm.org/D19684
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits