Re: [PATCH V7 0/8] Intel(R) MPX register support

[Walfred Tedeschi <walfred dot tedeschi at intel dot com>]

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" <walfred.tedeschi@intel.com>
> > 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

Attachment: mpx_no_x32.diff
Description: Text document