================ @@ -2856,7 +2916,7 @@ getFixIts(FixableGadgetSets &FixablesForAllVars, const FixitStrategy &S, } #ifndef NDEBUG Handler.addDebugNoteForVar( - VD, F->getBaseStmt()->getBeginLoc(), + VD, F->getSourceLoc(), ---------------- haoNoQ wrote:
This is the only place where we do any actual virtual dispatch over `getSourceLoc()` right? Otherwise we could merge it into `handleUnsafeOperation()` too. Maybe with a slightly different design we could eliminate this too. Dunno, I don't have anything specific in mind. https://github.com/llvm/llvm-project/pull/91777 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits