[PATCH] D56087: [TargetLowering][AMDGPU] Remove the SimplifyDemandedBits function that takes a User and OpIdx. Stop using it in AMDGPU target for simplifyI24.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 7 06:45:18 PST 2019


RKSimon added a comment.

In D56087#1348138 <https://reviews.llvm.org/D56087#1348138>, @arsenm wrote:

> LGTM. The immediate change is a little worse though, since the -7 is free and 0xfffff9 is not


I'm not certain whether we need the constant simplification in GetDemandedBits at all to be honest, but that is a separate issue.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56087





More information about the llvm-commits mailing list