This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGfd855c24c72c: [PowerPC] Restore FastMathFlags of Builder for
Vector FDiv Builtins (authored by quinnp, committed by kamaub).
Changed prior to commit
quinnp updated this revision to Diff 359338.
quinnp added a comment.
Removing entry checks from front end test and renaming a variable.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105984/new/
https://reviews.llvm.org/D105984
Files:
clang/lib/C
nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks for fixing this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105984/new/
https://reviews.llvm.org/D105984
__
quinnp updated this revision to Diff 359118.
quinnp added a comment.
Removing a newline.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105984/new/
https://reviews.llvm.org/D105984
Files:
clang/lib/CodeGen/CGBuiltin.cpp
clang/test/CodeGen/built
quinnp updated this revision to Diff 359116.
quinnp added a comment.
Adding `REQUIRES: powerpc-registered target` to front end test case`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105984/new/
https://reviews.llvm.org/D105984
Files:
clang/li
quinnp updated this revision to Diff 359093.
quinnp marked an inline comment as done.
quinnp added a comment.
Adressing review comment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105984/new/
https://reviews.llvm.org/D105984
Files:
clang/lib/C
qiucf added inline comments.
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:15140
+ Value *fdiv = Builder.CreateFDiv(X, Y, "recipdiv");
+ Builder.getFastMathFlags().operator&=(FMF);
+ return fdiv;
Repository:
rG LLVM Github Monorepo
CHANGES SINC
quinnp updated this revision to Diff 358613.
quinnp retitled this revision from "[PowerPC] Restore FastMathFlags of Builder
for Vector FDiv Builtins " to "[PowerPC] Restore FastMathFlags of Builder for
Vector FDiv Builtins".
quinnp added a comment.
Fixing indentation.
Repository:
rG LLVM Git