[PATCH] D45367: [CodeGen/AccelTable]: Don't emit accelerator entries for functions with no names

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 6 07:39:36 PDT 2018


JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.

I agree,  let's keep the linkage name in there.  LGTM.



================
Comment at: lib/CodeGen/AsmPrinter/DwarfDebug.cpp:389-391
 // TODO: Determine whether or not we should add names for programs
 // that do not have a DW_AT_name or DW_AT_linkage_name field - this
 // is only slightly different than the lookup of non-standard ObjC names.
----------------
labath wrote:
> I'm not sure what this TODO here is trying to say, but it could be that this patch resolves it...
Sounds like it :-) 


Repository:
  rL LLVM

https://reviews.llvm.org/D45367





More information about the llvm-commits mailing list