projects / platform / upstream / llvm.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: e86c8d3) | patch
NFC: DebugInfo: Reduce scope of DebugOffset to simplify code
authorDavid Blaikie <dblaikie@gmail.com>
Mon, 12 Nov 2018 18:53:28 +0000 (18:53 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Mon, 12 Nov 2018 18:53:28 +0000 (18:53 +0000)
commit582a5ebce065c19cadcef2334f995698bb306275
tree6e4656f7523d3f46840c591755cca166ddfcd8datree | snapshot
parente86c8d33b1d3eacaa616e3d8a6c99638cbe3f9e3commit | diff
NFC: DebugInfo: Reduce scope of DebugOffset to simplify code

This was being used as a sort of indirect out parameter from shouldDump
- seems simpler to use it as the actual result of the call. (this does
mean using a pointer to an Optional & actually using all 3 states (null,
None, and present) which is, admittedly, a tad subtle - but given the
limited scope, seems OK to me - open to discussion though, if others
feel strongly about it)

llvm-svn: 346691
llvm/lib/DebugInfo/DWARF/DWARFContext.cpp diff | blob | history
Domain: System / Toolchain;