<div class="gmail_quote">On Tue, Dec 20, 2011 at 6:54 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com">klimek@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
While searching for an example to model my use of SourceMgr after, I<br>
stumbled over LLLexer.h and found what looks like an unused variable -<br>
this patch would remove it, if there's not some hidden use of TheError<br>
that I'm missing.<br></blockquote><div><br></div><div>Looks good. As long as this kind of change doesn't break the build in some way, always feel free to commit as "obvious".</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
diff --git a/lib/AsmParser/LLLexer.h b/lib/AsmParser/LLLexer.h<br>
index 33b9135..09aea5b 100644<br>
--- a/lib/AsmParser/LLLexer.h<br>
+++ b/lib/AsmParser/LLLexer.h<br>
@@ -42,7 +42,6 @@ namespace llvm {<br>
     APFloat APFloatVal;<br>
     APSInt  APSIntVal;<br>
<br>
-    std::string TheError;<br>
   public:<br>
     explicit LLLexer(MemoryBuffer *StartBuf, SourceMgr &SM, SMDiagnostic &,<br>
                      LLVMContext &C);<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br>