This is an archive of the discontinued LLVM Phabricator instance.

[LNT] Be less strict about argument ordering in FakeCompiler
ClosedPublic

Authored by chatur01 on Jun 10 2015, 10:05 AM.

Details

Summary

Modify the FakeCompiler driver to be less strict about argument order.

The existing checks in the driver would fail when called from compilers.py if any extra cflags had been given on the lnt command line.

Instead, settle for looser definitions of what constitutes the trigger conditions for printing out version info:

  • I moved dumpmachine to the first checked option, because in the official driver, this option overrules all others if given anywhere on the command-line.
  • If both -v and -### occur in any order in the command line, dump the verbose information.
  • If -Wa,-v occurs anywhere, dump the assembler version.
  • If -Wl,-v occurs anywhere, dump the linker version.
  • If all of -S, -flto, -o, -, /dev/null are in the command-line, in any order, then dump the target information.

Diff Detail

Event Timeline

chatur01 retitled this revision from to [LNT] Be less strict about argument ordering in FakeCompiler.
chatur01 updated this object.
chatur01 edited the test plan for this revision. (Show Details)
chatur01 added a subscriber: Unknown Object (MLST).
cmatthews accepted this revision.Jun 10 2015, 10:12 AM
cmatthews edited edge metadata.

Besides above comment, LGTM.

tests/SharedInputs/FakeCompilers/fakecompiler.py
171

Make a sentence, with a .

This revision is now accepted and ready to land.Jun 10 2015, 10:12 AM
chatur01 closed this revision.Jun 11 2015, 2:35 AM

Thanks Chris! Landed r239517.