This is an archive of the discontinued LLVM Phabricator instance.

[test/Object/nm-trivial-object.test] - Remove 4 precompiled binaries.
ClosedPublic

Authored by grimar on Jul 5 2019, 6:31 AM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

grimar created this revision.Jul 5 2019, 6:31 AM
MaskRay added inline comments.Jul 7 2019, 10:12 PM
test/Object/nm-trivial-object.test
88 ↗(On Diff #208166)

What's the difference of textSym1 and textSym2?

grimar marked an inline comment as done.Jul 8 2019, 4:16 AM
grimar added inline comments.
test/Object/nm-trivial-object.test
88 ↗(On Diff #208166)

The intention was the same as for having dataSym1 vs dataSym2: to show that different address is printed for symbols with different values.
That is what original ELF-SEC-ADDR64 test case did I think.

grimar edited the summary of this revision. (Show Details)Jul 8 2019, 10:08 AM
jhenderson accepted this revision.Jul 9 2019, 4:36 AM

LGTM, with one comment.

test/Object/nm-trivial-object.test
80 ↗(On Diff #208166)

A comment describing this test case might be nice. Perhaps "Test different ELF symbols for 64-bit platform."

This revision is now accepted and ready to land.Jul 9 2019, 4:36 AM
grimar retitled this revision from [test/Object/nm-trivial-object.test] - Remove 5 precompiled binaries. to [test/Object/nm-trivial-object.test] - Remove 4 precompiled binaries..Jul 10 2019, 1:37 AM
grimar edited the summary of this revision. (Show Details)

At last moment I found IsNAN.o is used by another test case. I'll keep this object for now.

grimar edited the summary of this revision. (Show Details)Jul 10 2019, 1:39 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJul 10 2019, 1:42 AM