[libcxx-commits] [libcxx] Suppress a redundant hardening check in basic_string_view::substr (PR #91804)

Mark de Wever via libcxx-commits libcxx-commits at lists.llvm.org
Tue May 21 09:55:51 PDT 2024


mordante wrote:

I don't disagree that the hardening check is redundant in this case.
The safety checks were sold as so little overhead it could be used without issues in production. 
My concern is with adding code complexity and the precedent it sets.
I expect there are redundant checks in other places too, do we want to eliminate all of them?

https://github.com/llvm/llvm-project/pull/91804


More information about the libcxx-commits mailing list