pjalwadi wrote: @ziqingluo-90, Thanks for the review! I’ve updated the PR with the following: Consolidated warnings into warn_unsafe_buffer_usage_in_container using a %0 parameter for the class name. Added the UnsafeBufferUsageInStringView group for targeted suppression. I investigated isPtrBufferSafe but kept the specialized logic for string_view. The current helper is too strict for substrings (uses == instead of >=) and lacks StringLiteral length checks, which are core to this safety feature. Updated handleUnsafeOperationInContainer to pass the container name, ensuring existing std::span tests remain correct.
https://github.com/llvm/llvm-project/pull/180471 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
