[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