[llvm] [X86, SimplifyCFG] Support hoisting load/store with conditional faulting (Part II) (PR #108812)

[Phoebe Wang via llvm-commits]

phoebewang wrote:

> Good knowledge! Then it seems this optimization for this case w/ CFCMOV does not have any value...

No, they are different optimizations. They just have intersection in the specific example. The existing one do nothing with the test `single_predecessor`.

> We just need to add a tuning knob for `NumPHIInsts <= ProfitableToSinkInstructionThreshold`, from my perspective.

Yes, we can do that. I'll create one then :)

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