labath accepted this revision. labath added a comment. This revision is now accepted and ready to land.
seems reasonable ================ Comment at: lldb/test/Shell/SymbolFile/NativePDB/local-variables-registers.s:247-248 .Ltmp24: - .cv_def_range .Ltmp0 .Lfunc_end0, subfield_reg, 18, 0 - .cv_def_range .Ltmp1 .Lfunc_end0, subfield_reg, 3, 4 + .cv_def_range .Ltmp0 .Lfunc_end0, subfield_reg, 3, 4 + .cv_def_range .Ltmp1 .Lfunc_end0, subfield_reg,18, 0 .short 2 # Record length ---------------- Maybe add a comment about the order being deliberatele inverted. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124061/new/ https://reviews.llvm.org/D124061 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits