Cleanup else-after-return and add an early-return to llvm-nm
authorDavid Blaikie <dblaikie@gmail.com>
Mon, 23 Mar 2015 21:17:43 +0000 (21:17 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Mon, 23 Mar 2015 21:17:43 +0000 (21:17 +0000)
commit328c0c11a8033fa6b8d716e6a08be3e07a20a1af
tree2d832675533baa10c8e9fd5875a9bb4d83c49a1c
parentd1655cb1c02a8d7f97fb654a91c9bb12ac8bb74e
Cleanup else-after-return and add an early-return to llvm-nm

The loop and error handling in checkMachOAndArchFlags didn't make sense
to me (a loop that only ever executes once? An error path that uses the
element the loop stopped at (which must always be a buffer overrun if
I'm reading that right?)... I'm confused) but I've made a guess at what
was intended.

Based on a patch by Richard Thomson to simplify boolean expressions.

llvm-svn: 233025
llvm/tools/llvm-nm/llvm-nm.cpp