[PATCH] D62124: [LLD][COFF] Implement /filealign parameter

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 20 22:03:12 PDT 2019


ruiu added a comment.

Th



================
Comment at: lld/COFF/Driver.cpp:1190
+  if (auto *Arg = Args.getLastArg(OPT_filealign))
+    parseNumbers(Arg->getValue(), &Config->FileAlign);
+
----------------
Thank you for your explanation. One last thing -- can you add a check to make sure that a parsed value is a power of two?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62124/new/

https://reviews.llvm.org/D62124





More information about the llvm-commits mailing list