[PATCH] D144903: [X86] Drop single use check for freeze(undef) in LowerAVXCONCAT_VECTORS

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 9 12:28:23 PST 2023


efriedma added a comment.

I think the point of the hasOneUse check is to avoid a possible miscompile; if a FREEZE has more than one use, all users need to see the same value.  So not sure dropping the check is correct in general.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144903



More information about the llvm-commits mailing list