[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:55:17 PDT 2018


stefan_reinalter updated this revision to Diff 155875.
stefan_reinalter marked an inline comment as done.
stefan_reinalter added a comment.

I updated the implementation according to the recommendation by Peter. I also added a comment regarding /FUNCTIONPADMIN default padding on ARM platforms.
Also ran git-clang-format against the source files so that the diff shows only my changes.

Tests are still missing though, I'm waiting for your advice on this.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D49366

Files:
  COFF/Config.h
  COFF/Driver.cpp
  COFF/Driver.h
  COFF/DriverUtils.cpp
  COFF/Options.td
  COFF/Writer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49366.155875.patch
Type: text/x-patch
Size: 4301 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180717/147c14ae/attachment.bin>


More information about the llvm-commits mailing list