[PATCH] D28727: Add -fstrict-vtable-pointers to UserManual

2017-01-16 Thread Piotr Padlewski via Phabricator via cfe-commits
Prazek added a comment. I also have sent it on the branch Repository: rL LLVM https://reviews.llvm.org/D28727 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D28727: Add -fstrict-vtable-pointers to UserManual

2017-01-16 Thread Piotr Padlewski via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Prazek marked an inline comment as done. Closed by commit rL292112: Add -fstrict-vtable-pointers to UsersManual (authored by Prazek). Changed prior to commit: https://reviews.llvm.org/D28727?vs=84492&id=84551#toc Reposit

[PATCH] D28727: Add -fstrict-vtable-pointers to UserManual

2017-01-15 Thread Hans Wennborg via Phabricator via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. lgtm with comment Comment at: docs/UsersManual.rst:1103 + Enable optimizations based on the strict rules for overwriting polymorphic + C++ objects, e.g. vptr is invariant dur

[PATCH] D28727: Add -fstrict-vtable-pointers to UserManual

2017-01-15 Thread Piotr Padlewski via Phabricator via cfe-commits
Prazek updated this revision to Diff 84492. Prazek added a comment. Removed [no-] so hyperlink would work https://reviews.llvm.org/D28727 Files: docs/UsersManual.rst Index: docs/UsersManual.rst === --- docs/UsersManual.rst +++

[PATCH] D28727: Add -fstrict-vtable-pointers to UserManual

2017-01-15 Thread Piotr Padlewski via Phabricator via cfe-commits
Prazek updated this revision to Diff 84489. Prazek added a comment. suggested fix https://reviews.llvm.org/D28727 Files: docs/UsersManual.rst Index: docs/UsersManual.rst === --- docs/UsersManual.rst +++ docs/UsersManual.rst @@

[PATCH] D28727: Add -fstrict-vtable-pointers to UserManual

2017-01-14 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. Would it be possible to expand on, or give some kind of pointer to, what "strict rules for overwriting polymorphic C++ objects" means? https://reviews.llvm.org/D28727 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D28727: Add -fstrict-vtable-pointers to UserManual

2017-01-14 Thread Piotr Padlewski via Phabricator via cfe-commits
Prazek created this revision. Prazek added a reviewer: hans. Prazek added a subscriber: cfe-commits. It would be good to merge it to 4.0 branch. https://reviews.llvm.org/D28727 Files: docs/UsersManual.rst Index: docs/UsersManual.rst ==