[PATCH] D56392: [LLD][COFF] Support /ignore:4099. Support /ignore with comma-separated arguments.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 7 13:20:07 PST 2019


ruiu accepted this revision.
ruiu added a comment.

LGTM



================
Comment at: test/COFF/ignore-many.yaml:1
+Tests /ignore with more than one argument.
+
----------------
This file is not a YAML file, so let's not use the .yaml file extension. Please name this ignore-many.test.


================
Comment at: test/COFF/ignore-many.yaml:17
+SUPPRESSED-NOT: warning: Cannot use debug info for
+
----------------
Remove the extraneous last '\n'. (Your file ends with "\n\n", so you should remove the last '\n").


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D56392





More information about the llvm-commits mailing list