Page MenuHomePhabricator

[ELF, PR27091] - Implemented -t/--trace option
ClosedPublic

Authored by grimar on Mar 28 2016, 11:13 AM.

Details

Summary

-t
--trace
Print the names of the input files as ld processes them.
This fixes https://llvm.org/bugs/show_bug.cgi?id=27091.

At first when I saw that Config->Verbose already can be used to print input file list,
I thought that there is no need to add new option, but then realized that

  1. Users does not wan't to see all what Verbose prints when they want to see input files only.
  2. That is consistent with gold and can be already used by someone.

So that is good option to have I think.

Diff Detail

Repository
rL LLVM

Event Timeline

grimar updated this revision to Diff 51807.Mar 28 2016, 11:13 AM
grimar retitled this revision from to [ELF, PR27091] - Implemented -t/--trace option.
grimar updated this object.
grimar added reviewers: ruiu, rafael.
grimar added subscribers: llvm-commits, grimar.
emaste added a subscriber: emaste.Mar 28 2016, 11:16 AM

agree this is nice to have

ruiu accepted this revision.Mar 28 2016, 11:39 AM
ruiu edited edge metadata.

LGTM

test/ELF/trace.s
11–20 ↗(On Diff #51807)

We don't care about file output order, so I wouldn't add them.

This revision is now accepted and ready to land.Mar 28 2016, 11:39 AM
This revision was automatically updated to reflect the committed changes.