This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rG1438639a2f7e: [LLDB] Remove undefined behavior in TestConstStaticIntegralMember.py (authored by shafik). Herald added a project: LLDB.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131460/new/ https://reviews.llvm.org/D131460 Files: lldb/test/API/lang/cpp/const_static_integral_member/TestConstStaticIntegralMember.py lldb/test/API/lang/cpp/const_static_integral_member/main.cpp Index: lldb/test/API/lang/cpp/const_static_integral_member/main.cpp =================================================================== --- lldb/test/API/lang/cpp/const_static_integral_member/main.cpp +++ lldb/test/API/lang/cpp/const_static_integral_member/main.cpp @@ -47,7 +47,6 @@ std::numeric_limits<unsigned long long>::min(); const static Enum enum_val = enum_case2; - const static Enum invalid_enum_val = static_cast<Enum>(enum_case2 + 5); const static ScopedEnum scoped_enum_val = ScopedEnum::scoped_enum_case2; const static ScopedEnum invalid_scoped_enum_val = static_cast<ScopedEnum>(5); const static ScopedCharEnum scoped_char_enum_val = ScopedCharEnum::case2; @@ -102,7 +101,6 @@ int member_copy = ClassWithOnlyConstStatic::member; Enum e = A::enum_val; - e = A::invalid_enum_val; ScopedEnum se = A::scoped_enum_val; se = A::invalid_scoped_enum_val; ScopedCharEnum sce = A::scoped_char_enum_val; Index: lldb/test/API/lang/cpp/const_static_integral_member/TestConstStaticIntegralMember.py =================================================================== --- lldb/test/API/lang/cpp/const_static_integral_member/TestConstStaticIntegralMember.py +++ lldb/test/API/lang/cpp/const_static_integral_member/TestConstStaticIntegralMember.py @@ -53,8 +53,6 @@ # Test an unscoped enum. self.expect_expr("A::enum_val", result_value="enum_case2") - # Test an unscoped enum with an invalid enum case. - self.expect_expr("A::invalid_enum_val", result_value="enum_case1 | enum_case2 | 0x4") # Test a scoped enum. self.expect_expr("A::scoped_enum_val", result_value="scoped_enum_case2")
Index: lldb/test/API/lang/cpp/const_static_integral_member/main.cpp =================================================================== --- lldb/test/API/lang/cpp/const_static_integral_member/main.cpp +++ lldb/test/API/lang/cpp/const_static_integral_member/main.cpp @@ -47,7 +47,6 @@ std::numeric_limits<unsigned long long>::min(); const static Enum enum_val = enum_case2; - const static Enum invalid_enum_val = static_cast<Enum>(enum_case2 + 5); const static ScopedEnum scoped_enum_val = ScopedEnum::scoped_enum_case2; const static ScopedEnum invalid_scoped_enum_val = static_cast<ScopedEnum>(5); const static ScopedCharEnum scoped_char_enum_val = ScopedCharEnum::case2; @@ -102,7 +101,6 @@ int member_copy = ClassWithOnlyConstStatic::member; Enum e = A::enum_val; - e = A::invalid_enum_val; ScopedEnum se = A::scoped_enum_val; se = A::invalid_scoped_enum_val; ScopedCharEnum sce = A::scoped_char_enum_val; Index: lldb/test/API/lang/cpp/const_static_integral_member/TestConstStaticIntegralMember.py =================================================================== --- lldb/test/API/lang/cpp/const_static_integral_member/TestConstStaticIntegralMember.py +++ lldb/test/API/lang/cpp/const_static_integral_member/TestConstStaticIntegralMember.py @@ -53,8 +53,6 @@ # Test an unscoped enum. self.expect_expr("A::enum_val", result_value="enum_case2") - # Test an unscoped enum with an invalid enum case. - self.expect_expr("A::invalid_enum_val", result_value="enum_case1 | enum_case2 | 0x4") # Test a scoped enum. self.expect_expr("A::scoped_enum_val", result_value="scoped_enum_case2")
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits