amccarth accepted this revision. amccarth added a comment. This revision is now accepted and ready to land.
Thanks for the changes. If the tests pass, then this LGTM. Just check the one last question I added about the `AC1D` test. (The diff in Phabricator looks very confused, as though you deleted several hundred lines. Perhaps because the original diff had context and this one didn't. If I'm interpretting the diff correctly, this looks good now.) ================ Comment at: lit/SymbolFile/NativePDB/ast-types.cpp:22 ---------------- Should this one start with FIXME rather than CHECK? Isn't this the case that's currently broken? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60817/new/ https://reviews.llvm.org/D60817 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
