[PATCH] D121157: [AMDGPU] always use underlying object in the pointsToConstantMemory

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 7 14:13:10 PST 2022


arsenm added a comment.

In D121157#3365530 <https://reviews.llvm.org/D121157#3365530>, @rampitec wrote:

> @arsenm Do you think it makes sense to check if getUnderlyingObject actually brought one of: Argument, GlobalValue, or ConstantAggregate?

I would assume the base implementation handles that, but we're already checking for GlobalValue


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

https://reviews.llvm.org/D121157



More information about the llvm-commits mailing list