danix800 added inline comments.
================ Comment at: clang/test/SemaObjC/crash-on-val-dep-block-expr.m:1 +// RUN: %clang_cc1 -fblocks -fsyntax-only -verify %s +// no crash ---------------- danix800 wrote: > hokein wrote: > > can you move these tests to > > `llvm-project/clang/test/AST/ast-dump-recovery.c`? > No problem. But I noticed `llvm-project/clang/test/AST/ast-dump-recovery.m` > also exists, > should I put them into this `.m` file instead? I'd prefer `ast-dump-recovery.m` since this file is much shorter. Either file needs appending `-fblocks`. Repository: rZORG LLVM Github Zorg CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155396/new/ https://reviews.llvm.org/D155396 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits