[PATCH] D75224: [SelectionDAG][FPEnv] Take into account SelectionDAG continuous CSE when setting the nofpexcept flag for constrained intrinsics

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 17 16:14:03 PDT 2020


craig.topper updated this revision to Diff 250936.
craig.topper added a comment.

Fix typo in comment.

For reductions, do you think we can just check that the reduction flag is already set? Or do we need to just excluse all BinaryOperators from the geneirc handler and just fully manage FMF for BinaryOperators in its visitor function?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75224/new/

https://reviews.llvm.org/D75224

Files:
  llvm/include/llvm/CodeGen/SelectionDAG.h
  llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
  llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  llvm/test/CodeGen/X86/fp-intrinsics-flags.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75224.250936.patch
Type: text/x-patch
Size: 6395 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200317/85150a49/attachment.bin>


More information about the llvm-commits mailing list