From f694a5573623996b400a74eaa0cdd94e44b33ea5 Mon Sep 17 00:00:00 2001
From: Sean Callanan <scallanan@apple.com>
Date: Fri, 21 Jan 2011 22:30:25 +0000
Subject: [PATCH] Added a safeguard to ensure that the user does not create
 variables that override persistent result variables.

llvm-svn: 124001
---
 lldb/source/Expression/IRForTarget.cpp | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/lldb/source/Expression/IRForTarget.cpp b/lldb/source/Expression/IRForTarget.cpp
index bdcb225aecb4..bf4d7aea446e 100644
--- a/lldb/source/Expression/IRForTarget.cpp
+++ b/lldb/source/Expression/IRForTarget.cpp
@@ -901,9 +901,23 @@ IRForTarget::RewritePersistentAllocs(llvm::Module &llvm_module, llvm::BasicBlock
         Instruction &inst = *ii;
         
         if (AllocaInst *alloc = dyn_cast<AllocaInst>(&inst))
-            if (alloc->getName().startswith("$") &&
-                !alloc->getName().startswith("$__lldb"))
+        {
+            llvm::StringRef alloc_name = alloc->getName();
+            
+            if (alloc_name.startswith("$") &&
+                !alloc_name.startswith("$__lldb"))
+            {
+                if (alloc_name.find_first_of("0123456789") == 1)
+                {
+                    if (log)
+                        log->Printf("Rejecting a numeric persistent variable.");
+                    
+                    return false;
+                }
+                
                 pvar_allocs.push_back(alloc);
+            }
+        }
     }
     
     InstrIterator iter;