This patch isn't right; we're not building the correct AST.  -ast-dump shows 
the following, which is clearly wrong:

    |   `-CompoundStmt 0x7fddf282f888 <line:5:13, line:9:1>
    |     |-DeclStmt 0x7fddf282f820 <line:7:1, col:23>
    |     | `-VarDecl 0x7fddf282f790 <col:1, col:3> t 'int []':'int []'
    |     |   `-ImplicitValueInitExpr 0x7fddf282f810 <<invalid sloc>> 'int 
[]':'int []'

  Also, when you submit a patch, please include the testcase as a regression 
test in clang/test/.

http://llvm-reviews.chandlerc.com/D1700
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to