[PATCH] D56406: Remove check for single use in ShrinkDemandedConstant

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 9 13:09:55 PST 2019


fhahn added a comment.

In D56406#1350562 <https://reviews.llvm.org/D56406#1350562>, @arsenm wrote:

> LGTM
>
> In D56406#1348948 <https://reviews.llvm.org/D56406#1348948>, @fhahn wrote:
>
> > LGTM, thanks, but it would be good if someone more familiar with AMDGPU could also have a quick look.
>
>
> Can you commit a regression testcase for this?


Thanks for the patch :) I've committed rL350684 <https://reviews.llvm.org/rL350684> with a test case where we have an op with multiple users and a constant can be shrunk.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56406





More information about the llvm-commits mailing list