[PATCH] D110282: [PowerPC] SemaChecking for darn family of builtins

Lei Huang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 23 06:34:02 PDT 2021


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



More information about the cfe-commits mailing list