[PATCH] D43470: [DebugInfo] Prevent crash when .debug_line line_range is zero

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 27 13:04:45 PST 2020


probinson added a comment.

It looks like getOperationAdvance() and advanceAddr() now share a lot of parameters.  Is it feasible to combine them?  I see that they're not *always* called together, but it seems like it would improve clarity not to repeat 3 lines of parameters.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D43470/new/

https://reviews.llvm.org/D43470





More information about the llvm-commits mailing list