[PATCH] D34792: [lit] Remove dead code not referenced in the LLVM SVN repo.

Brian Gesiak via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 28 17:43:46 PDT 2017


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

Awesome, thank you!



================
Comment at: utils/lit/lit/formats/base.py:21
+    implemented.
+    """
+
----------------
I greatly appreciate the documentation here. Thanks!


================
Comment at: utils/lit/lit/formats/shtest.py:24
+
+        The 'execute_external' argument controls whether Lit uses its internal
+        logic for command pipelines, or passes the command to a shell
----------------
Question: The documentation for lit spells it with a lowercase 'l': http://llvm.org/docs/CommandGuide/lit.html. http://llvm.org/docs/TestingGuide.html, on the other hand, uses both capital and lowercase in some places. Which is correct?

I had always thought that lowercase "lit" was the proper name. If you agree, then this line and the "Args" section below should be updated.


https://reviews.llvm.org/D34792





More information about the llvm-commits mailing list