llvm

5068046b - [SYCL] Coverity: avoid std::string_view(nullptr) in sycl-trace collectors (#22111)

[SYCL] Coverity: avoid std::string_view(nullptr) in sycl-trace collectors (#22111) Related to #18506. ## Summary Coverity reported potential null pointer issues in SYCL trace collectors where `std::string_view` was constructed directly from `std::getenv(...)`. If the environment variable is not set, this can result in `std::string_view(nullptr)` (UB). Affected CIDs from the issue: CID `437235`: https://github.com/intel/llvm/blob/f9a7cc8aaea55c1c173638f8130985446d247abe/sycl/tools/sycl-trace/sycl_trace_collector.cpp#L138 CID `442884`: https://github.com/intel/llvm/blob/f9a7cc8aaea55c1c173638f8130985446d247abe/sycl/tools/sycl-trace/ur_trace_collector.cpp#L91 CID `440717`: https://github.com/intel/llvm/blob/f9a7cc8aaea55c1c173638f8130985446d247abe/sycl/tools/sycl-trace/verification_collector.cpp#L27 CID `437184`: https://github.com/intel/llvm/blob/f9a7cc8aaea55c1c173638f8130985446d247abe/sycl/tools/sycl-trace/ze_trace_collector.cpp#L311 ## What changed Replaced direct construction from `std::getenv(...)` with null-safe handling: ```cpp const char *PrinterTypeEnv = std::getenv("SYCL_TRACE_PRINT_FORMAT"); std::string_view PrinterType = PrinterTypeEnv ? PrinterTypeEnv : ""; ``` Applied in: - `sycl/tools/sycl-trace/sycl_trace_collector.cpp` - `sycl/tools/sycl-trace/ur_trace_collector.cpp` - `sycl/tools/sycl-trace/verification_collector.cpp` - `sycl/tools/sycl-trace/ze_trace_collector.cpp` - `sycl/tools/sycl-trace/cuda_trace_collector.cpp` (same pattern, proactively fixed) ## Behavior impact No functional change is intended for valid env values (`classic`, `verbose`, `compact`). When `SYCL_TRACE_PRINT_FORMAT` is unset, behavior is now explicitly safe and well-defined.