[llvm] [X86][SimplifyCFG] Support hoisting load/store with conditional faulting (PR #96878)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 4 00:42:52 PDT 2024


dtcxzyw wrote:

> > @KanRobert As there are some potential miscompilations/crashes caused by other passes after SimplifyCFGPass, should we limit this optimization to only run in the late pipeline via `SimplifyCFGOptions`?
> 
> Do you mean sth like `FPM.addPass(SimplifyCFGPass(SimplifyCFGOptions().convertSwitchRangeToICmp(true)));`?

Yeah.

> If so, where in the pipeline do you suggest doing this? @dtcxzyw

See https://github.com/dtcxzyw/llvm-project/commit/3f407c4d30eb877653c07f00bdcbe3e04742f762.


https://github.com/llvm/llvm-project/pull/96878


More information about the llvm-commits mailing list