[PATCH] Clang: Enable -ffunction-sections and -fdata-sections for CloudABI by default

Eric Christopher echristo at gmail.com
Thu Mar 26 10:49:51 PDT 2015


I guess, but you could just make it as simple as "CloudABI defaults to
-ffunction-sections/-fdata-sections" since the symbol lookup is a bit of a
red herring there (you can make -ffunction-sections work without it, you
just have to mark your functions with the appropriate visibility).

-eric

On Thu, Mar 26, 2015 at 10:34 AM Ed Schouten <ed at nuxi.nl> wrote:

> In http://reviews.llvm.org/D8635#147417, @echristo wrote:
>
> > The static linkage part is somewhat irrelevant in the comment, but with
> a simplified and rewritten comment this is OK.
>
>
> Thanks! I guess the following comment looks simple enough:
>
>   // CloudABI does not support symbol lookups at run-time. The linker
>   // can safely garbage collect at the symbol level.
>
>
> REPOSITORY
>   rL LLVM
>
> http://reviews.llvm.org/D8635
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150326/759e5ce1/attachment.html>


More information about the cfe-commits mailing list