[PATCH] D143593: [InstCombine] Don't fold freeze poison when it's used in shufflevector

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 12 03:44:41 PST 2023


RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstructionCombining.cpp:4024
+          match(U, m_Shuffle(m_Specific(&I), m_Value())))
+        return nullptr;
+    }
----------------
I'm worried this will start to  fail as soon as we have bitcasts inserted in the use chain somewhere - something that occurs a lot with x86 shuffles for instance.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143593



More information about the llvm-commits mailing list