[lldb][test] Run calling-conventions-arm.test with native PDB reader as well

[DavidSpickett]

And actually check the arm32 output. Lucky for us, it does work.

Separated the lines a bit too so they are easier to read.

Follow up to #152580.

[llvmbot]

@llvm/pr-subscribers-lldb

Author: David Spickett (DavidSpickett)

Changes

And actually check the arm32 output. Lucky for us, it does work.

Separated the lines a bit too so they are easier to read.

---

Full diff: https://github.com/llvm/llvm-project/pull/152670.diff

1 Files Affected:

• (modified) lldb/test/Shell/SymbolFile/PDB/calling-conventions-arm.test (+5)

diff --git a/lldb/test/Shell/SymbolFile/PDB/calling-conventions-arm.test b/lldb/test/Shell/SymbolFile/PDB/calling-conventions-arm.test
index 7dabf9157d47e..8b0b5b665274d 100644
--- a/lldb/test/Shell/SymbolFile/PDB/calling-conventions-arm.test
+++ b/lldb/test/Shell/SymbolFile/PDB/calling-conventions-arm.test
@@ -1,7 +1,12 @@
 REQUIRES: target-windows, lld, (target-arm || target-aarch64)
+
 RUN: %build --compiler=clang-cl --arch=32 --nodefaultlib --output=%t.exe %S/Inputs/CallingConventionsTest.cpp
+RUN: lldb-test symbols -dump-ast %t.exe | FileCheck %s
+RUN: env LLDB_USE_NATIVE_PDB_READER=1 lldb-test symbols -dump-ast %t.exe | FileCheck %s
+
 RUN: %build --compiler=clang-cl --arch=64 --nodefaultlib --output=%t.exe %S/Inputs/CallingConventionsTest.cpp
 RUN: lldb-test symbols -dump-ast %t.exe | FileCheck %s
+RUN: env LLDB_USE_NATIVE_PDB_READER=1 lldb-test symbols -dump-ast %t.exe | FileCheck %s
 
 CHECK: Module: {{.*}}
 CHECK-DAG: int (*FuncCCallPtr)();

[DavidSpickett]

@Nerixyz

[Nerixyz]

LGTM 👍 I can't approve, though

[Nerixyz]

Nice, just realized I didn't need to rename the built executables for 32/64 bit in the other test.

[DavidSpickett]

You would have done it in the other PR, and that's approved, so I'll just merge this.