[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