[Lldb-commits] [PATCH] D79563: [lldb/test] Make "inline" tests handle multiple statements at the same location

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu May 7 05:03:38 PDT 2020


labath added a reviewer: teemperor.
labath marked an inline comment as done.
labath added inline comments.


================
Comment at: lldb/test/API/commands/expression/dollar-in-variable/main.c:20
   //%self.expect("expr int $foo = 123", error=True, substrs=["declaration conflicts"])
-  return 0; //%self.expect("expr $0", substrs=['(int)', ' = 14'])
+  return 0; //TODO: self.expect("expr $0", substrs=['(int)', ' = 14'])
 }
----------------
Raphael: This check wasn't executed before this patch. As far as I can tell, this functionality hasn't worked since the test was first introduced (r365698).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79563





More information about the lldb-commits mailing list