omjavaid added inline comments.
================ 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 ---------------- mstorsjo wrote: > 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? Given we expect this to pass for Windows/Arm it will reduce coverage if made to run only for x86. 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