Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(48)

Issue 18621002: Make MachO gdbjit support compile again (Closed)

Created:
7 years, 5 months ago by danno
Modified:
7 years, 5 months ago
Reviewers:
Michael Starzinger
CC:
v8-dev
Visibility:
Public.

Description

Make MachO gdbjit support compile again R=mstarzinger@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=15460

Patch Set 1 #

Patch Set 2 : Remove stray flag toggles #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -12 lines) Patch
M src/gdb-jit.cc View 5 chunks +12 lines, -12 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
danno
ptal
7 years, 5 months ago (2013-07-03 07:16:29 UTC) #1
danno
ptal
7 years, 5 months ago (2013-07-03 07:16:29 UTC) #2
Michael Starzinger
LGTM.
7 years, 5 months ago (2013-07-03 08:08:18 UTC) #3
haitao.feng
7 years, 5 months ago (2013-07-03 11:53:07 UTC) #4
Message was sent while issue was closed.
It compiles, but does not work. Please see
https://codereview.chromium.org/13083002/ for more information.

In the future, we should make lldb work for V8 jit on Mac as gdb remains 6.3.50
on Mac but gdbjit requires 7.0. When I have free time, I will have a try :)

Powered by Google App Engine
This is Rietveld 408576698