[PATCH] D127950: [SimplifyCFG] Check isSafeToSpeculativelyExecute before computeSpeculationCost

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 16 08:26:16 PDT 2022

Previous message: [PATCH] D127950: [SimplifyCFG] Check isSafeToSpeculativelyExecute before computeSpeculationCost
Next message: [PATCH] D127950: [SimplifyCFG] Check isSafeToSpeculativelyExecute before computeSpeculationCost
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

nikic added a comment.

@bcl5980 Conceptually, yes., but I believe your specific patch does not handle `sdiv (i64 EXPR, i64 -1)` correctly. We also only need this for the signed case.


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

https://reviews.llvm.org/D127950

Previous message: [PATCH] D127950: [SimplifyCFG] Check isSafeToSpeculativelyExecute before computeSpeculationCost
Next message: [PATCH] D127950: [SimplifyCFG] Check isSafeToSpeculativelyExecute before computeSpeculationCost
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list