[Lldb-commits] [lldb] r337397 - Fix variables.test after D49018
Stella Stamenova via lldb-commits
lldb-commits at lists.llvm.org
Wed Jul 18 08:50:24 PDT 2018
Author: stella.stamenova
Date: Wed Jul 18 08:50:24 2018
New Revision: 337397
URL: http://llvm.org/viewvc/llvm-project?rev=337397&view=rev
Log:
Fix variables.test after D49018
Summary: This one fixes variables.test after D49018. The test was broken because D49018 adds a location information to variables, but I hadn't noticed that, because I used 32-bit build to run tests, so the test looked to me already broken before that commit (the test relies on mangled names, but the mangling schemes are different for 32-bit and 64-bit).
Reviewers: stella.stamenova, lldb-commits
Reviewed By: stella.stamenova
Patch By: Aleksandr Urakov
Differential Revision: https://reviews.llvm.org/D49475
Modified:
lldb/trunk/lit/SymbolFile/PDB/variables.test
Modified: lldb/trunk/lit/SymbolFile/PDB/variables.test
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/lit/SymbolFile/PDB/variables.test?rev=337397&r1=337396&r2=337397&view=diff
==============================================================================
--- lldb/trunk/lit/SymbolFile/PDB/variables.test (original)
+++ lldb/trunk/lit/SymbolFile/PDB/variables.test Wed Jul 18 08:50:24 2018
@@ -7,19 +7,19 @@ CHECK: Module [[MOD:.*]]
CHECK: SymbolVendor ([[MOD]])
CHECK: CompileUnit{{.*}}, language = "c++", file = '{{.*}}\VariablesTest.cpp'
CHECK-DAG: Variable{{.*}}, name = "g_IntVar"
-CHECK-SAME: scope = global, external
+CHECK-SAME: scope = global, location = {{.*}}, external
CHECK-DAG: Variable{{.*}}, name = "m_StaticClassMember"
-CHECK-SAME: scope = global, external
+CHECK-SAME: scope = global, location = {{.*}}, external
CHECK-DAG: Variable{{.*}}, name = "g_pConst"
-CHECK-SAME: scope = global, external
+CHECK-SAME: scope = global, location = {{.*}}, external
CHECK-DAG: Variable{{.*}}, name = "same_name_var"
-CHECK-SAME: scope = global, external
+CHECK-SAME: scope = global, location = {{.*}}, external
CHECK-DAG: Variable{{.*}}, name = "g_EnumVar"
-CHECK-SAME: scope = global, external
+CHECK-SAME: scope = global, location = {{.*}}, external
CHECK-DAG: Variable{{.*}}, name = "g_tls"
-CHECK-SAME: scope = thread local, external
+CHECK-SAME: scope = thread local, location = {{.*}}, external
CHECK-DAG: Variable{{.*}}, name = "ClassVar"
-CHECK-SAME: scope = global, external
+CHECK-SAME: scope = global, location = {{.*}}, external
CHECK-DAG: Variable{{.*}}, name = "g_Const"
CHECK-SAME: scope = ??? (2)
More information about the lldb-commits
mailing list