[PATCH] D30380: Teach lit to expand glob expressions

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 3 10:42:18 PST 2017


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm



================
Comment at: llvm/utils/lit/lit/ShCommands.py:38
 
+class GlobItem:
+    def __init__(self, pattern):
----------------
This is a nicer design than what I suggested. :)


================
Comment at: llvm/utils/lit/lit/ShUtil.py:101
+        # be re-written to work correctly without triggering this).
+        assert not (quoted_glob_char and unquoted_glob_char)
+        return GlobItem(str) if unquoted_glob_char else str
----------------
Seems like a reasonable limitation to me.


https://reviews.llvm.org/D30380





More information about the llvm-commits mailing list