[PATCH] D71702: [DebugInfo] Relax some checking in the debug line parser

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 20 00:37:35 PST 2019


jhenderson added a comment.

In D71702#1792118 <https://reviews.llvm.org/D71702#1792118>, @dblaikie wrote:

> Seems like a lot of changes that could probably be split out into independent patches that'd be easier to review/validate tests, etc?
>
> Some of this includes comments to existing codepaths, for instance - they should proabbly go in separately.
>
> & I'm guessing each change to a recoverable error can probably be committed and tested independently?


Yes, you're probably right. I'll take a look at splitting things up more. This is my last day in the office before the New Year. I might get some patches done, but the whole series will likely not be finished until I'm back.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71702





More information about the llvm-commits mailing list