[cfe-commits] r122511 - in /cfe/trunk/include/clang/AST: CanonicalType.h Type.h
John McCall
rjmccall at apple.com
Thu Dec 23 09:56:54 PST 2010
Author: rjmccall
Date: Thu Dec 23 11:56:54 2010
New Revision: 122511
URL: http://llvm.org/viewvc/llvm-project?rev=122511&view=rev
Log:
Kill RecordType::getAddressSpace() and its fixme; that's just not how
address spaces work.
Modified:
cfe/trunk/include/clang/AST/CanonicalType.h
cfe/trunk/include/clang/AST/Type.h
Modified: cfe/trunk/include/clang/AST/CanonicalType.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/CanonicalType.h?rev=122511&r1=122510&r2=122511&view=diff
==============================================================================
--- cfe/trunk/include/clang/AST/CanonicalType.h (original)
+++ cfe/trunk/include/clang/AST/CanonicalType.h Thu Dec 23 11:56:54 2010
@@ -639,7 +639,6 @@
LLVM_CLANG_CANPROXY_SIMPLE_ACCESSOR(RecordDecl *, getDecl)
LLVM_CLANG_CANPROXY_SIMPLE_ACCESSOR(bool, isBeingDefined)
LLVM_CLANG_CANPROXY_SIMPLE_ACCESSOR(bool, hasConstFields)
- LLVM_CLANG_CANPROXY_SIMPLE_ACCESSOR(unsigned, getAddressSpace)
};
template<>
Modified: cfe/trunk/include/clang/AST/Type.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/Type.h?rev=122511&r1=122510&r2=122511&view=diff
==============================================================================
--- cfe/trunk/include/clang/AST/Type.h (original)
+++ cfe/trunk/include/clang/AST/Type.h Thu Dec 23 11:56:54 2010
@@ -2616,10 +2616,6 @@
// const, it needs to return false.
bool hasConstFields() const { return false; }
- // FIXME: RecordType needs to check when it is created that all fields are in
- // the same address space, and return that.
- unsigned getAddressSpace() const { return 0; }
-
bool isSugared() const { return false; }
QualType desugar() const { return QualType(this, 0); }
@@ -3703,8 +3699,6 @@
if (const ArrayType *AT = dyn_cast<ArrayType>(CT))
return AT->getElementType().getAddressSpace();
- if (const RecordType *RT = dyn_cast<RecordType>(CT))
- return RT->getAddressSpace();
return 0;
}
More information about the cfe-commits
mailing list