[Lldb-commits] [lldb] [lldb][test] Avoid out-of-bounds reads in TestConflictingSymbol.py (PR #172792)

[Igor Kudrin via lldb-commits]

igorkudrin wrote:

> Can you explain what "generic data variable" means and how we decide to classify this as such? Is it something to do with the conflicting symbols or is that unrelated.

These are variables found via symbol tables; there is no debug information for them. I derived the term from this comment in `ClangExpressionDeclMap::FindExternalVisibleDecls()`:
```
    // We couldn't find a non-symbol variable for this.  Now we'll hunt for a
    // generic data symbol, and -- if it is found -- treat it as a variable.
```

In `TestConflictingSymbol.py`, `OneConstant.c` and `TwoConstant.c` are deliberately compiled with `$(CFLAGS_NO_DEBUG)`.

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