hubert.reinterpretcast accepted this revision. hubert.reinterpretcast added a comment. This revision is now accepted and ready to land.
Thanks. LGTM with minor nits. ================ Comment at: clang/test/CodeGen/builtins-ppc-xlcompat-fetch.c:19 // -void test_builtin_ppc_fetch_and_add(unsigned int a, unsigned int b) { +void test_builtin_ppc_fetch_and_add(int a, unsigned int b) { __fetch_and_add(&a, b); ---------------- ================ Comment at: clang/test/CodeGen/builtins-ppc-xlcompat-fetch.c:34 // -void test_builtin_ppc_fetch_and_addlp(unsigned long a, unsigned long b) { +void test_builtin_ppc_fetch_and_addlp(long a, unsigned long b) { __fetch_and_addlp(&a, b); ---------------- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106920/new/ https://reviews.llvm.org/D106920 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits