On Thu, Oct 21, 2010 at 12:05 AM, Ted Kremenek <[email protected]> wrote:
> I went ahead applied this patch (with a few modifications) here:
>
>  http://llvm.org/viewvc/llvm-project?view=rev&revision=116996

Thanks Ted, I'm out of town and didn't have time to get to it yet.
Where is isa<> defined, by the way?

> I mainly incorporated my minor nits I commented on in my last email.
> I also needed to modify the test case slightly to use 'typedef'
> instead of '#define'.  Even though they might appear to be correct,
> please make sure that the test cases actually pass before submitting
> patches.

Well that's embarrassing. I'll have to be more careful to make sure
what I test and what I send actually sync up.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to