[PATCH] D151301: [clang][Diagnostics] Print empty lines in multiline snippets

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM!



================
Comment at: clang/test/Misc/diag-style.cpp:10
+// CHECK-NEXT: {{^}}      |               ^~~~~~~~{{$}}
+// CHECK-NEXT: {{^}}    5 | {{$}}
+// CHECK-NEXT: {{^}}    6 |               true, "");{{$}}
----------------
tbaeder wrote:
> Questionable whether the space after the `|` is expected.
I don't have a good intuition for whether it is expected or not, but I think it's defensible.


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

https://reviews.llvm.org/D151301