Hi Mark, What I mean is the appended patch. This is what I added on top of v7. 1. It removes MPX support for x32. 2. Improves i386-xstate.h macros. (identation was weird) 3. Fix the regression on MPX initialization on i386-tdep.c I considered that to be easy to review than having all the patch series again. :) Thanks a lot! Best regards, -Fred Am 11/11/2013 11:08 AM, schrieb Mark Kettenis: >> From: "Tedeschi, Walfred" >> Date: Mon, 11 Nov 2013 08:15:36 +0000 >> >> Hello Mark, >> >> I am not sure I got it right. Is this a ok to commit? > Pretty much. > >> If so, I still would like to do some changes before proceeding. >> Basically I want to remove the x32 support since it makes no sense >> and a regression I caused. Those changes where mentioned here: >> https://sourceware.org/ml/gdb-patches/2013-10/msg00513.html > I'll await your patch then ;). > >> To avoid sending you a full patch again my Idea is to send an ad-hoc >> patch to be added on top of my V7. While doing the commit then I >> would join then on the respective place. > Not sure what you mean by that. As long as all the intermediate > states compile and you don't introduce regressions for non-MPX > support, this should be fine. Intel GmbH Dornacher Strasse 1 85622 Feldkirchen/Muenchen, Deutschland Sitz der Gesellschaft: Feldkirchen bei Muenchen Geschaeftsfuehrer: Christian Lamprechter, Hannes Schwaderer, Douglas Lusk Registergericht: Muenchen HRB 47456 Ust.-IdNr./VAT Registration No.: DE129385895 Citibank Frankfurt a.M. (BLZ 502 109 00) 600119052