[PATCH] D150990: [split-file] Add flags to support comments and adding extension to output files

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 23 12:58:35 PDT 2023


MaskRay added a comment.

> I go into more details on motivation in Discourse thread.

Thanks for the link, but it seems that the thread doesn't give examples how `--allow-comments` and `--add-file-extension` are supposed to be used.

> `// dr164: 16 becomes //--- dr164: 16, which produces dr164.cpp file with the corresponding test.`

I am confused by this comment. Does the option ignore everything starting from the `:`? Why?
The Discourse thread gives a link to https://github.com/llvm/llvm-project/blob/main/clang/test/CXX/drs/dr5xx.cpp which uses this notation, but the discourse thread doesn't elaborate on the syntax. This kind of detail is actually what I expect from the commit message/Phabricator summary, instead of some information embedded in a very long C++ DR specific forum post.

For testing, use `llvm/test/tools/split-file/` and `ninja check-llvm-tools-split-file`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150990



More information about the llvm-commits mailing list