[clang] [compiler-rt] [llvm] [ASan] Add metadata to renamed instructions so ASan doesn't use the i… (PR #119387)

[via cfe-commits]

================
@@ -137,6 +137,10 @@ llvm::AllocaInst 
*CodeGenFunction::CreateTempAlloca(llvm::Type *Ty,
     Alloca =
         new llvm::AllocaInst(Ty, CGM.getDataLayout().getAllocaAddrSpace(),
                              ArraySize, Name, AllocaInsertPt->getIterator());
+  if (Alloca->getName() != Name.str() &&
----------------
gbMattN wrote:

I think they want the `Alloca->getName() != Name.str()` check removed, so that 
when running ASan, we emit this name metadata on every `AllocaInst`
I thought it would be nice to reduce the amount of additional metadata emitted, 
but I don't have particularly strong feelings on the matter

https://github.com/llvm/llvm-project/pull/119387
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits