labath accepted this revision. labath added a comment. This revision is now accepted and ready to land.
I don't know omitting this had any practical effects (my experiments with rax were... inconclusive) , but it definitely sounds like the right to do. ================ Comment at: lldb/source/Plugins/Process/Utility/RegisterInfos_x86_64.h:93 { \ #reg #i, nullptr, sizeof(uint64_t), \ LLVM_EXTENSION FPR_OFFSET( \ ---------------- mgorny wrote: > To be honest, I completely don't get the indentation below. Yeah... You could try explicitly re-indenting the block to see whether it produces something better. If not, it's likely a clang-format bug (though I can't really blame it for getting this mess wrong) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91728/new/ https://reviews.llvm.org/D91728 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits