Re: [Lldb-commits] [PATCH] D91734: [FastISel] Flush local value map on every instruction

2020-12-04 Thread Robinson, Paul via lldb-commits
> -Original Message- > From: Jim Ingham > Sent: Thursday, December 3, 2020 5:51 PM > To: David Blaikie > ; David Blaikie > via Phabricator > Cc: Robinson, Paul ; r...@google.com; > echri...@gmail.com; sontuan.vu...@gmail.com; mcros...@codeaurora.org; > nikola.te...@syrmia.com;

Re: [Lldb-commits] [lldb] 1fae85a - [DWARF] Add instructions to regenerate this test, if needed.

2020-04-17 Thread Robinson, Paul via lldb-commits
Hi Davide, I reread the review, and I see I was confused by two things: (1) the name of the test is static_scope.s even though what it's testing is the scope of a var with a const_value, nothing to do with `static`; (2) there's this comment in the review: https://reviews.llvm.org/D77698#1973122

Re: [Lldb-commits] [lldb] 1fae85a - [DWARF] Add instructions to regenerate this test, if needed.

2020-04-16 Thread Robinson, Paul via lldb-commits
> -Original Message- > From: lldb-commits On Behalf Of > Davide Italiano via lldb-commits > Sent: Thursday, April 16, 2020 4:32 PM > To: lldb-commits@lists.llvm.org > Subject: [Lldb-commits] [lldb] 1fae85a - [DWARF] Add instructions to > regenerate this test, if needed. > > > Author:

Re: [Lldb-commits] [lldb] r319653 - Makefile.rules: compile all tests with -fno-limit-debug-info

2017-12-06 Thread Robinson, Paul via lldb-commits
> -Original Message- > From: Pavel Labath [mailto:lab...@google.com] > Sent: Wednesday, December 06, 2017 4:09 AM > To: Jason Molenda > Cc: lldb-commits@lists.llvm.org; Robinson, Paul > Subject: Re: [Lldb-commits] [lldb] r319653 - Makefile.rules: compile all > tests with

Re: [Lldb-commits] [PATCH] D40211: Add comments to DWARFCompileUnit length fields/methods

2017-11-20 Thread Robinson, Paul via lldb-commits
FTR, the size of the compile-unit header also changed in DWARF version 5, independent of the 32/64 format. On a different topic, I had thought there was a goal of nuking lldb's copy of the DWARFxxx headers and converting to use LLVM's? Did I imagine this? If I do remember that correctly,

Re: [Lldb-commits] [PATCH] D33504: Fix FDE indexing while scan debug_info section

2017-05-25 Thread Robinson, Paul via lldb-commits
IANA lldb developer, but I should think lldb would want to understand a DWARF 4 .debug_frame section. (And it didn't change in DWARF 5, either!) --paulr > -Original Message- > From: lldb-commits [mailto:lldb-commits-boun...@lists.llvm.org] On Behalf > Of Pavel Labath via lldb-commits >

[Lldb-commits] [PATCH] Fix Ubuntu build break with gcc 5.4

2017-05-02 Thread Robinson, Paul via lldb-commits
Recently I started seeing a build error from a tree that has lldb in it; I don't know whether the problem is my configuration, or Ubuntu, or gcc, or what, but gcc complains that it can't convert 'int' to 'sigset_t' on the return statement. This naïve one-liner fixes it, although I don't know

Re: [Lldb-commits] [PATCH] D22463: [RFC] Moving to GitHub Proposal: NOT DECISION!

2016-07-19 Thread Robinson, Paul via lldb-commits
> > I think we could emulate any pre-commit hook we like via GitHub > > WebHooks by having two repositories: llvm and llvm-staging (say). > > > > People push to llvm-staging, which notifies some LLVM server we own. > > That does basic sanity checks and pushes to llvm proper if passed. > > I think