[lld] r205163 - [core] support .gnu.linkonce sections

Shankar Easwaran shankare at codeaurora.org
Mon Mar 31 12:23:41 PDT 2014


On 3/31/2014 2:11 PM, Rui Ueyama wrote:
> I'm not sure if you get my suggestion. This suggestion is basically to 
> keep your code as is, except that replacing conditions inside if's 
> with atom->isComdatGroup(). That would look better than writing the 
> same conditions in many places. We already have a precedence of a 
> utility method that is to be used inside a if condition, which is 
> DefinedAtom::occupiesDiskSpace().
Ah!!! Again, .gnu.linkonce doesnot call for itself a comdatGroup, 
developers might get a wrong impression of calling .gnu.linkonce for a 
COMDAT group.

Thanks

Shankar Easwaran

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation




More information about the llvm-commits mailing list