[llvm] [Attributor] Fix an issue that an access is skipped by mistake (PR #101862)

Shilei Tian via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 5 12:39:37 PDT 2024


shiltian wrote:

> > > Testcase?
> > 
> > 
> > Well, the thing is, this can only be exposed in an Attributor use that doesn't have `AAInterFnReachability`. Regular `AttributorPass` will not have this issue. Only `AMDGPUAttributorPass` can do this, but the enabling of `AAIndirectCallInfo ` has been reverted so even that can't expose it.
> > What I can do I guess is still to include the test case in this PR and the enabling of `AAIndirectCallInfo ` in `AMDGPUAttributor` should not break it. Is that okay?
> 
> Pre-commit the test that would break in the next patch in its working state?

Hmm, I didn't follow.

https://github.com/llvm/llvm-project/pull/101862


More information about the llvm-commits mailing list