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

Reply via email to