Well msvc implies system-windows, but i can have both On Fri, Nov 16, 2018 at 7:37 PM Stella Stamenova via Phabricator < revi...@reviews.llvm.org> wrote:
> stella.stamenova accepted this revision. > stella.stamenova added a comment. > This revision is now accepted and ready to land. > > Feel free to check this in. At this point this will only improve the > results on Windows and it works correctly on Linux. > > > > ================ > Comment at: lldb/lit/SymbolFile/PDB/class-layout.test:1 > -REQUIRES: system-windows > -RUN: clang-cl -m32 /Z7 /c /GS- %S/Inputs/ClassLayoutTest.cpp /o > %T/ClassLayoutTest.cpp.obj > -RUN: link %T/ClassLayoutTest.cpp.obj /DEBUG /nodefaultlib /ENTRY:main > /OUT:%T/ClassLayoutTest.cpp.exe > +REQUIRES: msvc > +RUN: %clang_cl -m32 /Z7 /c /GS- %S/Inputs/ClassLayoutTest.cpp /o > %T/ClassLayoutTest.cpp.obj > ---------------- > Why only msvc and not system-windows like the rest of the tests? > > > https://reviews.llvm.org/D54567 > > > >
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
