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

via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 7 20:57:33 PDT 2024


DianQK wrote:

> > 1. run these three transforms of SimplifyCFG rightly before CodeGenPrepare if the target supports conditional faulting, to avoid the awkwardness?
> > 2. extend LLVM IR to support scalar versioned masked.load/store.
> > 3. enhance the support for masked loads/stores for 1x %T throughout the middle-end passes after SimplifyCFG
> > 
> > dtcxzyw: I prefer the second one.
> 
> @nikic @DianQK Vote?

Sorry, I don't really have any good suggestions. :) But from the discussion above, it looks like we still have a lot of work to do. I would suggest going with the first one, and then gradually working through the second one and the various mis-compilations. At least that seems smoother?

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


More information about the llvm-commits mailing list