yamaguchi added inline comments.

================
Comment at: lib/Sema/SemaInit.cpp:875
 
   if (!VerifyOnly) {
     StructuredSubobjectInitList->setType(T);
----------------
ruiu wrote:
> Is it intentional that you run the new code only when !VerifyOnly?
I think VerifyOnly is used to check if it is formal verification mode or not, 
and if VerifyOnly == 1, it is supposed to emit no diagnostics. So I think it is 
OK that this code is here, because this part of is code emits -Wmissing-braces 
if braces are not appropriate.


https://reviews.llvm.org/D32646



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

Reply via email to