[llvm] [LLVM][Transforms][Attributor] - Add configurable access cap for AAPointerInfo (PR #190105)

[Pranav Bhandarkar via llvm-commits]

bhandarkar-pranav wrote:

> Thanks for this work @bhandarkar-pranav. It looks good to me module these 2 questions.
> 
> 1. Is there is a way to write a test for this new flag?
Let me explore this.

> 2. I was wondering if 0 is the right default. Although it maintains the current behavior, but it will require user to know about this flag if they are experiencing a slow down at link time. Would a higher value, where this quadratic behavior starts biting, be a better default?

Yes, you are right in that 0 was chosen as the default because it maintains current behavior. The rationale for not picking a higher value as the default, other than keeping current behavior, was that there already is at least one other such flag (albeit in `OpenMPOpt.cpp` and not the attributor that the user might will need to know about if they start experiencing long link times (`openmp-opt-max-iterations`). Having said that if you strongly prefer, I am happy to change the default to a higher value (my preference would be 512 - see table above).



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