VV8 LUCI CQ[v8windbg] Fix cctest with no V8_SANDBOX
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
[tools] Add v8windbg, a WinDbg extension for V8 Please take a look at tools/v8windbg/README.md for an overview of what v8windbg can do and how it's structured. This platform-specific debugging plugin makes use of the data provided by the V8 postmortem debugging API in tools/debug_helper. Note: This code began as https://github.com/billti/v8dbg and then moved into the Edge repository, where I added features gradually and got code reviews for individual changes. Now, taken in its entirety, it's an obnoxiously large CL. I'm open to breaking it up into a few chunks if that would be preferable. Bug: v8:9376 Change-Id: I3e503de00bb1aea870ae83e9bd99e4e2eab9ef98 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2031700 Reviewed-by: Michael Stanton <mvstanton@chromium.org> Reviewed-by: Tamer Tas <tmrts@chromium.org> Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Seth Brenith <seth.brenith@microsoft.com> Cr-Commit-Position: refs/heads/master@{#66319} | 6 年前 | |
Stop using /Zc:twoPhase- for building v8windbg Clang has recently deprecated the flag, and it isn't needed to parse the DbgModelh.h header anymore. The header does need some extra includes though. Bug: chromium:1495100 Change-Id: I97d5a3bf8064c6a3c27761a834ef72d16676ad27 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4983808 Reviewed-by: Seth Brenith <seth.brenith@microsoft.com> Commit-Queue: Hans Wennborg <hans@chromium.org> Cr-Commit-Position: refs/heads/main@{#90664} | 2 年前 | |
[v8windbg] Fix external strings in v8windbg This old commit ( https://github.com/v8/v8/commit/bb66e39d681ce18f6fc52d9f20a87487f1a759eb#diff-f1fd8a71261d2e2dff86552cc24089694ecd88ad56d96c12b79f893fe83d547b ) broke the ability to access V8 external strings in v8windbg. This commit fixes the issue by ensuring that external strings are correctly handled in the V8 Windbg extension. Change-Id: Iaec2305594a90a24c4008cd9a3ecda714f01fc03 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6616543 Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Paolo Severini <paolosev@microsoft.com> Cr-Commit-Position: refs/heads/main@{#100660} | 1 年前 | |
[v8windbg] Fix cctest with no V8_SANDBOX A recent change to V8 windbg expects the V8_SANDBOX to be set. Change-Id: I51f269421de529daf6fe0f04a80131a232c20be1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6688888 Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Paolo Severini <paolosev@microsoft.com> Cr-Commit-Position: refs/heads/main@{#101123} | 1 年前 | |
[tools] Add v8windbg, a WinDbg extension for V8 Please take a look at tools/v8windbg/README.md for an overview of what v8windbg can do and how it's structured. This platform-specific debugging plugin makes use of the data provided by the V8 postmortem debugging API in tools/debug_helper. Note: This code began as https://github.com/billti/v8dbg and then moved into the Edge repository, where I added features gradually and got code reviews for individual changes. Now, taken in its entirety, it's an obnoxiously large CL. I'm open to breaking it up into a few chunks if that would be preferable. Bug: v8:9376 Change-Id: I3e503de00bb1aea870ae83e9bd99e4e2eab9ef98 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2031700 Reviewed-by: Michael Stanton <mvstanton@chromium.org> Reviewed-by: Tamer Tas <tmrts@chromium.org> Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Seth Brenith <seth.brenith@microsoft.com> Cr-Commit-Position: refs/heads/master@{#66319} | 6 年前 |