[PATCH] D138073: [clang-doc] Move file layout to the generators.

Paul Kirth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 22 18:49:06 PST 2022


paulkirth reopened this revision.
paulkirth added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tools-extra/test/clang-doc/single-file-public.cpp:9
+//   for a name with a 40-char USR name.
+// RUN: find %t -regex ".*[0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z].*" -exec cat {} \; | FileCheck %s --check-prefix=CHECK
 // RUN: rm -rf %t
----------------
paulkirth wrote:
> 
since this failed, maybe:
```
// RUN: find %t/docs -regex ".*/[0-9A-F]*.yaml" -exec cat {} \; | FileCheck %s
```
That works locally for me on both mac and linux. you lose the nice check for 40 chars, but at least it functions. if its very important, you can always have a single run line that does `ls` and `grep`.

FYI: the BSD `find` accepts the failing regex w/o `-regextype sed`. It's unfortunate that GNU `find` doesn't have the same default regex, and doesn't accept that version w/o the `-regextype` being set.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138073



More information about the cfe-commits mailing list