[PATCH] D19312: Warn about UB at member function calls from base class ctor initializers.
Filipe Cabecinhas via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 20 09:46:24 PDT 2016
filcab added a comment.
I meant changing the diff, but if you prefer to have a smaller comment, I'm ok with the different "paths" in the standards being mentioned only in the commit message.
================
Comment at: lib/Sema/SemaDeclCXX.cpp:3941
@@ +3940,3 @@
+ // Calling a member function from a ctor-initializer
+ // before the base class results in undefined behavior [12.6.2 16].
+ // FIXME: We only check for member functions directly called from this
----------------
For someone reading the source code, it's probably best to mention the different "paths" in the standards here too.
For my usual source browsing, as long as it's in the comment *or* the commit message, I'll eventually see it. But it might throw some people off when they look at C++11, and there's no p16 in that place :-)
http://reviews.llvm.org/D19312
More information about the cfe-commits
mailing list