steakhal accepted this revision. steakhal added a comment. This revision is now accepted and ready to land.
Aside from the inline nit, I think it's good to go. Let some time for the others to catch up, they might have objections. ================ Comment at: clang/test/Analysis/return-ptr-range.cpp:17 return arr; // no-warning - } while (0); - return ptr; // expected-warning{{Returned pointer value points outside the original object (potential buffer overflow)}} + } + return ptr; // expected-warning{{Returned pointer value points outside the original object (potential buffer overflow) [alpha.security.ReturnPtrRange]}} ---------------- I don't think we need this extra scope. Same for the others. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107051/new/ https://reviews.llvm.org/D107051 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits