Page MenuHomePhabricator

[lld] Remove 2>&1 from --trace-symbol tests

Authored by sbc100 on May 14 2019, 7:43 PM.



The tracing goes to stdout so this is not needed.

Also remove the "not" from the final check in ELF/trace-symbols.s.
According the comment the check is that we don't crash, so we should
be checking for success here. Previously this step is erroring with
undefined symbols because it didn't include all the needed objects.

Diff Detail

rLLD LLVM Linker

Event Timeline

sbc100 created this revision.May 14 2019, 7:43 PM
Herald added a project: Restricted Project. · View Herald Transcript
sbc100 added a reviewer: ruiu.May 14 2019, 7:44 PM
sbc100 retitled this revision from [LLD] Remove 2>&1 from --trace-symbol tests to [lld] Remove 2>&1 from --trace-symbol tests.
sbc100 edited the summary of this revision. (Show Details)May 14 2019, 8:13 PM
sbc100 updated this revision to Diff 199550.May 14 2019, 8:13 PM
  • Fix special symbol crash
ruiu accepted this revision.May 14 2019, 10:54 PM


This revision is now accepted and ready to land.May 14 2019, 10:54 PM
This revision was automatically updated to reflect the committed changes.