[Lldb-commits] [PATCH] D53086: [PDB] Fix flaky `variables-locations.test` after PR38857

Zachary Turner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Oct 10 11:15:57 PDT 2018


zturner added a comment.

You didn't include it here, but I notice the test file just writes `clang-cl /Zi`.  Should we be passing `-m32` or `-m64`?  Currently, the test just runs with whatever architecture happens to be set via the VS command prompt.  The behavior here is different on x86 and x64 so perhaps it requires separate tests.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D53086





More information about the lldb-commits mailing list