teemperor added a comment.

So the actual code that hit his use case was when one steps into the expression 
wrapper function we create in LLDB (which uses these dollar signs). So we 
(luckily) don't have any user code that triggers this.

I should note that I anyway discovered that the variable loading optimization 
that caused this regression doesn't support any unicode characters (not that we 
encourage that), so I'll anyway rewrite this parsing code with something that 
reuses the Clang lexer instead of our own ad-hoc parsing (which probably wil 
save us from more weird corner cases in the future).


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D64194



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to