Yes I expect someone else to review more thoroughly, but I included you anyway since you may have to dtart caring about this soon :)<br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 9, 2019 at 6:39 AM Pavel Labath via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br>
<br>
Is there any chance we can get someone who knows something about pdbs take a look at this (@amccarth ? @aleksandr.urakov ?) ?<br>
<br>
<br>
<br>
================<br>
Comment at: lldb/lit/SymbolFile/NativePDB/typedefs.cpp:3<br>
+<br>
+// REQUIRES: system-windows<br>
+// RUN: %build --compiler=clang-cl --nodefaultlib -o %t.exe -- %s<br>
----------------<br>
What's the reason for this? Does the test need msvc linker or something?<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D56461/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56461/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D56461" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56461</a><br>
<br>
<br>
<br>
</blockquote></div>