Author: eugene Date: Mon Apr 30 20:06:05 2018 New Revision: 331242 URL: http://llvm.org/viewvc/llvm-project?rev=331242&view=rev Log: Fix type_lookup test to make buildbots happy
Added: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestCppTypeLookup.py - copied, changed from r331239, lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py Removed: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py Modified: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/Makefile lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/main.cpp Modified: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/Makefile URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/Makefile?rev=331242&r1=331241&r2=331242&view=diff ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/Makefile (original) +++ lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/Makefile Mon Apr 30 20:06:05 2018 @@ -1,3 +1,3 @@ LEVEL = ../../../make -C_SOURCES := main.c +CXX_SOURCES := main.cpp include $(LEVEL)/Makefile.rules Copied: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestCppTypeLookup.py (from r331239, lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py) URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestCppTypeLookup.py?p2=lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestCppTypeLookup.py&p1=lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py&r1=331239&r2=331242&rev=331242&view=diff ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py (original) +++ lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestCppTypeLookup.py Mon Apr 30 20:06:05 2018 @@ -10,7 +10,7 @@ import lldbsuite.test.lldbutil as lldbut from lldbsuite.test.lldbtest import * from lldbsuite.test import decorators -class TestTypeLookup(TestBase): +class TestCppTypeLookup(TestBase): mydir = TestBase.compute_mydir(__file__) Removed: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py?rev=331241&view=auto ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py (original) +++ lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/TestTypeLookup.py (removed) @@ -1,89 +0,0 @@ -""" -Test that we can lookup types correctly in the expression parser -""" - -from __future__ import print_function - - -import lldb -import lldbsuite.test.lldbutil as lldbutil -from lldbsuite.test.lldbtest import * -from lldbsuite.test import decorators - -class TestTypeLookup(TestBase): - - mydir = TestBase.compute_mydir(__file__) - - def check_value(self, value, ivar_name, ivar_value): - self.assertTrue(value.GetError().Success(), - "Invalid valobj: %s" % ( - value.GetError().GetCString())) - ivar = value.GetChildMemberWithName(ivar_name) - self.assertTrue(ivar.GetError().Success(), - "Failed to fetch ivar named '%s'" % (ivar_name)) - self.assertEqual(ivar_value, - ivar.GetValueAsSigned(), - "Got the right value for ivar") - - def test_namespace_only(self): - """ - Test that we fail to lookup a struct type that exists only in a - namespace. - """ - self.build() - self.main_source_file = lldb.SBFileSpec("main.cpp") - (target, process, thread, bkpt) = lldbutil.run_to_source_breakpoint( - self, "Set a breakpoint here", self.main_source_file) - - # Get frame for current thread - frame = thread.GetSelectedFrame() - - # Make sure we don't accidentally accept structures that exist only - # in namespaces when evaluating expressions with top level types. - # Prior to the revision that added this test, we would accidentally - # accept types from namespaces, so this will ensure we don't regress - # to that behavior again - expr_result = frame.EvaluateExpression("*((namespace_only *)&i)") - self.assertTrue(expr_result.GetError().Fail(), - "'namespace_only' exists in namespace only") - - # Make sure we can find the correct type in a namespace "a" - expr_result = frame.EvaluateExpression("*((a::namespace_only *)&i)") - self.check_value(expr_result, "a", 123) - # Make sure we can find the correct type in a namespace "b" - expr_result = frame.EvaluateExpression("*((b::namespace_only *)&i)") - self.check_value(expr_result, "b", 123) - - # Make sure we can find the correct type in the root namespace - expr_result = frame.EvaluateExpression("*((namespace_and_file *)&i)") - self.check_value(expr_result, "ff", 123) - # Make sure we can find the correct type in a namespace "a" - expr_result = frame.EvaluateExpression( - "*((a::namespace_and_file *)&i)") - self.check_value(expr_result, "aa", 123) - # Make sure we can find the correct type in a namespace "b" - expr_result = frame.EvaluateExpression( - "*((b::namespace_and_file *)&i)") - self.check_value(expr_result, "bb", 123) - - # Make sure we don't accidentally accept structures that exist only - # in namespaces when evaluating expressions with top level types. - # Prior to the revision that added this test, we would accidentally - # accept types from namespaces, so this will ensure we don't regress - # to that behavior again - expr_result = frame.EvaluateExpression("*((in_contains_type *)&i)") - self.assertTrue(expr_result.GetError().Fail(), - "'in_contains_type' exists in struct only") - - # Make sure we can find the correct type in the root namespace - expr_result = frame.EvaluateExpression( - "*((contains_type::in_contains_type *)&i)") - self.check_value(expr_result, "fff", 123) - # Make sure we can find the correct type in a namespace "a" - expr_result = frame.EvaluateExpression( - "*((a::contains_type::in_contains_type *)&i)") - self.check_value(expr_result, "aaa", 123) - # Make sure we can find the correct type in a namespace "b" - expr_result = frame.EvaluateExpression( - "*((b::contains_type::in_contains_type *)&i)") - self.check_value(expr_result, "bbb", 123) Modified: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/main.cpp URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/main.cpp?rev=331242&r1=331241&r2=331242&view=diff ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/main.cpp (original) +++ lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/type_lookup/main.cpp Mon Apr 30 20:06:05 2018 @@ -1,4 +1,4 @@ -//===-- main.c --------------------------------------------------*- C++ -*-===// +//===-- main.cpp ------------------------------------------------*- C++ -*-===// // // The LLVM Compiler Infrastructure // _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits