[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