Re: [PATCH] D19684: Power9 - Support for -mcpu=pwr9 in the front end

2016-05-09 Thread Nemanja Ivanovic via cfe-commits
nemanjai closed this revision. nemanjai added a comment. Committed revision 268951. Repository: rL LLVM http://reviews.llvm.org/D19684 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

Re: [PATCH] D19684: Power9 - Support for -mcpu=pwr9 in the front end

2016-05-03 Thread Eric Christopher via cfe-commits
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.defineMacr

Re: [PATCH] D19684: Power9 - Support for -mcpu=pwr9 in the front end

2016-05-03 Thread Kit Barton via cfe-commits
kbarton accepted this revision. kbarton added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM http://reviews.llvm.org/D19684 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bi

Re: [PATCH] D19684: Power9 - Support for -mcpu=pwr9 in the front end

2016-04-28 Thread Nemanja Ivanovic via cfe-commits
nemanjai added inline comments. Comment at: lib/Basic/Targets.cpp:1239 @@ -1227,1 +1238,3 @@ + if (defs & ArchDefinePwr9) +Builder.defineMacro("_ARCH_PWR9"); if (defs & ArchDefineA2) Come to think of it - I should add a test case for this macro. It will be

[PATCH] D19684: Power9 - Support for -mcpu=pwr9 in the front end

2016-04-28 Thread Nemanja Ivanovic via cfe-commits
nemanjai created this revision. nemanjai added reviewers: hfinkel, wschmidt, kbarton, amehsan, seurer, cycheng. nemanjai added a subscriber: cfe-commits. nemanjai set the repository for this revision to rL LLVM. This patch simply adds support for the new CPU in anticipation of Power9. There isn't