[cfe-commits] Please review: add tbaa metadata to vtable pointer loads/stores

Chandler Carruth chandlerc at google.com
Thu Mar 22 18:40:46 PDT 2012


On Thu, Mar 22, 2012 at 5:49 PM, Kostya Serebryany <kcc at google.com> 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.

Thanks for working on this!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120322/d58ea90f/attachment.html>


More information about the cfe-commits mailing list