http://llvm.org/bugs/show_bug.cgi?id=21899
Reid Kleckner <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #13 from Reid Kleckner <[email protected]> --- (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.
_______________________________________________ LLVMbugs mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs
