mehdi_amini added a comment.

@dblaikie mentioned before that it is not clear that this is something we want 
to do at all (initialize all local variables).
Would it be something we should have policy on in the first place before 
integrating such a patch? (Reminds me of constructor that don't initializes all 
members "on purpose").
Maybe send an RFC to LLVM-dev and codify this in the coding guidelines?

In https://reviews.llvm.org/D17983#371040, @ariccio wrote:

> ... I'd rather crash on a nullptr deref than some undefined behavior.


Wait... Isn't nullptr deref UB?


https://reviews.llvm.org/D17983



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to