[PATCH] D49366: Add support for /FUNCTIONPADMIN command-line option
Stefan Reinalter via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 17 06:13:48 PDT 2018
stefan_reinalter marked 3 inline comments as done.
stefan_reinalter added inline comments.
================
Comment at: COFF/DriverUtils.cpp:271-273
+ } else {
+ error("/functionpadmin: invalid argument for this machine: " + Arg);
+ }
----------------
pcc wrote:
> stefan_reinalter wrote:
> > zturner wrote:
> > > What about ARM64? The documentation doesn't say what the default value is here, but it should be easy to figure out.
> > Understood, I will try figuring this out.
> And ARM32.
When building for ARM using MSVC/link.exe, using /FUNCTIONPADMIN yields:
LINK : fatal error LNK1146: no argument specified with option '/FUNCTIONPADMIN'
Seems like there is no default for these platforms.
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D49366
More information about the llvm-commits
mailing list