[clang] Diagnose potential size confusion with VLA params [contd.] (PR #181550)

[Aaron Ballman via cfe-commits]

AaronBallman wrote:

> Note that this confusion in principle applies to any use of any identifier, not just constants but also others in sizeof / typeof.

It does, but I think `-Wshadow` is the general purpose diagnostic used to find those kinds of potential naming surprises.

I see the purpose to this diagnostic as being more specific to situations where potential changes to lookup rules (allowing for late parsing of function parameter names) might cause behavioral changes for users. But I can see why that might be too narrow of a focus.

Where do you see the line being drawn between the two given that they're related?

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