This is an archive of the discontinued LLVM Phabricator instance.

[ELF] - Remove ScriptLexer::Error field and check ErrorCount instead.
ClosedPublic

Authored by grimar on Aug 3 2017, 5:14 AM.

Details

Summary

D35945 introduces change when there is useless to check Error flag
in few places, but ErrorCount must be checked instead.

But then we probably can just check ErrorCount always. That should simplify
things. Patch do that.

Diff Detail

Event Timeline

grimar created this revision.Aug 3 2017, 5:14 AM
ruiu accepted this revision.Aug 4 2017, 2:23 AM

LGTM

This revision is now accepted and ready to land.Aug 4 2017, 2:23 AM
This revision was automatically updated to reflect the committed changes.