[PATCH] D154130: [lit] Avoid os.path.realpath on Windows due to MAX_PATH limitations
Joel E. Denny via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 10 16:46:26 PDT 2023
jdenny added a comment.
If the `%>` feature is going to remain (depending on, for example, the answer to @tahonermann's question <https://reviews.llvm.org/D154130#4486898> about modulemap), please:
1. Add it to the lit documentation at https://llvm.org/docs/TestingGuide.html#substitutions and https://llvm.org/docs/CommandGuide/lit.html#substitutions. (Aside: I wish someone would reconcile those lists.)
2. Consider whether there's a clearer syntax. For example, we already have `%{/t:regex_replacement}`. What about `%{t:real}` and `%{/t:real}`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154130/new/
https://reviews.llvm.org/D154130
More information about the cfe-commits
mailing list