nemanjai accepted this revision. nemanjai added a comment. This revision is now accepted and ready to land.
LGTM aside from a small nit. ================ Comment at: clang/lib/Basic/Targets/PPC.cpp:124 Builder.defineMacro("__stfiw", "__builtin_ppc_stfiw"); + Builder.defineMacro("__stbcx", "__builtin_ppc_stbcx"); Builder.defineMacro("__stdcx", "__builtin_ppc_stdcx"); ---------------- Nit: move just under `sthcx` so the order makes sense: doubleword, word, haflword, byte. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106484/new/ https://reviews.llvm.org/D106484 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits