[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