[llvm] [IR] Add support for ignoring poisons in `getSplatValue` (PR #89155)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 17 17:24:51 PDT 2024
nikic wrote:
I don't think this is the change we want to do. I've posted an alternative at https://github.com/llvm/llvm-project/pull/89159.
Basically:
* In the spirit of https://github.com/llvm/llvm-project/pull/88217, we should remove support for undef splats to the degree possible, not support both.
* Allowing poison splats by default in m_APInt/m_APFloat is a separate change, and requires at least some due diligence (at least a cursory review of select combines, where this is most likely to cause issues.)
https://github.com/llvm/llvm-project/pull/89155
More information about the llvm-commits
mailing list