echristo added a comment.

In https://reviews.llvm.org/D24644#562286, @mehdi_amini wrote:

> What about function attributes? Hey that's the trend :)
>  You could have a subset of the functions in their own sections but not all. 
> With LTO it means that you can disable this for a single input file.


True, but we'd need data attributes too for -fdata-sections. That's the main 
reason I haven't migrated the options out of TargetOptions and into the IR 
here. Rough sketch on module merge time: We'd probably want to error on 
functions/data that had separate section set in one module but not in another - 
there are a few ways to make that not error at link time, but at that point 
you're really relying on weird linker side effects and it's probably not what 
you intended anyhow.

-eric


https://reviews.llvm.org/D24644



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to