Committed with that change, thanks Richard Sandiford. Pan
-----Original Message----- From: Richard Sandiford <richard.sandif...@arm.com> Sent: Thursday, May 18, 2023 4:57 PM To: Li, Pan2 <pan2...@intel.com> Cc: gcc-patches@gcc.gnu.org; juzhe.zh...@rivai.ai; kito.ch...@sifive.com; Wang, Yanzhang <yanzhang.w...@intel.com>; jeffreya...@gmail.com; rguent...@suse.de Subject: Re: [PATCH] Machine_Mode: Extend machine_mode from 8 to 16 bits pan2...@intel.com writes: > diff --git a/gcc/rtl-ssa/accesses.h b/gcc/rtl-ssa/accesses.h index > c5180b9308a..c2103a5cb5c 100644 > --- a/gcc/rtl-ssa/accesses.h > +++ b/gcc/rtl-ssa/accesses.h > @@ -215,7 +215,11 @@ private: > > // The values returned by the accessors above. > unsigned int m_regno; > - access_kind m_kind : 8; > + > + // The value returned by the accessor above. > + machine_mode m_mode : MACHINE_MODE_BITSIZE; > + > + access_kind m_kind : 2; There's no need to repeat the comment. Just: // The values returned by the accessors above. unsigned int m_regno; machine_mode m_mode : MACHINE_MODE_BITSIZE; access_kind m_kind : 2; would be enough. OK with that change, thanks. (There's no need to post the updated patch.) Richard