This reduces a duplicate condition on tool type and removes a deeply
nested ternary.
Author: Keith Smiley <keithbsmiley@gmail.com>
Paths
| Differential D89713
[llvm-objcopy][NFC] extract arg parsing logic to function ClosedPublic Authored by keith on Oct 19 2020, 10:10 AM.
Details Summary This reduces a duplicate condition on tool type and removes a deeply Author: Keith Smiley <keithbsmiley@gmail.com>
Diff Detail
Event TimelineComment Actions Looks good, with one suggestion. Also, couple of notes about the summary:
This revision is now accepted and ready to land.Oct 20 2020, 1:21 AM keith retitled this revision from llvm-objcopy: extract arg parsing logic to function to [llvm-objcopy] extract arg parsing logic to function.Oct 20 2020, 8:30 AM
Comment Actions
Summary == patch/commit title, i.e. the full title should be "[llvm-objcopy][NFC] extract arg parsing logic to function" (or something like that). keith retitled this revision from [llvm-objcopy] extract arg parsing logic to function to [llvm-objcopy][NFC] extract arg parsing logic to function.Oct 21 2020, 9:20 AM This revision was landed with ongoing or failed builds.Oct 22 2020, 11:33 PM Closed by commit rG310a8e1d1284: [llvm-objcopy][NFC] Extract arg parsing logic into a helper function (authored by keith, committed by alexander-shaposhnikov). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 299088 llvm/tools/llvm-objcopy/llvm-objcopy.cpp
|
nit: static (since this function is not used outside o llvm-objcopy.cpp)