[PATCH] D41800: [analyzer] Use a custom program point for the check::NewAllocator callback.

Aleksei Sidorin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 11 05:15:51 PST 2018


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





More information about the cfe-commits mailing list