[PATCH] D56406: Check for single use in cvt_f32_ubyte combine instead of ShrinkDemandedConstant

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 8 03:58:20 PST 2019


arsenm requested changes to this revision.
arsenm added a comment.
This revision now requires changes to proceed.

Something seems wrong here. How/why is this valid on AArch64 in some situation? A test is really needed for whatever this regression is


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

https://reviews.llvm.org/D56406





More information about the llvm-commits mailing list