[PATCH] D70162: Implement /driver, /driver:wdm and /driver:uponly

[Nico Weber via Phabricator via llvm-commits]

thakis accepted this revision.
thakis added a comment.
This revision is now accepted and ready to land.

Looks good, some suggestions on additional things that might be interesting to check below.

Did someone ask for this, or is this just for fun / completeness? (Either's fine of course, just curious)



================
Comment at: lld/test/COFF/driver-opt.s:4
+
+# RUN: rm -f %t.dir/foo.sys
+# RUN: cd %t.dir; lld-link /driver foo.obj
----------------
What does link.exe do with `/fixed /driver` and `/driver /fixed`?


================
Comment at: lld/test/COFF/driver-opt.s:20
+# RUN: rm -f %t.dir/foo.sys
+# RUN: cd %t.dir; lld-link /driver:wdm foo.obj
+# RUN: llvm-readobj --file-headers %t.dir/foo.sys | FileCheck -check-prefix=WDM %s
----------------
What does link.exe say to `/driver:uponly,wdm` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70162