It's just that it seemed weird/hacky to use asm just to stick assembler directives in the middle of the generated code instead of actual assembly, so I generated an assembly version and cleaned it up a bit. I'm happy to go back to the original shorter C++ version if that's not a concern.

Sep 6 2019, 6:15 PM · Restricted Project

jgorbe updated the diff for D66904: [libunwind] Fix memory leak in handling of DW_CFA_remember_state and DW_CFA_restore_state.

Added a test case that seems to work well with the existing testing setup if I rebuild LLVM with ASan enabled. I have checked out other usages of .cfi_* directives in other LLVM assembly test files and they seem to run them through some llvm tool with -triple <something including linux>, so I have added a REQUIRES: linux line to avoid breakage.

Sep 6 2019, 5:59 PM · Restricted Project

Aug 30 2019

jgorbe added a comment to D66904: [libunwind] Fix memory leak in handling of DW_CFA_remember_state and DW_CFA_restore_state.

Looking into how to add a test now. I'm not familiar at all with libunwind code, so I have no idea how to cause this leak to happen from the outside (and then the test would only work when running under asan). Any pointers would be much appreciated.

Aug 30 2019, 12:50 PM · Restricted Project

Aug 28 2019

jgorbe added reviewers for D66904: [libunwind] Fix memory leak in handling of DW_CFA_remember_state and DW_CFA_restore_state: phosek, mstorsjo.

Aug 28 2019, 2:10 PM · Restricted Project

jgorbe created D66904: [libunwind] Fix memory leak in handling of DW_CFA_remember_state and DW_CFA_restore_state.

Aug 28 2019, 1:55 PM · Restricted Project

Aug 2 2019

jgorbe added a comment to rL366570: [MachineCSE][MachinePRE] Avoid hoisting code from code regions into hot BBs..

FYI we have detected a 2-3% performance regression on Haswell machines in test-suite/MicroBenchmarks/ImageProcessing/Interpolation that seems to be caused by this commit.

Aug 2 2019, 11:04 AM

Jun 7 2019

jgorbe committed rG13427e64de2b: [lldb] Fix msan use-of-uninitialized-value in DWARFDebugLine::FileNameEntry. (authored by jgorbe).

[lldb] Fix msan use-of-uninitialized-value in DWARFDebugLine::FileNameEntry.

Jun 7 2019, 2:07 PM

jgorbe committed rL362843: [lldb] Fix msan use-of-uninitialized-value in DWARFDebugLine::FileNameEntry..

[lldb] Fix msan use-of-uninitialized-value in DWARFDebugLine::FileNameEntry.

Jun 7 2019, 2:06 PM

Jun 6 2019

jgorbe abandoned D62980: [lldb] Replace "127.0.0.1" with "localhost" in GDBRemoteCommunication.cpp.

Jun 6 2019, 2:00 PM

jgorbe removed reviewers for D62980: [lldb] Replace "127.0.0.1" with "localhost" in GDBRemoteCommunication.cpp: clayborg, labath.

Jun 6 2019, 2:00 PM

jgorbe added a comment to D62980: [lldb] Replace "127.0.0.1" with "localhost" in GDBRemoteCommunication.cpp.

Hm, I just realized I'm undoing https://reviews.llvm.org/D58883.

Jun 6 2019, 1:55 PM

jgorbe created D62980: [lldb] Replace "127.0.0.1" with "localhost" in GDBRemoteCommunication.cpp.

Jun 6 2019, 1:53 PM

jgorbe added a comment to D62948: lit/Register: Avoid stdio in register write tests.

About %T not working for "process launch", what about something like RUN: %lldb -b --one-line-before-file "process launch --stdout %T/x86-zmm-write.out" -s %s %t and then FileCheck-ing?

Jun 6 2019, 1:46 PM

May 23 2019

jgorbe committed rG56d69ef8ca1f: [lldb] Make sure RegularExpression constructors always initialize member… (authored by jgorbe).

[lldb] Make sure RegularExpression constructors always initialize member…

May 23 2019, 1:12 PM

jgorbe committed rLLDB361546: [lldb] Make sure RegularExpression constructors always initialize member….

[lldb] Make sure RegularExpression constructors always initialize member…

May 23 2019, 1:11 PM

jgorbe committed rL361546: [lldb] Make sure RegularExpression constructors always initialize member….

[lldb] Make sure RegularExpression constructors always initialize member…

