[Lldb-commits] [lldb] [lldb-dap] Don't emit memory reference for constants (PR #197645)

[Sergei Druzhkov via lldb-commits]

DrSergei wrote:

I double checked constants.
Before:
evaluate("2 + 2") has not memory reference
evaluate("10") has memory reference
evaluate("0x00007fffffffd684") has memory reference
evaluate("abc") has memory reference
After:
None has memory reference.

I think in case when constant is valid address in memory current behavior is good. I am not sure that checking every load address is a good solution. I will wait 2 weeks before merge if anybody has another suggestion how to avoid this regression

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