[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