[PATCH] D49366: Add support for /FUNCTIONPADMIN command-line option

[Stefan Reinalter via Phabricator via llvm-commits]

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