[PATCH] D115352: [AArch64] Add mayRaiseFPException to appropriate instructions

John Brawn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 31 06:48:59 PDT 2022


john.brawn added a comment.

In D115352#3399965 <https://reviews.llvm.org/D115352#3399965>, @dmgreen wrote:

> The original version of this patch was altering the non-strict codegen because the flags were lost in the machine combiner. Have you done any other testing to make sure that doesn't happen anywhere else? To make sure this doesn't pessimise the performance for the default non-strict case.

I've build all of the MultiSource and SingleSource tests in llvm-test-suite with and without this patch (and those it depends on) at -O3 and there are no changes to the generated code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115352



More information about the llvm-commits mailing list