[PATCH] D105895: [FPEnv][EarlyCSE] Add support for CSE of constrained FP intrinsics

[Kevin P. Neal via Phabricator via llvm-commits]

kpn added a comment.

This patch may be overkill. If the reading of the floating point status register is restricted to "fpexcept.strict" then perhaps we don't need all of this code. But if we want to allow checking the status register after "fpexcept.maytrap" instructions then we do need all this. The LangRef restricts those reads to "fpexcept.strict". Are we certain we want to stick with that language?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105895