This is an archive of the discontinued LLVM Phabricator instance.

Add support for mllvm to the MinGW driver
ClosedPublic

Authored by martell on Sep 11 2017, 2:13 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

martell created this revision.Sep 11 2017, 2:13 PM
ruiu added inline comments.Sep 11 2017, 2:14 PM
MinGW/Driver.cpp
162–163 ↗(On Diff #114690)

Can this appear more than once?

martell added inline comments.Sep 11 2017, 2:18 PM
MinGW/Driver.cpp
162–163 ↗(On Diff #114690)

Yes,

both ELF/Driver.cpp and COFF/Driver.cpp handle it like this

std::vector<const char *> V;
V.push_back("lld-link (LLVM option parsing)");
for (auto *Arg : Args.filtered(OPT_mllvm))
  V.push_back(Arg->getValue());
cl::ParseCommandLineOptions(V.size(), V.data());
ruiu accepted this revision.Sep 11 2017, 2:20 PM

LGTM

This revision is now accepted and ready to land.Sep 11 2017, 2:20 PM
This revision was automatically updated to reflect the committed changes.