Change a NOTE to a FIXME based on feedback from clattner.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@56775 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Sema/SemaExpr.cpp b/lib/Sema/SemaExpr.cpp
index c6115a8..269db21 100644
--- a/lib/Sema/SemaExpr.cpp
+++ b/lib/Sema/SemaExpr.cpp
@@ -312,8 +312,17 @@
/// DeclDefinedWithinScope - Return true if the specified decl is defined at or
/// within the 'Within' scope. The current Scope is CurScope.
///
-/// NOTE: This method is extremely inefficient (linear scan), this should not be
-/// used in common cases.
+/// FIXME: This method is extremely inefficient (linear scan), this should not
+/// be used in common cases. Replace with the more modern DeclContext. We need
+/// to make sure both assignments below produce an error.
+///
+/// int main(int argc) {
+/// int xx;
+/// ^(int X) {
+/// xx = 4; // error (variable is not assignable)
+/// argc = 3; // no error.
+/// };
+/// }
///
static bool DeclDefinedWithinScope(ScopedDecl *D, Scope *Within,
Scope *CurScope) {