[PATCH] D41318: Start setting dso_local in clang

Rafael Avila de Espindola via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 22 08:36:30 PST 2017


Is this direction OK? Should a put the time to update the existing tests
to account for dso_local?

I do volunteer to implement the rest of ELF, COFF and MachO once this is


Rafael Avila de Espindola <rafael.espindola at gmail.com> writes:

> Reid Kleckner via Phabricator <reviews at reviews.llvm.org> writes:
>> rnk added inline comments.
>> ================
>> Comment at: lib/CodeGen/CodeGenModule.cpp:690-692
>> +  // Only handle ELF for now.
>> +  if (!CGM.getTriple().isOSBinFormatELF())
>> +    return false;
>> ----------------
>> Handling COFF here is probably trivial. Everything is dso_local unless it's dllimport. Does that matter, or is dso_local intended to be an ELF-specific annotation?
> The dso_local attribute is for all file formats. This patch is ELF only
> in the interest of being incremental.
> In addition of what this patch shows there will be a lot of test updates
> because of dso_local showing up, so splitting up the patches is probably
> a good thing.
> Cheers,
> Rafael

More information about the cfe-commits mailing list