[clang] [llvm] [analysis] Software Bill of Mitigations (PR #130103)

[Oskar Wirga via llvm-commits]

================
@@ -2512,6 +2516,28 @@ void CodeGenModule::SetLLVMFunctionAttributesForDefinition(const Decl *D,
   else if (isStackProtectorOn(LangOpts, getTriple(), LangOptions::SSPReq))
     B.addAttribute(llvm::Attribute::StackProtectReq);
 
+  bool noStackProtectionAttr = D && D->hasAttr<NoStackProtectorAttr>();
----------------
oskarwirga wrote:

We should either refactor our approach to stack protectors or add comment explaining the caveat here that our mitigation analysis pass runs before the stack protectors are lowered so this is more of a heuristic rather than actual hard signal. 

https://github.com/llvm/llvm-project/pull/130103