[LLVMbugs] [Bug 21899] Crash with undefined variables with the name prefix "result"

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=21899

Reid Kleckner <rnk at google.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #13 from Reid Kleckner <rnk at google.com> ---
(In reply to comment #12)
> You may be able to shrink that test case a bit more by dropping the "else
> printf(...)". ;)
> 
> I just tried out your patch against the original preprocessed .cpp file and
> it seems to work fine in addition to looking correct to me (got the
> suggestion of 'result0' in place of 'resulta' instead of getting a
> stackdump). Feel free to commit your patch with the reduced test case.
> 
> Oh, and one small cleanup since you're touching the line,
> "!Result.isInvalid() && Result.get()" is the same as and can be replaced
> with "Result.isUsable()" (which I'd missed / didn't know about when I wrote
> that code).

Thanks, I did this in r224172 and r224173. Should be fixed!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20141213/74dcca7d/attachment.html>