[PATCH] D72157: [test][llvm-dwarfdump] Add extra test case for invalid MD5 form

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 27 02:34:03 PST 2020


jhenderson added a comment.

In D72157#1827583 <https://reviews.llvm.org/D72157#1827583>, @dblaikie wrote:

> Looks good - James, I might've asked this on another thread - but could you write up a brief goals/design email of what your overall goal is with these patches? There seem to be enough of them that it'd be helpful to link to in each review (and summarize where the patch fits in the broader scheme of things) & provide framing when reviewing patches to better understand the overall goals.


Thanks. As you probably saw, I did previously write up comments in D72155 <https://reviews.llvm.org/D72155>. I'll post something on the mailing list at some point in the next couple of days, in order to clarify everything.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72157





More information about the llvm-commits mailing list