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

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 30 20:37:38 PDT 2024


dtcxzyw wrote:

> > @KanRobert I think this patch is far away from being ready for review. Please make sure that this patch doesn't cause crashes or miscompilation on the following benchmarks:
> > 
> > * [ ]   Clang stage2 build
> > * [ ]   SPEC 2006/2017
> > * [ ]   llvm-test-suite
> > * [ ]   https://github.com/dtcxzyw/llvm-opt-benchmark
> > * [ ]   fuzzers (e.g., csmith)
> 
> Not familiar with other benchmarks. But I will validate SPEC2017 + llvm-test-suite on intel SDE with this PR, and comment the result on this page.

Great! As we have a similar prototype on RISCV, I can validate this patch with fuzzers :)



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


More information about the llvm-commits mailing list