[PATCH] D53227: [hwasan] add stack frame descriptions.

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 22 16:28:41 PDT 2018


eugenis added inline comments.


================
Comment at: lib/Transforms/Instrumentation/HWAddressSanitizer.cpp:742
+  // Put GV into the F's Comadat so that if F is deleted GV can be deleted too.
+  if (!F.getName().startswith("hwasan"))
+    if (auto Comdat = GetOrCreateFunctionComdat(F, CurModuleUniqueId))
----------------
kcc wrote:
> eugenis wrote:
> > Why would F's name start with "hwasan"? Did you mean "__hwasan"?
> No, this one is for the CTOR (HwasanCtorFunction above)
Why not compare F with HwasanCtorFunction?


Repository:
  rL LLVM

https://reviews.llvm.org/D53227





More information about the llvm-commits mailing list