rsmith added a comment.
Looks good to go once Daniel's and my comments are addressed. Thank you.
================
Comment at: lib/AST/Expr.cpp:1887
+bool InitListExpr::isIdiomaticZeroInitializer(const LangOptions &LangOpts)
const {
+ assert(!getSyntacticForm() && "only test syntactic form as zero
initializer");
+
----------------
`!isSyntacticForm()` would be preferable here instead of `!getSyntacticForm()`.
https://reviews.llvm.org/D28148
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
