[PATCH] D158164: [FileCheck] Added --match-full-lines-leading and --match-full-lines-trailing option

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 18 01:22:43 PDT 2023


jhenderson added reviewers: jdenny, thopre.
jhenderson added a subscriber: jdenny.
jhenderson added a comment.

In D158164#4595420 <https://reviews.llvm.org/D158164#4595420>, @strimo378 wrote:

> --match-full-lines is require to check that nothing trailing is missing. Current approach is to add in 100 lines {{$}}

Okay, I think I see your point. You are simply looking to find a way to apply that regex pattern to every line, much like `--match-full-lines` adds an implicit `{{^}}` and `{{$}}` around each line. Is that correct? And you don't want to use `--match-full-lines` because of the noisy prefixes (hence I'm not sure @MaskRay's suggestion will work).

I'm not strongly opposed to the new options. However, they should be a) documented, and b) I think the names are wrong, since they contradict themselves - trailing only means not "full-lines". I would suggest that the names might be `--match-leading-lines` and `--match-trailing-lines` or something to that effect.

@jdenny/@thopre, any thoughts?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158164



More information about the llvm-commits mailing list