[PATCH] D39281: [libunwind] Express Registers_*::lastDwarfReg using _LIBUNWIND_HIGHEST_DWARF_REGISTER

John Baldwin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 26 07:10:57 PDT 2017


bsdjhb added a comment.

I think one source of truth is better than two.  I do find the constant's value a bit off in general though.  That is, the name 'HIGHEST' implies to me that it is the highest value used, not N + 1 as we currently define it.


https://reviews.llvm.org/D39281





More information about the cfe-commits mailing list