[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