[Lldb-commits] [PATCH] D112706: [lldb/test] Allow indentation in inline tests

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Oct 28 05:18:41 PDT 2021


labath marked 2 inline comments as done.
labath added inline comments.


================
Comment at: lldb/test/API/test_utils/TestInlineTest.py:10
+
+    def test(self):
+        filename = self.getBuildArtifact("test_file.cpp")
----------------
teemperor wrote:
> teemperor wrote:
> > nit: maybe give this a more unique name + a doc string.
> Also I think this should be a NO_DEBUG_INFO_TESTCASE ?
Fun fact: tests which inherit from `Base` (instead of `TestBase`) don't automatically get debug info multiplication. (They also don't get a lot of other things, which is why this isn't suitable for regular tests, but this isn't a regular test.)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112706/new/

https://reviews.llvm.org/D112706



More information about the lldb-commits mailing list