[llvm] Add support for DebugLexicalBlock. (PR #217965)

Juan Manuel Martinez CaamaƱo via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 27 02:32:21 PDT 2026


https://github.com/jmmartinez commented:

Hi! This comment was generated using [fuzz-fill](https://github.com/ROCm/fuzz-fill), a new tool we're building to help improve LLVM test coverage by spotting lines introduced in a PR that don't yet appear to be exercised by the full test suite, including any tests added in this PR.
 
Looking at PR head 399561cad7b1, we found 4 lines added or modified here that we couldn't find a test for:
 
1. https://github.com/llvm/llvm-project/blob/399561cad7b198e1b89d0fccb19e3b3f35fd2c99/llvm/lib/Target/SPIRV/SPIRVNonSemanticDebugHandler.cpp#L687
2. https://github.com/llvm/llvm-project/blob/399561cad7b198e1b89d0fccb19e3b3f35fd2c99/llvm/lib/Target/SPIRV/SPIRVNonSemanticDebugHandler.cpp#L706
3. https://github.com/llvm/llvm-project/blob/399561cad7b198e1b89d0fccb19e3b3f35fd2c99/llvm/lib/Target/SPIRV/SPIRVNonSemanticDebugHandler.cpp#L717
4. https://github.com/llvm/llvm-project/blob/399561cad7b198e1b89d0fccb19e3b3f35fd2c99/llvm/lib/Target/SPIRV/SPIRVNonSemanticDebugHandler.cpp#L853
 
If it makes sense, adding a regression test that validates these lines would be great. And if any of them are already tested and we missed it, we'd love to hear so we can improve the tool. Feel free to give us some feedback on whether this review was useful from your perspective. Thanks!

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


More information about the llvm-commits mailing list