mstorsjo added a comment. I presume this is ok - if the main intent of this test is to test handling of pointers and the size thereof in PDBs.
================ Comment at: lldb/test/Shell/SymbolFile/PDB/pointers.test:2 REQUIRES: system-windows, msvc RUN: %build --compiler=clang-cl --mode=compile --arch=32 --nodefaultlib --output=%T/PointerTypeTest.cpp.obj %S/Inputs/PointerTypeTest.cpp RUN: %build --compiler=msvc --mode=link --arch=32 --nodefaultlib --output=%T/PointerTypeTest.cpp.exe %T/PointerTypeTest.cpp.obj ---------------- I wonder if it'd be better to hardcode this to build for `i386` (assuming the `%build` script allows that?) to keep it testing what it was originally made to test? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128668/new/ https://reviews.llvm.org/D128668 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
