This is an archive of the discontinued LLVM Phabricator instance.

[ELF] - Change all messages to be lowercase to be consistent.
ClosedPublic

Authored by grimar on Mar 11 2016, 7:07 AM.

Diff Detail

Event Timeline

grimar updated this revision to Diff 50428.Mar 11 2016, 7:07 AM
grimar retitled this revision from to [ELF] - Change all messages to be lowercase to be consistent..
grimar updated this object.
grimar added reviewers: ruiu, rafael.
grimar added subscribers: llvm-commits, grimar.
grimar added inline comments.
test/ELF/invalid-elf.test
22

I had to add INVALID-SECTION-INDEX-LLD check line. Because this text comes from lld,

26

while this comes from \llvm\lib\Object\error.cpp file

rafael accepted this revision.Mar 11 2016, 7:13 AM
rafael edited edge metadata.

LGTM

This revision is now accepted and ready to land.Mar 11 2016, 7:13 AM
This revision was automatically updated to reflect the committed changes.
ruiu added inline comments.Mar 11 2016, 10:25 AM
lld/trunk/ELF/InputFiles.cpp
439–465 ↗(On Diff #50436)

Isn't this an accidental change?

Gah, this basically reverted my patch.

Gah, this basically reverted my patch.

I am really sorry, don't sure how such thing happened.