[cfe-commits] r111977 - in /cfe/trunk: lib/Sema/SemaExpr.cpp test/SemaCXX/address-of-register-var.cpp
Fariborz Jahanian
fjahanian at apple.com
Tue Aug 24 15:21:48 PDT 2010
Author: fjahanian
Date: Tue Aug 24 17:21:48 2010
New Revision: 111977
URL: http://llvm.org/viewvc/llvm-project?rev=111977&view=rev
Log:
It is not error in c++ to take address of
register variable (c++03 7.1.1P3). radar 8108252.
Added:
cfe/trunk/test/SemaCXX/address-of-register-var.cpp
Modified:
cfe/trunk/lib/Sema/SemaExpr.cpp
Modified: cfe/trunk/lib/Sema/SemaExpr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExpr.cpp?rev=111977&r1=111976&r2=111977&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaExpr.cpp (original)
+++ cfe/trunk/lib/Sema/SemaExpr.cpp Tue Aug 24 17:21:48 2010
@@ -6278,7 +6278,10 @@
// We have an lvalue with a decl. Make sure the decl is not declared
// with the register storage-class specifier.
if (const VarDecl *vd = dyn_cast<VarDecl>(dcl)) {
- if (vd->getStorageClass() == VarDecl::Register) {
+ // in C++ it is not error to take address of a register
+ // variable (c++03 7.1.1P3)
+ if (vd->getStorageClass() == VarDecl::Register &&
+ !getLangOptions().CPlusPlus) {
Diag(OpLoc, diag::err_typecheck_address_of)
<< "register variable" << op->getSourceRange();
return QualType();
Added: cfe/trunk/test/SemaCXX/address-of-register-var.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/address-of-register-var.cpp?rev=111977&view=auto
==============================================================================
--- cfe/trunk/test/SemaCXX/address-of-register-var.cpp (added)
+++ cfe/trunk/test/SemaCXX/address-of-register-var.cpp Tue Aug 24 17:21:48 2010
@@ -0,0 +1,9 @@
+// RUN: %clang_cc1 -fsyntax-only -verify %s
+// rdar // 8018252
+
+void f0() {
+ extern void f0_1(int*);
+ register int x;
+ f0_1(&x);
+}
+
More information about the cfe-commits
mailing list