[PATCH] D45576: [RFC] Allow target to handle STRICT floating-point nodes

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 24 05:07:01 PDT 2018


uweigand added a comment.

In https://reviews.llvm.org/D45576#1109818, @hfinkel wrote:

> To take a hint from the suggestion you relayed from Chris, how about we just add MMOs to these instructions, and then let mayLoad/mayStore look at optional MMOs when returning their answer? Maybe this lets us do what we want without duplicating all of the patterns?


That sound like a good idea.  I'll experiment with this approach a bit to see how far I can get.  I guess we'll have to review the optimizers to catch places that simply look at the mayLoad/mayStore flag without detailed checking of MMOs -- we certainly want to avoid regressing on current behavior in the default case.


Repository:
  rL LLVM

https://reviews.llvm.org/D45576





More information about the llvm-commits mailing list