[PATCH] Declare as Inheritable attributes that are already processed that way

Eli Friedman eli.friedman at gmail.com
Thu Jul 18 11:46:56 PDT 2013


On Thu, Jul 18, 2013 at 11:29 AM, Dean Sutherland <dsutherland at cert.org> wrote:
> Change declarations of 8 attributes to InheritableAttr from Attr.  All eight are already processed as inheritable attributes. This change makes their declaration match the processing. This is clean-up in preparation for tablegen improvements to reduce redundancy in SemaDeclAttr and friends. No functional changes intended.
>
> The effected attributes are:
> OpenCLKernel
> OpenCLImageAccess
> ExtVectorType
> Mode
> ObjCPreciseLifetime
> ObjCReturnsInnerPointer
> ObjCRootClass
> ObjCOwnership

Hmm...

For ExtVectorType, Mode, and OpenCLImageAccess, I'm not sure it makes
sense to say they're inherited: processTypeAttrs in SemaType.cpp
always handles them.

The others are fine.  (You don't need to get confirmation from the
people who wrote them.)

-Eli




More information about the cfe-commits mailing list