[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