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

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 3 05:14:51 PDT 2017


grimar created this revision.
Herald added a subscriber: emaste.

https://reviews.llvm.org/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.


https://reviews.llvm.org/D36266

Files:
  ELF/ScriptLexer.cpp
  ELF/ScriptLexer.h
  ELF/ScriptParser.cpp
  test/ELF/version-script-err.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36266.109526.patch
Type: text/x-patch
Size: 7251 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170803/0bdf0c14/attachment.bin>


More information about the llvm-commits mailing list