Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lldb jlh command has stopped working #29667

Closed
danbev opened this issue Sep 23, 2019 · 4 comments
Closed

lldb jlh command has stopped working #29667

danbev opened this issue Sep 23, 2019 · 4 comments
Labels
question Issues that look for answers.

Comments

@danbev
Copy link
Contributor

danbev commented Sep 23, 2019

  • Version: v13.0.0-pre
  • Platform: Darwin MBP-1206.lan 18.7.0 Darwin Kernel Version 18.7.0: Thu Jun 20 18:42:21 PDT 2019; root:xnu-4903.270.47~4/RELEASE_X86_64 x86_64
  • Subsystem: v8

The lldb command jlh has stopped working for me. At the moment this is what happens if I run it:

(lldb) jlh arg

(The output is just a blank line). The expected output in this case would be:

(lldb) jlh arg
#undefined

Looking at lldb_commands.py this cast seems to be the issue, at least for the jlh command and removing it fixes the issue for me.

I know that we don't maintain lldb_commands.py but I wanted to ask if anyone else has run into this before looking into it closer and perhaps opening an issue upstream.

@joyeecheung
Copy link
Member

I think I've ran into this in the v8 upstream and I had to take some ptr_ value out of some types heavily wrapped into some Torque types first, instead of just casting the original pointer and passing it to _v8_internal_Print_Object().

@joyeecheung joyeecheung added the question Issues that look for answers. label Sep 24, 2019
@mmarchini
Copy link
Contributor

V8 doesn't use heap classes as pointers to the heap anymore. Instead, those classes are concrete C++ objects with a ptr_ field. This was part of a larger initiative to make V8 UBSAN-compliant, the commits which lead to this change can be seen in the tracking issue: https://bugs.chromium.org/p/v8/issues/detail?id=3770&q=3770&can=2. Design document to move from object pointers to concrete objects can be found here: https://docs.google.com/document/d/1_w49sakC1XM1OptjTurBDqO86NE16FH8LwbeUAtrbCo/edit#heading=h.8wg7tpqbpt7m

I'll close this as answered, but feel free to reopen if you want.

@danbev
Copy link
Contributor Author

danbev commented Mar 3, 2020

I've opened a pull request for the lldb jlh command issue in V8:
https://chromium-review.googlesource.com/c/v8/v8/+/2083477/2

@mmarchini
Copy link
Contributor

In the meantime, the gdb shortcuts work great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Issues that look for answers.
Projects
None yet
Development

No branches or pull requests

3 participants