[PATCH] D83894: [lit] Don't expand escapes until all substitutions have been applied
Sergej Jaskiewicz via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 15 12:22:46 PDT 2020
broadwaylamb marked an inline comment as done.
broadwaylamb added inline comments.
================
Comment at: llvm/utils/lit/lit/TestRunner.py:1203
+
+ return list([unescape(process(escape(ln))) for ln in script])
----------------
ldionne wrote:
> I guess `list()` is now redundant.
Oh, for some reason I thought list comprehensions are lazy. One moment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83894/new/
https://reviews.llvm.org/D83894
More information about the llvm-commits
mailing list