This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PING: [PATCH] Add SUBREG_REG_P predicate and change REG_SUBREG_P to be global


On 06/23/2009 04:29 PM, Olivier Hainque wrote:
Now, IMVHO, having two different predicates with so similar names is a
potential source of confusion and mistakes, for both code readers and
code writers.

If we're going the extra predicate way as suggested, could we please
take the opportunity to introduce slightly more descriptive names, for
example SUBREG_OF_REG_P and REG_OR_SUBREG_OF_REG_P ?

I think the disambiguation-hence-mistake-avoidance is well worth
having slightly longer names to type and read. In this particular
case, I'd actually rather not have a second predicate instead of
one with so similar a name.
Thanks Olivier, I think you are right.

I don't know much about RTL, actually I'm a beginner of gcc. But it seems the name of SUBREG_OF_REG_P and REG_OR_SUBREG_OF_REG_P is not consistent with the other predicates.
Can some maintainer comment?


Thanks
Pearly


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]