[PATCH] D93455: Constrained fp OpBundles

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 17 09:16:11 PST 2020


kpn added a comment.

Do you need tests for the verifier part?

Would replacing some of the constrained intrinsics with operand bundles on other intrinsics have the same safety guarantee in the middle end? Currently the middle end doesn't know about the constrained intrinsics so it handles code conservatively. What about intrinsics the middle end already knows about? Would the new operand bundle be as safe as the constrained intrinsics are now?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93455



More information about the llvm-commits mailing list