[PATCH] D52999: [FileCheck] Annotate input dump (1/7)

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 10 14:29:29 PST 2018


jdenny added a comment.

In D52999#1326125 <https://reviews.llvm.org/D52999#1326125>, @probinson wrote:

> I like having the extra explanations at the end of the tilde lines, that's a great idea.
>  One unnecessary include left, and LGTM.


Thanks!  I'll fix the include soon.

I won't commit this immediately as I'm thinking it's best to commit the entire patch series at the same time.  Let me know if you feel differently.


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

https://reviews.llvm.org/D52999





More information about the llvm-commits mailing list