On Mon, Jul 15, 2013 at 12:25:58PM -0500, Peter Bergner wrote: > On Mon, 2013-07-15 at 11:55 -0400, David Edelsohn wrote: > > Thanks for the explanation. Leave it in. > > > > Jakub was asking about the status of the HTM patch and support, so > > please check it in. > > Ok, committed to mainline (after another bootstrap) as revision 200960. > Thanks!
Looking at the patch, you've changed FIRST_PSEUDO_REGISTER (internal gcc thing, so fine), but that also changed DWARF_FRAME_REGISTER which was computed based on FIRST_PSEUDO_REGISTER. Isn't that an ABI break in the unwinder (think about having parts of unwinder linked from different versions of libgcc)? Can't we just tweak DWARF_FRAME_REGISTER not to include those registers? Jakub