[PATCH] D88432: [InstCombine] Fix pr47668
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 28 10:52:37 PDT 2020
lebedev.ri added a comment.
This is missing alive link.
I think the fix is wrong, based solely on the fact that it is huge.
I would think the fix should be:
- if the shift amount is a variable, check that it is not undef
- if it is a constant, sanitize undef elts to 0
Am i wrong?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88432/new/
https://reviews.llvm.org/D88432
More information about the llvm-commits
mailing list