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

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 15 16:22:04 PST 2017


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 during the lifetime of object.
+   This enables better devirtualization. Turned off by default.
----------------
e.g should be i.e. (or you could even spell it out: "that is")

I think it should also be "*the* vptr" and "*the* object* or "during an object's lifetime".


https://reviews.llvm.org/D28727





More information about the cfe-commits mailing list