[PATCH] D93586: [InstCombine] use poison as placeholder for undemanded elems

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 27 13:15:25 PST 2020


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp:1073
     for (unsigned i = 0; i != VWidth; ++i) {
       if (!DemandedElts[i]) {   // If not demanded, set to undef.
+        Elts.push_back(Poison);
----------------
undef -> poison


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93586/new/

https://reviews.llvm.org/D93586



More information about the llvm-commits mailing list