================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to