[PATCH] D121869: [WebAssembly] Disable SimplifyDemandedVectorElts after legalization
    Thomas Lively via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Mar 16 18:39:40 PDT 2022
    
    
  
tlively accepted this revision.
tlively added a comment.
This revision is now accepted and ready to land.
LGTM! This seems fairly reasonable as a solution, all things considered. Have you been able to check whether this has any broad impacts on the quality of our codegen? My guess is that it doesn't since it seems that any code that would have had the vector elements simplified previously would have ended up in that infinite loop, but we went a long time without ever seeing that bug in the wild.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121869/new/
https://reviews.llvm.org/D121869
    
    
More information about the llvm-commits
mailing list