r153447, thank you!
--kcc

On Mon, Mar 26, 2012 at 9:25 AM, John McCall <[email protected]> wrote:

> On Mar 22, 2012, at 6:40 PM, Chandler Carruth wrote:
>
> On Thu, Mar 22, 2012 at 5:49 PM, Kostya Serebryany <[email protected]> wrote:
>
>> Hello,
>>
>> Please review the following patch which adds tbaa metadata nodes to
>> vtable pointer loads/stores.
>> These metadata nodes will be immediately useful for ThreadSanitizer
>> instrumentation (it needs to treat vtable updates in a special way).
>> They may also help various optimization phases that use the tbaa
>> information.
>> http://codereview.appspot.com/5881053
>>
>
> The patch looks great to me, but you should wait for one of the Clang
> IRGen folks to look at it before submitting.
>
>
> It looks fine to me.
>
> John.
>
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to