May 23 2019, 1:11 PM

jgorbe closed D62334: [lldb] Make sure RegularExpression constructors always initialize member variables.

May 23 2019, 1:11 PM · Restricted Project

jgorbe updated the diff for D62334: [lldb] Make sure RegularExpression constructors always initialize member variables.

Fixed indentation.

May 23 2019, 11:39 AM · Restricted Project

jgorbe created D62334: [lldb] Make sure RegularExpression constructors always initialize member variables.

May 23 2019, 11:36 AM · Restricted Project

May 22 2019

jgorbe committed rG33dbab82715c: [lldb] Fix use-of-uninitialized-value in Driver (authored by jgorbe).

[lldb] Fix use-of-uninitialized-value in Driver

May 22 2019, 4:36 PM

jgorbe committed rL361444: [lldb] Fix use-of-uninitialized-value in Driver.

[lldb] Fix use-of-uninitialized-value in Driver

May 22 2019, 4:36 PM

jgorbe committed rLLDB361444: [lldb] Fix use-of-uninitialized-value in Driver.

[lldb] Fix use-of-uninitialized-value in Driver

May 22 2019, 4:36 PM

May 9 2019

jgorbe committed rGadf427b82d40: [lldb] build.py: fix behavior when passing --compiler=/path/to/compiler (authored by jgorbe).

[lldb] build.py: fix behavior when passing --compiler=/path/to/compiler

May 9 2019, 9:46 AM

jgorbe committed rLLDB360355: [lldb] build.py: fix behavior when passing --compiler=/path/to/compiler.

[lldb] build.py: fix behavior when passing --compiler=/path/to/compiler

May 9 2019, 9:46 AM

jgorbe committed rL360355: [lldb] build.py: fix behavior when passing --compiler=/path/to/compiler.

[lldb] build.py: fix behavior when passing --compiler=/path/to/compiler

May 9 2019, 9:46 AM

jgorbe closed D61713: [lldb] build.py: fix behavior when passing --compiler=/path/to/compiler.

May 9 2019, 9:46 AM · Restricted Project, Restricted Project

jgorbe updated the diff for D61713: [lldb] build.py: fix behavior when passing --compiler=/path/to/compiler.

Removed trailing blank lines from test case, will commit now. Thanks for the review!

May 9 2019, 9:46 AM · Restricted Project, Restricted Project

jgorbe updated the diff for D61713: [lldb] build.py: fix behavior when passing --compiler=/path/to/compiler.

Added a new test that checks that, when a full path to a compiler is specified:

May 9 2019, 9:38 AM · Restricted Project, Restricted Project

May 8 2019

jgorbe created D61713: [lldb] build.py: fix behavior when passing --compiler=/path/to/compiler.

May 8 2019, 7:25 PM · Restricted Project, Restricted Project

Apr 26 2019

jgorbe committed rG1dbd42ab5bab: Revert Fix interactions between __builtin_constant_p and constexpr to match… (authored by jgorbe).

Revert Fix interactions between __builtin_constant_p and constexpr to match…

Apr 26 2019, 5:31 PM

jgorbe committed rL359361: Revert Fix interactions between __builtin_constant_p and constexpr to match….

Revert Fix interactions between __builtin_constant_p and constexpr to match…

Apr 26 2019, 5:30 PM

jgorbe committed rC359361: Revert Fix interactions between __builtin_constant_p and constexpr to match….

Revert Fix interactions between __builtin_constant_p and constexpr to match…

Apr 26 2019, 5:30 PM

Apr 22 2019

jgorbe added a comment to D60608: Make TestVSCode_step pass reliably.

Thanks for the fix!

Apr 22 2019, 11:30 AM · Restricted Project

Apr 16 2019

jgorbe added a comment to D60153: Re-enable most lldb-vscode tests on Linux..

Thanks! Please let me know if it happens again and I'll try my best to debug it.

Apr 16 2019, 3:03 PM · Restricted Project, Restricted Project

Apr 3 2019

jgorbe committed rG060bf99f493e: Re-enable most lldb-vscode tests on Linux. (authored by jgorbe).

Re-enable most lldb-vscode tests on Linux.

Apr 3 2019, 1:44 PM

jgorbe committed rLLDB357633: Re-enable most lldb-vscode tests on Linux..

Re-enable most lldb-vscode tests on Linux.

Apr 3 2019, 1:42 PM

jgorbe committed rL357633: Re-enable most lldb-vscode tests on Linux..

