lei accepted this revision as: lei. lei added a comment. This revision is now accepted and ready to land.
LGTM. Pleases address the test issue on commit. ================ Comment at: clang/test/CodeGen/builtins-ppc-xlcompat-darn-32.c:13 +int testdarn_32(void) { + return __darn_32(); +} ---------------- Isn't this valid for both 32 and 64bit? Maybe change one of the run lines above to a 64bit test. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110282/new/ https://reviews.llvm.org/D110282 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits