[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