[Openmp-commits] [openmp] [libomp] Add kmp_str_ref (ADT 1/2) (PR #176162)

[Michael Kruse via Openmp-commits]

================

----------------
Meinersbur wrote:

If this is just wrapping `std::string_ref`, why not using `std::string_ref` in the first place? For instance, some string_view operations in libstdc++ call `__throw_out_of_range_fmt` from libstdc++.so/a. If this is only a stop-gap to eventually completely remove dependence on `std::`: I don't think implementing string_view is that hard.

The discussion for LLVM for replacing StringRef with string_view took place at https://discourse.llvm.org/t/migrating-llvm-stringref-to-std-string-view/82785. AFAIC it has not yet concluded, but it shows the direction.

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