[PATCH] D112256: [FPEnv][EarlyCSE] Add support for CSE of constrained FP intrinsics, take 2

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 31 12:25:32 PDT 2022


spatel added inline comments.


================
Comment at: llvm/test/Transforms/EarlyCSE/tfpropagation.ll:125
 
 define double @branching_maytrap(i64 %a) #0 {
 ; CHECK-LABEL: @branching_maytrap(
----------------
Am I seeing this correctly - all tests now transform?
Should there be tests where one or both of the constrained ops is "fpexcept.strict"?
Please add some explanatory comments for these tests.


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

https://reviews.llvm.org/D112256



More information about the llvm-commits mailing list