This revision was automatically updated to reflect the committed changes.
Closed by commit rC338636: [AST][2/4] Move the bit-fields from FunctionDecl and
CXXConstructorDecl into… (authored by erichkeane, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D49732
Files:
bricci updated this revision to Diff 158501.
bricci added a comment.
rebased after the great whitespace trimming
Repository:
rC Clang
https://reviews.llvm.org/D49732
Files:
include/clang/AST/Decl.h
include/clang/AST/DeclCXX.h
lib/AST/Decl.cpp
lib/AST/DeclCXX.cpp
bricci updated this revision to Diff 157289.
bricci marked 2 inline comments as done.
bricci added a comment.
- ran clang-format on the changes
- removed the unnecessary setPureImpl, isMultiVersionImpl and
setMultiVersionImpl. However unfortunately I think that isDeletedImpl has to
stay because
erichkeane added a comment.
Anotherone in need of clang-format, otherwise seems alright to me.
Comment at: include/clang/AST/Decl.h:1809
+ // and setMultiVersion do not simply set/read the corresponding bit.
+ void setPureImpl(bool isPure) { FunctionDeclBits.IsPure = isPure;
bricci created this revision.
bricci added a project: clang.
Herald added a subscriber: cfe-commits.
This patch follows https://reviews.llvm.org/D49729
and moves the bits from FunctionDecl and CXXConstructorDecl
into DeclContext.
Repository:
rC Clang
https://reviews.llvm.org/D49732
Files: