lkml.org 
[lkml]   [2020]   [Jun]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH 17/22] arm64: vdso: disable LTO
    On Wed, Jun 24, 2020 at 2:52 PM Sami Tolvanen <samitolvanen@google.com> wrote:
    >
    > On Wed, Jun 24, 2020 at 01:58:57PM -0700, 'Nick Desaulniers' via Clang Built Linux wrote:
    > > On Wed, Jun 24, 2020 at 1:33 PM Sami Tolvanen <samitolvanen@google.com> wrote:
    > > >
    > > > Filter out CC_FLAGS_LTO for the vDSO.
    > >
    > > Just curious about this patch (and the following one for x86's vdso),
    > > do you happen to recall specifically what the issues with the vdso's
    > > are?
    >
    > I recall the compiler optimizing away functions at some point, but as
    > LTO is not really needed in the vDSO, it's just easiest to disable it
    > there.

    Sounds fishy; with extern linkage then I would think it's not safe to
    eliminate functions. Probably unnecessary for the initial
    implementation, and something we can follow up on, but always good to
    have an answer to the inevitable question "why?" in the commit
    message.
    --
    Thanks,
    ~Nick Desaulniers

    \
     
     \ /
      Last update: 2020-06-25 01:07    [W:3.024 / U:0.692 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site