Can probably do the same for XStateType as well:
enum class XStateType { Invalid, FXSAVE, XSAVE };
On Wed, Sep 14, 2016 at 10:46 AM Zachary Turner <ztur...@google.com> wrote:
> On Wed, Sep 14, 2016 at 10:36 AM Valentina Giusti via lldb-commits <
> lldb-commits@lists.llvm.org> wrote:
>
>
> ==============================================================================
> ---
> lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h
> (original)
> +++
> lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h
> Wed Sep 14 12:27:48 2016
> @@ -77,7 +77,8 @@ protected:
>
> private:
> // Private member types.
> - enum FPRType { eFPRTypeNotValid = 0, eFPRTypeFXSAVE, eFPRTypeXSAVE };
> + enum XStateType { eXStateTypeNotValid = 0, eXStateTypeFXSAVE,
> eXStateTypeXSAVE };
> + enum RegSet { gpr, fpu, avx, mpx };
>
>
> This still isn't an enum class. You need to write:
>
> enum class RegSet { gpr, fpu, avx, mpx };
>
> Can you make this change and submit it as a followup please?
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
