[PATCH] D38074: Fix TBAA information for reference accesses

2017-09-26 Thread Ivan A. Kosarev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314209: Fix TBAA information for reference accesses (authored by kosarev). Changed prior to commit: https://reviews.llvm.org/D38074?vs=116326&id=116721#toc Repository: rL LLVM https://reviews.llvm.o

[PATCH] D38074: Fix TBAA information for reference accesses

2017-09-22 Thread John McCall via Phabricator via cfe-commits
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Sure, that's fine. LGTM. https://reviews.llvm.org/D38074 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-b

[PATCH] D38074: Fix TBAA information for reference accesses

2017-09-22 Thread Ivan A. Kosarev via Phabricator via cfe-commits
kosarev updated this revision to Diff 116326. kosarev added a comment. Refined the TODO wording. https://reviews.llvm.org/D38074 Files: lib/CodeGen/CodeGenTBAA.cpp test/CodeGen/tbaa-reference.cpp Index: test/CodeGen/tbaa-reference.cpp ==

[PATCH] D38074: Fix TBAA information for reference accesses

2017-09-22 Thread Ivan A. Kosarev via Phabricator via cfe-commits
kosarev updated this revision to Diff 116325. kosarev added a comment. Added a check for the initialization of the reference member in B::B(S&). (That in fact was my original intent; thanks for catching it.) As to reference loads, we generate them with CodeGenFunction::EmitLoadOfReference() tha