[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