[PATCH] align_value attribute in Clang
hfinkel at anl.gov
hfinkel at anl.gov
Thu Oct 2 13:10:31 PDT 2014
Alright, after much discussion, I think this is ready.
To quickly recap, there had been some discussion regarding whether or not align_value needed to be part of the type system (so that it would be propagated by template type deduction, for example), after after receiving feedback from Richard, Alexey, et al., we're settled on no. This is not part of the type system, and will only "propagate" through templates, auto, etc. by optimizer deduction after inlining. This seems consistent with Intel's implementation.
Aaron, this new revision should account for all additional points from your last review.
http://reviews.llvm.org/D4635
Files:
include/clang/Basic/Attr.td
include/clang/Basic/AttrDocs.td
include/clang/Basic/DiagnosticSemaKinds.td
include/clang/Sema/AttributeList.h
include/clang/Sema/Sema.h
lib/CodeGen/CGCall.cpp
lib/Sema/SemaDeclAttr.cpp
lib/Sema/SemaTemplateInstantiateDecl.cpp
test/CodeGen/align_value.cpp
test/Sema/align_value.c
test/SemaCXX/align_value.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4635.14343.patch
Type: text/x-patch
Size: 13713 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141002/1a2db9fb/attachment.bin>
More information about the cfe-commits
mailing list