================
@@ -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

Reply via email to