This patch fixes incorrect array name generation for a cppcoreguidelines-pro-bounds-constant-array-index warning.
For now I just updated tests. The problem is in BaseRange definition, as it holds EndLoc and BeginLoc pointing to the beginning of ArrayExpression base https://github.com/llvm-mirror/clang-tools-extra/blob/e0441f6939da38f26bea9c1d75bb33024daa4e40/clang-tidy/cppcoreguidelines/ProBoundsConstantArrayIndexCheck.cpp#L78. I'm investigating this.
Thank you for working on this!
Could you please run the check over LLVM or any other significant codebase once you have the fix implemented and report if the transformation still breaks code?