[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

[Artem Dergachev via cfe-commits]

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