Skip to content

Commit

Permalink
Fix the test to deal with non-deterministic output. (#96800)
Browse files Browse the repository at this point in the history
When we check for the "struct CustomType" in the NODWP, we can just make
sure that we have both types showing up, the next tests will validate
the types are correct. Also added a "-DAG" to the integer and float
types. This should fix the flakiness in this test.
  • Loading branch information
clayborg committed Jun 28, 2024
1 parent 3f24561 commit f0bffb1
Showing 1 changed file with 6 additions and 16 deletions.
22 changes: 6 additions & 16 deletions lldb/test/Shell/SymbolFile/DWARF/x86/dwp-foreign-type-units.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -30,24 +30,14 @@
// RUN: -o "type lookup CustomType" \
// RUN: -b %t | FileCheck %s --check-prefix=NODWP
// NODWP: (lldb) type lookup IntegerType
// NODWP-NEXT: int
// NODWP-NEXT: unsigned int
// NODWP-DAG: int
// NODWP-DAG: unsigned int
// NODWP: (lldb) type lookup FloatType
// NODWP-NEXT: double
// NODWP-NEXT: float
// NODWP-DAG: double
// NODWP-DAG: float
// NODWP: (lldb) type lookup CustomType
// NODWP-NEXT: struct CustomType {
// NODWP-NEXT: typedef int IntegerType;
// NODWP-NEXT: typedef double FloatType;
// NODWP-NEXT: CustomType::IntegerType x;
// NODWP-NEXT: CustomType::FloatType y;
// NODWP-NEXT: }
// NODWP-NEXT: struct CustomType {
// NODWP-NEXT: typedef unsigned int IntegerType;
// NODWP-NEXT: typedef float FloatType;
// NODWP-NEXT: CustomType::IntegerType x;
// NODWP-NEXT: CustomType::FloatType y;
// NODWP-NEXT: }
// NODWP: struct CustomType {
// NODWP: struct CustomType {

// Check when we make the .dwp file with %t.main.dwo first so it will
// pick the type unit from %t.main.dwo. Verify we find only the types from
Expand Down

0 comments on commit f0bffb1

Please sign in to comment.