EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

LGTM minus inline comments.



================
Comment at: test/support/nasty_macros.hpp:55
 
+#define __output NASTY_MACRO
+#define __input NASTY_MACRO
----------------
Shouldn't these not be defined when running the tests with a CHER compiler? 
Otherwise the macro will conflict with the keyword?


Repository:
  rL LLVM

https://reviews.llvm.org/D39537



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D39537: Renam... Eric Fiselier via Phabricator via cfe-commits

Reply via email to