Re-enable most lldb-vscode tests on Linux.

Apr 3 2019, 1:42 PM

jgorbe closed D60153: Re-enable most lldb-vscode tests on Linux..

Apr 3 2019, 1:42 PM · Restricted Project, Restricted Project

jgorbe updated the diff for D60153: Re-enable most lldb-vscode tests on Linux..

Increased wait time in test binary for TestVSCode_attach.py from 5 to 10 seconds.

Apr 3 2019, 1:38 PM · Restricted Project, Restricted Project

Apr 2 2019

jgorbe created D60153: Re-enable most lldb-vscode tests on Linux..

Apr 2 2019, 2:42 PM · Restricted Project, Restricted Project

Apr 1 2019

jgorbe committed rG4665aca8ca76: [lldb-vscode] Add logic to handle EOF when reading from lldb-vscode stdout. (authored by jgorbe).

[lldb-vscode] Add logic to handle EOF when reading from lldb-vscode stdout.

Apr 1 2019, 1:37 PM

jgorbe committed rLLDB357426: [lldb-vscode] Add logic to handle EOF when reading from lldb-vscode stdout..

[lldb-vscode] Add logic to handle EOF when reading from lldb-vscode stdout.

Apr 1 2019, 1:36 PM

jgorbe committed rL357426: [lldb-vscode] Add logic to handle EOF when reading from lldb-vscode stdout..

[lldb-vscode] Add logic to handle EOF when reading from lldb-vscode stdout.

Apr 1 2019, 1:36 PM

jgorbe closed D59849: [lldb-vscode] Add logic to handle EOF when reading from lldb-vscode stdout..

Apr 1 2019, 1:36 PM · Restricted Project, Restricted Project

Mar 26 2019

jgorbe resigned from D47092: downgrade strong type info names to weak_odr linkage.

Mar 26 2019, 5:15 PM · Restricted Project

jgorbe resigned from D58654: Move Config.h from Host to Utility.

Mar 26 2019, 5:15 PM

jgorbe created D59849: [lldb-vscode] Add logic to handle EOF when reading from lldb-vscode stdout..

Mar 26 2019, 2:53 PM · Restricted Project, Restricted Project

jgorbe committed rGd7aba7678308: Add lldb-vscode as a dependency of lldb tests. (authored by jgorbe).

Add lldb-vscode as a dependency of lldb tests.

Mar 26 2019, 11:37 AM

jgorbe committed rLLDB357016: Add lldb-vscode as a dependency of lldb tests..

Add lldb-vscode as a dependency of lldb tests.

Mar 26 2019, 11:37 AM

jgorbe committed rL357016: Add lldb-vscode as a dependency of lldb tests..

Add lldb-vscode as a dependency of lldb tests.

Mar 26 2019, 11:37 AM

jgorbe closed D59828: Add lldb-vscode as a dependency of lldb tests..

Mar 26 2019, 11:37 AM · Restricted Project, Restricted Project

jgorbe created D59828: Add lldb-vscode as a dependency of lldb tests..

Mar 26 2019, 11:03 AM · Restricted Project, Restricted Project

Mar 14 2019

jgorbe committed rG717b1c804b29: [lldb-vscode] Fix dangling pointer in request_evaluate. (authored by jgorbe).

[lldb-vscode] Fix dangling pointer in request_evaluate.

Mar 14 2019, 6:49 PM

jgorbe committed rL356227: [lldb-vscode] Fix dangling pointer in request_evaluate..

[lldb-vscode] Fix dangling pointer in request_evaluate.

Mar 14 2019, 6:46 PM

jgorbe committed rLLDB356227: [lldb-vscode] Fix dangling pointer in request_evaluate..

[lldb-vscode] Fix dangling pointer in request_evaluate.

Mar 14 2019, 6:46 PM

jgorbe closed D59400: [lldb-vscode] Fix dangling pointer in request_evaluate..

Mar 14 2019, 6:46 PM · Restricted Project

jgorbe added a comment to D59400: [lldb-vscode] Fix dangling pointer in request_evaluate..

I considered assigning it to a std::string, but GetCString() can return a null pointer and I'm not sure you can construct a std::string directly from that.

Mar 14 2019, 6:43 PM · Restricted Project

jgorbe created D59400: [lldb-vscode] Fix dangling pointer in request_evaluate..

Mar 14 2019, 6:05 PM · Restricted Project