Re: [PATCH] D13223: Generate assume loads only with -fstrict-vtable-pointers

2015-09-28 Thread Eric Christopher via cfe-commits
echristo added a subscriber: echristo. echristo accepted this revision. echristo added a reviewer: echristo. echristo added a comment. One inline comment reword. Comment at: lib/CodeGen/CGClass.cpp:2111 @@ -2110,1 +2110,3 @@ + // FIXME: It looks like InstCombine is very ineffic

Re: [PATCH] D13223: Generate assume loads only with -fstrict-vtable-pointers

2015-09-28 Thread Reid Kleckner via cfe-commits
rnk added a subscriber: rnk. rnk accepted this revision. rnk added a reviewer: rnk. rnk added a comment. This revision is now accepted and ready to land. lgtm http://reviews.llvm.org/D13223 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D13223: Generate assume loads only with -fstrict-vtable-pointers

2015-09-28 Thread Piotr Padlewski via cfe-commits
Prazek created this revision. Prazek added reviewers: rsmith, samsonov, majnemer. Prazek added a subscriber: cfe-commits. Temporary fix till InstCombine and other possible passes will be efficient to handle multiple assumes. http://reviews.llvm.org/D13223 Files: lib/CodeGen/CGClass.cpp test