[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