[PATCH] D143954: [ValueTracking] It is not safe to execute FDIV/FREM speculatively

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 14 03:51:44 PST 2023


nikic requested changes to this revision.
nikic added a comment.
This revision now requires changes to proceed.
Herald added a subscriber: StephenFan.

As pointed out by @craig.topper, the premise of this patch is incorrect. We can't (generally) speculate a constrained fdiv intrinsic, but we already don't do that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143954



More information about the llvm-commits mailing list