[PATCH] D12763: [PATCH] [compiler-rt] [sanitizers] Add VMA size check at runtime

Kristof Beyls via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 11 02:32:56 PDT 2015


kristof.beyls added a comment.

In http://reviews.llvm.org/D12763#244164, @rengolin wrote:

> In http://reviews.llvm.org/D12763#244146, @kristof.beyls wrote:
>
> > I don't see why name mangling would add run-time overhead - but it would avoid having to build different variants of compiler-rt for AArch64.
> >  If name mangling does add run-time overhead, could you explain why?
>
>
> I'd rather discuss this outside of this patch, as it is just a safety check, not a decision making change. Feel free to send an email to the list if you want to start the discussion now.


Fair enough - I hadn't noticed we're  already requiring different compiler-rt builds for different variants of AArch64-linux.


http://reviews.llvm.org/D12763





More information about the llvm-commits mailing list