a.sidorin accepted this revision. a.sidorin added inline comments.
================ Comment at: test/Analysis/NewDelete-path-notes.cpp:44 // CHECK-NEXT: <dict> -// CHECK-NEXT: <key>line</key><integer>6</integer> +// CHECK-NEXT: <key>line</key><integer>7</integer> // CHECK-NEXT: <key>col</key><integer>3</integer> ---------------- Not even a minor concern for this patch, but I think that placing `//RUN` and `//CHECK` after the code being tested could save us from massive changes of line numbers. https://reviews.llvm.org/D41800 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits