[PATCH] D27512: Move clang's ParseHelper into Support (llvm part)

Hans Wennborg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 8 15:06:21 PST 2016


hans added a comment.

In https://reviews.llvm.org/D27512#616974, @olista01 wrote:

> I'm going to use this for https://reviews.llvm.org/D27514, adding similar diagnostic verifying functionality to llvm-mc.


Interesting, that looks like cool functionality.

It seems unfortunate that not more code could be shared between llvm-mc and clang's -verify though. It looks like you're essientially implementing a large chunk of it again.

It would be really cool if we could build some kind of DiagnosticVerifier utility that would work for both clang and llvm-mc. And then this ParserHelper would be an implementation of that.


Repository:
  rL LLVM

https://reviews.llvm.org/D27512





More information about the llvm-commits mailing list