[Lldb-commits] [PATCH] D52101: [lldb-mi] Correct regex in the symbol-list-lines test

Alexander Polyakov via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Sep 14 09:03:30 PDT 2018


apolyakov created this revision.
apolyakov added reviewers: tatyana-krasnukha, aprantl, clayborg.
Herald added a subscriber: ki.stfu.

The test failed in case of compiling a test suite with
gcc (checked versions are 5.2.0 and 7.3.0) because it
adds one more line entry comparing to clang. It doesn't
break the test's logic, so I just added a regex that matches
this case.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D52101

Files:
  lit/tools/lldb-mi/symbol/symbol-list-lines.test


Index: lit/tools/lldb-mi/symbol/symbol-list-lines.test
===================================================================
--- lit/tools/lldb-mi/symbol/symbol-list-lines.test
+++ lit/tools/lldb-mi/symbol/symbol-list-lines.test
@@ -14,7 +14,7 @@
 # CHECK: ^error,msg="File Handler. Invalid file name path"
 
 -symbol-list-lines symbol-list-lines.c
-# CHECK: ^done,lines=[{pc="0x{{[0-9A-Fa-f]+}}",line="3"},{pc="0x{{[0-9A-Fa-f]+}}",line="4"},{pc="0x{{[0-9A-Fa-f]+}}",line="5"}]
+# CHECK: ^done,lines=[{pc="0x{{[0-9A-Fa-f]+}}",line="3"},{pc="0x{{[0-9A-Fa-f]+}}",line="4"},{pc="0x{{[0-9A-Fa-f]+}}",line="5"}{{.*}}]
 
 -symbol-list-lines list-lines-helper.c
-# CHECK: ^done,lines=[{pc="0x{{[0-9A-Fa-f]+}}",line="1"},{pc="0x{{[0-9A-Fa-f]+}}",line="2"},{pc="0x{{[0-9A-Fa-f]+}}",line="3"}]
+# CHECK: ^done,lines=[{pc="0x{{[0-9A-Fa-f]+}}",line="1"},{pc="0x{{[0-9A-Fa-f]+}}",line="2"},{pc="0x{{[0-9A-Fa-f]+}}",line="3"}{{.*}}]


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52101.165520.patch
Type: text/x-patch
Size: 917 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20180914/8c81e99d/attachment-0001.bin>


More information about the lldb-commits mailing list