[PATCH] D105984: [PowerPC] Restore FastMathFlags of Builder for Vector FDiv Builtins

2021-07-20 Thread Kamau Bridgeman via Phabricator via cfe-commits
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

[PATCH] D105984: [PowerPC] Restore FastMathFlags of Builder for Vector FDiv Builtins

2021-07-16 Thread Quinn Pham via Phabricator via cfe-commits
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

[PATCH] D105984: [PowerPC] Restore FastMathFlags of Builder for Vector FDiv Builtins

2021-07-16 Thread Nemanja Ivanovic via Phabricator via cfe-commits
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 __

[PATCH] D105984: [PowerPC] Restore FastMathFlags of Builder for Vector FDiv Builtins

2021-07-15 Thread Quinn Pham via Phabricator via cfe-commits
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

[PATCH] D105984: [PowerPC] Restore FastMathFlags of Builder for Vector FDiv Builtins

2021-07-15 Thread Quinn Pham via Phabricator via cfe-commits
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

[PATCH] D105984: [PowerPC] Restore FastMathFlags of Builder for Vector FDiv Builtins

2021-07-15 Thread Quinn Pham via Phabricator via cfe-commits
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

[PATCH] D105984: [PowerPC] Restore FastMathFlags of Builder for Vector FDiv Builtins

2021-07-14 Thread Qiu Chaofan via Phabricator via cfe-commits
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

[PATCH] D105984: [PowerPC] Restore FastMathFlags of Builder for Vector FDiv Builtins

2021-07-14 Thread Quinn Pham via Phabricator via cfe-commits
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