This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG0268e123bea5: [PowerPC] swdiv_nochk Builtins for XL Compat
(authored by quinnp, committed by kamaub).
Changed prior to commit:
https://reviews.llv
quinnp updated this revision to Diff 359327.
quinnp added a comment.
Removing entry checks from front end test.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106150/new/
https://reviews.llvm.org/D106150
Files:
clang/include/clang/Basic/BuiltinsP
quinnp updated this revision to Diff 359325.
quinnp added a comment.
Fixing a syntax error.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106150/new/
https://reviews.llvm.org/D106150
Files:
clang/include/clang/Basic/BuiltinsPPC.def
clang/lib/B
quinnp updated this revision to Diff 359310.
quinnp added a comment.
Adressing review comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106150/new/
https://reviews.llvm.org/D106150
Files:
clang/include/clang/Basic/BuiltinsPPC.def
clang/l
nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.
LGTM other than a couple of nits.
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:15627
+Builder.getFastMathFlags().setFast();
+Value *fdiv = Builder.CreateFDiv(Ops[0
quinnp created this revision.
Herald added subscribers: shchenz, kbarton, nemanjai.
quinnp requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This patch is in a series of patches to provide builtins for
compatibility with the XL compiler. This
