[PATCH] D18706: [DWARF] Force a linkage_name on an inlined subprogram's abstract origin

Paul Robinson via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 13 14:54:58 PDT 2016


probinson added a comment.

In http://reviews.llvm.org/D18706#400343, @probinson wrote:

> Try to do fewer lookups.


This means every call to addLinkageName has to be guarded.  When I did the option originally (as an on/off), Eric was not a fan of guarding the calls, which is why it ended up inside the method. But now the condition is different in the two places that call the method, so maybe that's okay?


http://reviews.llvm.org/D18706





More information about the llvm-commits mailing list