[libcxx-commits] [PATCH] D142993: [libcxx] Add _LIBCPP_NODEBUG to global std::ranges::__cpo variables

[Michael Buch via Phabricator via libcxx-commits]

Michael137 added a comment.

In D142993#4165116 <https://reviews.llvm.org/D142993#4165116>, @ldionne wrote:

> In D142993#4161082 <https://reviews.llvm.org/D142993#4161082>, @Michael137 wrote:
>
>> Just to clarify, if we want to get expression evaluation to work we will have to remove these attributes again, since calling these function objects requires debug-info for these variables to be present. But that feature will take some time to complete and is unlikely to make it into this release. So if we're fine with the churn for the immediate benefit then I'll go ahead and land this
>
> I am fine with that. I imagine we'll need to add something else to those variables to tell LLDB not to show them in the globals view?

Actually we decided to just purely handle this in the debugger, in order to not break expression evaluation

Workaround is here:  https://reviews.llvm.org/D145245


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142993/new/

https://reviews.llvm.org/D142993