The same plugin is being used in both cases, the patch only touches the nonnative PDB reader.  The followup changes to the plugin itself are necessary because the results were not identical (another reason i want to de-support ParseDeclsForContext at global acope - having 2 implementations of the same thing means they will differ slightly).<br><br>In this case it happened that the ParseAllDebugSymbolsPath reconstructed both the lldb type hierarchy and the clang ast hierarchy, but the ParseAllDebugSymbols path missed a few things in the clang ast hierarchy )<br><br>The changes to the test that you see are because the two paths — even though they’re still in the same plugin — result in things being constructed in a different order<br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 8, 2019 at 2:38 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 accepted this revision.<br>
labath added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
The lldb-test change itself looks fine. I don't feel like I know the full impact of the proposed follow-up changes (but they sounds reasonable).<br>
<br>
<br>
<br>
================<br>
Comment at: lldb/lit/SymbolFile/PDB/enums-layout.test:3<br>
 RUN: %build --compiler=msvc --arch=32 --nodefaultlib --output=%T/SimpleTypesTest.cpp.enums.exe %S/Inputs/SimpleTypesTest.cpp<br>
-RUN: lldb-test symbols %T/SimpleTypesTest.cpp.enums.exe | FileCheck %s<br>
+RUN: lldb-test symbols %T/SimpleTypesTest.cpp.enums.exe | FileCheck --check-prefix=ENUM %s<br>
+RUN: lldb-test symbols %T/SimpleTypesTest.cpp.enums.exe | FileCheck --check-prefix=ENUM_CONST %s<br>
----------------<br>
I guess this is because the order in the two plugins ends up being different?<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D56418/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56418/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D56418" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56418</a><br>
<br>
<br>
<br>
</blockquote></div>