Regression test to ensure that we handling importing of anonymous enums correctly

[shafik]

Summary

https://reviews.llvm.org/D51633 added error handling in the ASTImporter.cpp which uncovered an underlying bug in which we used the wrong name when handling naming conflicts. This could cause a segmentation fault when attempting to cast an int to an enum during expression parsing.

This test should pass once https://reviews.llvm.org/D59665 is committed.

[davide]

LGTM, just clang format main.cpp before committing.

[aprantl]

packages/Python/lldbsuite/test/expression_command/cast_int_to_anonymous_enum/main.cpp

2

You can speed up compilation time of this test significantly by removing the printf and the include of cstdio.

9

It looks like this variable is not actually used by the test?

[shafik]

Small updated to test, remove use of printf and associated include.

packages/Python/lldbsuite/test/expression_command/cast_int_to_anonymous_enum/main.cpp

9

I need the type in the test, so this forces the debug information to be generated.

packages/Python/lldbsuite/test/expression_command/cast_int_to_anonymous_enum/Makefile

+LEVEL = ../../make
+
+CXX_SOURCES := main.cpp
+
+include $(LEVEL)/Makefile.rules

packages/Python/lldbsuite/test/expression_command/cast_int_to_anonymous_enum/TestCastIntToAnonymousEnum.py

+"""
+Test Expression Parser regression text to ensure that we handle anonymous
+enums importing correctly.
+"""
+
+
+import lldb
+from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test import lldbutil
+
+class TestCastIntToAnonymousEnum(TestBase):
+
+    mydir = TestBase.compute_mydir(__file__)
+
+    def test_cast_int_to_anonymous_enum(self):
+        self.build()
+
+        lldbutil.run_to_source_breakpoint(self, '// break here',
+                lldb.SBFileSpec("main.cpp", False))
+
+        self.expect("expr (flow_e)0", substrs=['(flow_e) $0 = A'])

packages/Python/lldbsuite/test/expression_command/cast_int_to_anonymous_enum/main.cpp

+typedef enum {
+    A=0,
+} flow_e;
+
+int main() {
+   flow_e f;
+
+   return 0; // break here
+}