[PATCH] D146350: [InstCombine] More aggressively try and fold irem/idiv/mul into selects.
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 30 03:23:58 PDT 2023
nikic added a comment.
In D146350#4222729 <https://reviews.llvm.org/D146350#4222729>, @goldstein.w.n wrote:
> In D146350#4204295 <https://reviews.llvm.org/D146350#4204295>, @nikic wrote:
>
>> This is not generally legal for div/rem, see https://alive2.llvm.org/ce/z/2iiR26 using one of your tests. This is only legal if the operation is speculatable.
>
> Is there a flag for speculatable?
There is isSafeToSpeculativelyExecute().
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146350/new/
https://reviews.llvm.org/D146350
More information about the llvm-commits
mailing list