HighCommander4 wrote: > I have mixed feelings about this. The change is small, but it seems tied to a > particular coding style, and the implementation appears to be "optimized" > specifically for that style, which isn’t we do generally.
Would you feel any differently if the implementation was generalized to detect and ignore a one-character prefix from either operand of the comparison, so that e.g. `mFoo: kFoo` would be considered a match as well? https://github.com/llvm/llvm-project/pull/119162 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits