Author: zturner
Date: Mon Nov 19 08:41:31 2018
New Revision: 347223
URL: http://llvm.org/viewvc/llvm-project?rev=347223&view=rev
Log:
Remove non-ASCII characters at the beginning of file.
It's not clear how these ended up in the file, but this fixes it.
Modified:
lldb/trunk/lit/helper/toolch
Author: zturner
Date: Mon Nov 19 08:47:06 2018
New Revision: 347224
URL: http://llvm.org/viewvc/llvm-project?rev=347224&view=rev
Log:
[lit] On Windows, don't error if MSVC is not in PATH.
We had some logic backwards, and as a result if MSVC was not found
in PATH we would throw a string concatenat
Lgtm
On Wed, Nov 21, 2018 at 12:10 PM Stella Stamenova via Phabricator <
revi...@reviews.llvm.org> wrote:
> stella.stamenova created this revision.
> stella.stamenova added reviewers: labath, zturner, asmith.
> Herald added subscribers: lldb-commits, jfb.
>
> Right now only some platforms add pthr
There’s actually been a slow push away from cl::opt. It’s less flexible and
doesn’t support some things that the TableGen approach does. Recently
there’s been a few efforts to port existing tools onto TableGen options
from cl::opt.
I don’t think cl::opt is going away anytime soon so if it works I
One potentially nice way to test this would be to have all the -O, -o, -S,
-s, -Q, and -q just run "script print(N)" where N is some number that
changes at each line, then FileCheck the output
On Mon, Nov 26, 2018 at 6:38 PM Jim Ingham via Phabricator <
revi...@reviews.llvm.org> wrote:
> jingham
I can repro this, so I'll work on a fix.
On Wed, Nov 28, 2018 at 12:38 PM Stella Stamenova via Phabricator <
revi...@reviews.llvm.org> wrote:
> stella.stamenova added a comment.
>
> I ran the tests with this change and there are about a dozen new failures:
>
> 2018-11-28T20:25:29.2437909Z
Yea, I don't really want to fix it there though because the problem will
just happen again. I think it should be done inside of ToolSubst.
On Wed, Nov 28, 2018 at 12:59 PM Stella Stamenova via Phabricator <
revi...@reviews.llvm.org> wrote:
> stella.stamenova added a comment.
>
> It works for me
view?
On Thu, Nov 29, 2018 at 2:39 PM Greg Clayton wrote:
>
>
> > On Nov 29, 2018, at 2:02 PM, Zachary Turner via Phabricator <
> revi...@reviews.llvm.org> wrote:
> >
> > zturner added a comment.
> >
> > In D53368#1313238 <https://reviews.llvm.o
It would be great if future work could go towards the native pdb plugin. To
be honest, I’m trying very hard to get it to the point that the DIA plugin
can be deleted, so large pieces of new functionality that only go to the
DIA plugin are, in a way, wasted effort. I’m currently working on getting
a
Author: zturner
Date: Fri Nov 30 16:22:21 2018
New Revision: 348058
URL: http://llvm.org/viewvc/llvm-project?rev=348058&view=rev
Log:
[lit] Add a generic build script with a lit substitution.
This adds a script called build.py as well as a lit substitution
called %build that we can use to invoke
In that case, can we implement that method first? I was planning to do it
this week, but if someone else wants to do it immediately that’s fine too
On Mon, Dec 3, 2018 at 8:11 AM Leonid Mashinskiy via Phabricator <
revi...@reviews.llvm.org> wrote:
> leonid.mashinskiy added a comment.
>
> Seems lik
Is the only reason we need these methods so that we can use lldb-test? If
so, perhaps the tests can be written in terms of lldb itself, similar to
the existing NativePDB tests
On Mon, Dec 3, 2018 at 8:39 AM Aleksandr Urakov via Phabricator <
revi...@reviews.llvm.org> wrote:
> aleksandr.urakov adde
ght? But this FPO conversion is
> applicable to locals only, so we need to implement locals creation in the
> new plugin first, before moving the conversion there. That's why we need
> these functions.
>
> Am Mo., 3. Dez. 2018, 19:41 hat Zachary Turner
> geschrieben:
>
>
Perhaps use %t.bogus? instead of /bogus?
On Mon, Dec 3, 2018 at 12:39 PM Jonas Devlieghere via lldb-commits <
lldb-commits@lists.llvm.org> wrote:
> Author: jdevlieghere
> Date: Mon Dec 3 12:36:21 2018
> New Revision: 348186
>
> URL: http://llvm.org/viewvc/llvm-project?rev=348186&view=rev
> Log:
some network drive.
>
> On Mon, Dec 3, 2018 at 12:53 PM Zachary Turner wrote:
>
>> Perhaps use %t.bogus? instead of /bogus?
>>
>> On Mon, Dec 3, 2018 at 12:39 PM Jonas Devlieghere via lldb-commits <
>> lldb-commits@lists.llvm.org> wrote:
>>
>>
If you have -o and multiple input files, which output file are you
specifying? It seems easier to just say that if multiple input files are
present, the output file names are automatically calculated
On Mon, Dec 3, 2018 at 2:48 PM Stella Stamenova via Phabricator <
revi...@reviews.llvm.org> wrote:
all of the input files to be compiled
> into a single executable?
>
>
>
> *From:* Zachary Turner
> *Sent:* Monday, December 3, 2018 3:36 PM
> *To:* reviews+d55230+public+74273d963dffb...@reviews.llvm.org
> *Cc:* pa...@labath.sk; Stella Stamenova ;
> aleksandr.ura...@jetbr
orbid
multiple inputs entirely with -mode=compile and require the user to write
multiple RUN lines, but it seems likely that all of the options will be the
same across all invocations, so this would be a nice shortcut.
On Mon, Dec 3, 2018 at 3:59 PM Zachary Turner wrote:
> If that's the cas
Author: zturner
Date: Tue Dec 4 13:48:27 2018
New Revision: 348305
URL: http://llvm.org/viewvc/llvm-project?rev=348305&view=rev
Log:
[build.py] A few general improvements.
This makes -mode=compile support multiple inputs (and hence
multiple outputs).
It also makes the value of -arch for compili
Author: zturner
Date: Tue Dec 4 14:46:01 2018
New Revision: 348314
URL: http://llvm.org/viewvc/llvm-project?rev=348314&view=rev
Log:
Fix LLDB build script.
A local patch was omitted from the original commit. This makes
the tests pass.
Modified:
lldb/trunk/lit/helper/build.py
Modified: lld
Author: zturner
Date: Tue Dec 4 15:56:25 2018
New Revision: 348319
URL: http://llvm.org/viewvc/llvm-project?rev=348319&view=rev
Log:
[build.py] Disable tests on non-Windows.
This won't work until we get the GCC / clang builder implemented.
Modified:
lldb/trunk/lit/BuildScript/modes.test
If you can commit the same fix I did for the other two that would be great,
otherwise I’ll get to it when i get into the office in ~2 hours
On Wed, Dec 5, 2018 at 3:03 AM Pavel Labath wrote:
> On 05/12/2018 00:56, Zachary Turner via lldb-commits wrote:
> > Author: zturner
> > Date
Yes, if the tests pass for you with current builder, please commit
On Wed, Dec 5, 2018 at 6:06 AM Aleksandr Urakov via Phabricator <
revi...@reviews.llvm.org> wrote:
> aleksandr.urakov added a comment.
>
> It seems that all the reviews this one depends on are already in. Can we
> proceed with it?
platforms
On Wed, Dec 5, 2018 at 7:35 AM Pavel Labath wrote:
> On 05/12/2018 16:03, Zachary Turner wrote:
> > If you can commit the same fix I did for the other two that would be
> > great, otherwise I’ll get to it when i get into the office in ~2 hours
>
>
> The thing is, I&
It is not possible to specify object file name in compile and link mode.
But perhaps we can just change the default object file name to include
something from the output file as well
On Wed, Dec 5, 2018 at 12:26 PM Aleksandr Urakov via Phabricator <
revi...@reviews.llvm.org> wrote:
> aleksandr.ura
in path with directories. May be we can replace slashes with
> underscores in the output file path and concatenate it with the object file
> name? Or even replace slashes in the source file path and concatenate it
> with the output file path - so object files will be located in the same
> plac
Lgtm. Btw I noticed StreamBuffer, which might be another easy one
On Thu, Dec 6, 2018 at 3:27 AM Pavel Labath via Phabricator <
revi...@reviews.llvm.org> wrote:
> labath updated this revision to Diff 176948.
> labath added a comment.
>
> That's a good point, thanks for reminding me.
>
> Re-clang-f
Yea that sounds reasonable
On Thu, Dec 6, 2018 at 1:07 AM Pavel Labath wrote:
> On 06/12/2018 09:25, Aleksandr Urakov via lldb-commits wrote:
> >
> > I thought about what Stella have said, and it seems that I have found a
> > good solution for this. We can use something like `tempfile`
> > (https
Author: zturner
Date: Thu Dec 6 08:55:00 2018
New Revision: 348499
URL: http://llvm.org/viewvc/llvm-project?rev=348499&view=rev
Log:
Support skewed stream arrays.
VarStreamArray was built on the assumption that it is backed by a
StreamRef, and offset 0 of that StreamRef is the first byte of the
Author: zturner
Date: Thu Dec 6 09:49:15 2018
New Revision: 348505
URL: http://llvm.org/viewvc/llvm-project?rev=348505&view=rev
Log:
[PDB] Move some code around. NFC.
Added:
lldb/trunk/source/Plugins/SymbolFile/NativePDB/DWARFLocationExpression.cpp
lldb/trunk/source/Plugins/SymbolFile/N
Author: zturner
Date: Thu Dec 6 10:39:58 2018
New Revision: 348511
URL: http://llvm.org/viewvc/llvm-project?rev=348511&view=rev
Log:
[build.py] Embed the output file name in generated object file names.
In compile-and-link mode, the user doesn't specify the name of the
object files to generate,
s unique
> enough.
>
>
>
> Thanks,
>
> -Stella
>
>
>
> *From:* Aleksandr Urakov
> *Sent:* Thursday, December 6, 2018 12:25 AM
> *To:* Zachary Turner
> *Cc:* reviews+d54942+public+2603ca548f36d...@reviews.llvm.org; Stella
> Stamenova ; llvm-comm...@lists.ll
Author: zturner
Date: Thu Dec 6 10:45:07 2018
New Revision: 348514
URL: http://llvm.org/viewvc/llvm-project?rev=348514&view=rev
Log:
Fix line endings in build.py
Modified:
lldb/trunk/lit/helper/build.py
Modified: lldb/trunk/lit/helper/build.py
URL:
http://llvm.org/viewvc/llvm-project/lldb/
Fwiw, I think mode=compile-and-link and compiler=any are both defaults, so
those 2 options could be omitted to make this shorter. Not a big deal
though
On Thu, Dec 6, 2018 at 3:28 PM Stella Stamenova via lldb-commits <
lldb-commits@lists.llvm.org> wrote:
> Author: stella.stamenova
> Date: Thu Dec
Author: zturner
Date: Fri Dec 7 11:34:02 2018
New Revision: 348631
URL: http://llvm.org/viewvc/llvm-project?rev=348631&view=rev
Log:
[NativePDB] Reconstruct function declarations from debug info.
Previously we would create an lldb::Function object for each function
parsed, but we would not add t
Author: zturner
Date: Mon Dec 10 16:15:03 2018
New Revision: 348813
URL: http://llvm.org/viewvc/llvm-project?rev=348813&view=rev
Log:
Fix undefined behavior in Variable.h
m_loc_is_constant_data was uninitialized, so unless someone
explicitly called SetLocIsConstantData(), this would be UB.
I thi
should even be a hardcoded default - it's just a
> convenient but temporary hack. I'm open to any alternative ideas we can use
> until we implement a SymbolVendor.
>
> On Tue, Dec 11, 2018 at 10:39 AM Zachary Turner via Phabricator <
> revi...@reviews.llvm.o
(see
> Symbols::LocateExecutableSymbolFile). So maybe it's not the terrible hack I
> made it look like.
>
> On Tue, Dec 11, 2018 at 10:48 AM Zachary Turner
> wrote:
>
>> But if the minidump and PDBs are in the same directory, then wouldn't my
>> proposed
On Tue, Dec 11, 2018 at 11:57 AM Pavel Labath wrote:
> The part I know nothing about is whether something similar could be done
> for PE/COFF files (and I'll need something like that there too). Adrian,
> Zachary, what is the relation ship between "image base" of an object
> file and its sections
y replacing a DLL. Replacing a PDB is probably less of a
>>> concern.
>>>
>>>
>>> https://docs.microsoft.com/en-us/windows/desktop/dlls/dynamic-link-library-search-order#standard-search-order-for-desktop-applications
>>>
>>> So I +1 Zach's
ult search
path. But if people disagree, it seems like target symbols add would still
address the problem in a permanent way.
On Tue, Dec 11, 2018 at 3:04 PM Zachary Turner wrote:
> Only one way to know for sure, and that's to test it :) So I did.
>
> Yes, it will search the dir
27;s worth, recent versions of VC++ debugger
> are more aligned with ntsd/cdb/windbg.
>
> Anyway, for this patch I think the question is: what's the simplest
> temporary solution? All of these ideas are interesting but more appropriate
> to evaluate when we start designing the PD
On Tue, Dec 11, 2018 at 4:22 PM Leonard Mosescu wrote:
> I guess I don't see why we need a temporary solution at all. If we can
>> have logic that can be rolled into the SymbolVendor when we get it, and
>> makes sense there, and is also simple, why not go with it? Failing that,
>> doesn't the `
It's fine, I was mostly just curious.
On Wed, Dec 12, 2018 at 8:52 AM Pavel Labath via Phabricator <
revi...@reviews.llvm.org> wrote:
> labath marked an inline comment as done.
> labath added inline comments.
>
>
>
> Comment at: lldb/trunk/lit/helper/build.py:630
> +a
directory to this path.
So, if you want this behavior in a supported way that isn't temporary, we
should move the code for findMatchingPDBFile() out of SymbolFilePDB and
into this function. Then everyone is happy I think.
What do you think?
On Tue, Dec 11, 2018 at 4:42 PM Zachary Turner
Author: zturner
Date: Wed Dec 12 09:17:53 2018
New Revision: 348941
URL: http://llvm.org/viewvc/llvm-project?rev=348941&view=rev
Log:
[ast] CreateParameterDeclaration should use an appropriate DeclContext.
Previously CreateParameterDeclaration was always using the translation
unit DeclContext. W
On irc earlier i was talking about this with Greg and he said it should be
fine in his opinion. I’ll point him to this review in the morning so he can
comment
On Thu, Dec 13, 2018 at 3:30 AM Pavel Labath via Phabricator <
revi...@reviews.llvm.org> wrote:
> labath added inline comments.
>
>
> =
Author: zturner
Date: Thu Dec 13 10:17:51 2018
New Revision: 349067
URL: http://llvm.org/viewvc/llvm-project?rev=349067&view=rev
Log:
[NativePDB] Add support for local variables.
This patch adds support for parsing and evaluating local variables.
using the native pdb plugin.
Differential Revisio
ted about is the implicit lookup next to the
> .dmp or .dll/.exe - it's highly specific to a local debugging scenario
> which may be appropriate for something like an IDE, ex. VisualStudio but
> not for a general purpose debugger)
>
> So my preference is for the latest patch - wh
I think we can fix that by changing the line to:
```
if (!object_file || object_file->GetFileSpec() == symbol_fspec) {
}
```
On Thu, Dec 13, 2018 at 12:04 PM Pavel Labath wrote:
> On 13/12/2018 19:32, Leonard Mosescu wrote:
> > What's the consensus?
> >
> > Personally I think that, even conside
is why it seemed safe).
>
> At this point I have to ask: what is the problem with the current
> directory lookup again? I seems to me that we complicated ourselves for no
> good reason (using a questionable functionality in a high level Windows IDE
> as reference).
>
>
&
At this point it seems like perpetuating the hack, or at least even if
that's the direction we decide to go longterm, not implementing that
solution fully and missing some of the corner cases.
So I think I'd rather just go with the original hack of checking the
current directory at this point. St
o have
>> to implement it anyway, so we could avoid this by just implementing the
>> permanent solution first
>>
> The catch22 is that we can't test anything else involving minidumps + PDBs
> in the meantime. I found that exercising that combination to be very useful
>
this gets us there quicker then I can get behind it. I trust your
judgement here :)
On Thu, Dec 13, 2018 at 2:19 PM Zachary Turner wrote:
> The permanent solution would be figuring out what to do about the
> ObjectFile situation (e.g. do we want to make an ObjectFilePDB or do we
> wan
I can't access the buildbot right now, it's just timing out and never
loading the webpage. If you have it open and can paste the output here I
can try to figure out what's wrong, otherwise I'll have to wait for it to
become available again.
On Fri, Dec 14, 2018 at 10:31 AM Stella Stamenova via Ph
Ahh, that seems easy enough to fix. Just need to change the 7 to a
{{.*}}. Since we're actually compiling and running a process on the
buildbot, different versions of Windows will affect this.
On Fri, Dec 14, 2018 at 10:39 AM Stella Stamenova via Phabricator <
revi...@reviews.llvm.org> wrote:
>
Author: zturner
Date: Fri Dec 14 10:43:42 2018
New Revision: 349175
URL: http://llvm.org/viewvc/llvm-project?rev=349175&view=rev
Log:
[NativePDB] Fix local-variables.cpp test.
Since we're actually running an executable on the host now, different
versions of Windows could load different system lib
be
hard to make this work.
On Fri, Dec 14, 2018 at 10:51 AM Pavel Labath wrote:
> On 14/12/2018 19:43, Zachary Turner via lldb-commits wrote:
> > Author: zturner
> > Date: Fri Dec 14 10:43:42 2018
> > New Revision: 349175
> >
> > URL: http://llvm.org/view
I've fixed it. It looks like the bot should be green again once this test
starts passing. Can you make it noisy and starting to send emails now?
On Fri, Dec 14, 2018 at 10:42 AM Zachary Turner wrote:
> Ahh, that seems easy enough to fix. Just need to change the 7 to a
> {{.*}}.
failure.
> With Pavel’s change to add the gcc builder, most of those can now be
> enabled, so I am going to do that today, but there’s still a couple that
> are “unexpectedly passing” when they are in fact failing. We’ll need to fix
> that before making it noisy.
>
>
>
>
are of.
>
>
>
> Thanks,
>
> -Stella
>
>
>
> *From:* Zachary Turner
> *Sent:* Friday, December 14, 2018 11:04 AM
> *To:* Stella Stamenova
> *Cc:* reviews+d55575+public+5ea6161324294...@reviews.llvm.org;
> pa...@labath.sk; mose...@google.com; aleksandr.ur
ly 1 module
dumped anyway because we don't have PDBs for any of the system libraries,
so DumpClangAST is a no-op in those cases).
On Fri, Dec 14, 2018 at 11:00 AM Pavel Labath wrote:
> On 14/12/2018 19:52, Zachary Turner wrote:
> > I don't know if there's a good way to do th
Don't we print the environment in verbose mode? Can we see that output as
well?
On Fri, Dec 14, 2018 at 11:53 AM Stella Stamenova
wrote:
> The verbose output is below. If I run the two commands on the command
> line, they both succeed, but when I run them through build.py, the link
> command is
Author: zturner
Date: Mon Dec 17 08:15:13 2018
New Revision: 349360
URL: http://llvm.org/viewvc/llvm-project?rev=349360&view=rev
Log:
[Clang AST Context] Add a few helper functions.
The first one allows us to add an enumerator to an enum if we
already have an APSInt, since ultimately the implemen
Author: zturner
Date: Mon Dec 17 11:43:33 2018
New Revision: 349383
URL: http://llvm.org/viewvc/llvm-project?rev=349383&view=rev
Log:
[NativePDB] Decouple AST reconstruction from lldb Symbol creation.
Previously the code that parsed debug info to create lldb's Symbol
objects such as Variable, Typ
Author: zturner
Date: Mon Dec 17 13:33:08 2018
New Revision: 349399
URL: http://llvm.org/viewvc/llvm-project?rev=349399&view=rev
Log:
Fix case of source file in CMakeLists.txt
Modified:
lldb/trunk/source/Plugins/SymbolFile/NativePDB/CMakeLists.txt
Modified: lldb/trunk/source/Plugins/SymbolFi
Author: zturner
Date: Tue Dec 18 15:12:08 2018
New Revision: 349565
URL: http://llvm.org/viewvc/llvm-project?rev=349565&view=rev
Log:
[NativePDB] Correctly reconstruct DeclContext for nested enums.
We reconstruct the AST hierarchy by trying to hack up a mangled
name for the parent type using the
Author: zturner
Date: Wed Dec 19 11:45:30 2018
New Revision: 349673
URL: http://llvm.org/viewvc/llvm-project?rev=349673&view=rev
Log:
[NativePDB] Fix a use after free and enable corresponding native test.
We had a use after free where we were assigning the result of a function
that returned a str
Author: zturner
Date: Wed Dec 19 12:00:25 2018
New Revision: 349675
URL: http://llvm.org/viewvc/llvm-project?rev=349675&view=rev
Log:
[NativePDB] Enable function-level-linking.test in native mode.
This test passes with the native reader, so run it in both modes.
Modified:
lldb/trunk/lit/Symb
Author: zturner
Date: Wed Dec 19 14:50:22 2018
New Revision: 349692
URL: http://llvm.org/viewvc/llvm-project?rev=349692&view=rev
Log:
Fix line endings.
Modified:
lldb/trunk/lit/SymbolFile/NativePDB/nested-types.cpp
Modified: lldb/trunk/lit/SymbolFile/NativePDB/nested-types.cpp
URL:
http://l
Author: zturner
Date: Thu Dec 20 15:32:37 2018
New Revision: 349854
URL: http://llvm.org/viewvc/llvm-project?rev=349854&view=rev
Log:
[NativePDB] Create VarDecls for global variables.
Previously we would create these for local variables but not for
global variables.
Also updated existing tests w
Sorry, this comment was supposed to be deleted after I realized i was wrong.
On Sat, Dec 29, 2018 at 9:12 PM Aleksandr Urakov via Phabricator <
revi...@reviews.llvm.org> wrote:
> aleksandr.urakov marked an inline comment as done.
> aleksandr.urakov added inline comments.
>
>
>
>
Just to be sure, this new test will fail without your Symtab changes
right? I'm not in a state where I can look at code right now, and you say
anything that symbolicates an address *can* use the symtab, but I don't
know if you really meant *must* use the symtab.
On Wed, Jan 2, 2019 at 12:19 AM Pa
Author: zturner
Date: Wed Jan 2 10:32:50 2019
New Revision: 350240
URL: http://llvm.org/viewvc/llvm-project?rev=350240&view=rev
Log:
[NativePDB] Fix setting breakpoint by file and line.
There were several problems preventing this from working. The
first is that when the PDB had an absolute path
Author: zturner
Date: Wed Jan 2 10:33:32 2019
New Revision: 350242
URL: http://llvm.org/viewvc/llvm-project?rev=350242&view=rev
Log:
[NativePDB] Update function-types-classes test to check VarDecls.
A Previous patch added support for creating VarDecls for global
variables. This patch updates th
Author: zturner
Date: Wed Jan 2 10:33:54 2019
New Revision: 350243
URL: http://llvm.org/viewvc/llvm-project?rev=350243&view=rev
Log:
[NativePDB] Implement ParseDeclsForContext.
This is a first step towards getting lldb-test symbols working
with the native plugin. There is a remaining issue, whi
Author: zturner
Date: Wed Jan 2 10:53:11 2019
New Revision: 350244
URL: http://llvm.org/viewvc/llvm-project?rev=350244&view=rev
Log:
Use map::insert instead of try_emplace.
try_emplace is C++17.
Modified:
lldb/trunk/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp
Modified: lldb/trunk/sour
Author: zturner
Date: Wed Jan 2 13:04:22 2019
New Revision: 350262
URL: http://llvm.org/viewvc/llvm-project?rev=350262&view=rev
Log:
Try to fix Green Dragon bot.
It doesn't like this std::tie() for some reason, hopefuly this
fixes it.
Modified:
lldb/trunk/source/Plugins/SymbolFile/NativePDB
Very excited to see this. I'm technically on vacation so I might not be
able to review it immediately, but I'm looking forward to getting to it
soon.
On Wed, Jan 2, 2019 at 4:36 PM Aaron Smith via Phabricator <
revi...@reviews.llvm.org> wrote:
> asmith created this revision.
> asmith added revie
Ok, lgtm then
On Wed, Jan 2, 2019 at 11:53 PM Pavel Labath wrote:
> On 02/01/2019 18:32, Zachary Turner wrote:
> > Just to be sure, this new test will fail without your Symtab changes
> > right? I'm not in a state where I can look at code right now, and you
> > say any
For those kinds of cases, we could use obj2yaml and check in yaml right?
Fwiw I tried to round-trip an exe through obj->yaml->obj recently and the
resulting exe was incorrect but it was close, so I think there’s only some
small fixes needed.
In regards to your previous response, couldn’t we have l
It seems like this test could be made to work with non dwarf debug info by
compiling a real source file. WDYT?
On Fri, Jan 4, 2019 at 5:31 AM Pavel Labath via Phabricator <
revi...@reviews.llvm.org> wrote:
> labath created this revision.
> labath added reviewers: clayborg, zturner.
> Herald added
> ouldn’t we have lldb generate the mini dump itself as the first step of
>> the test?
>>
>
> How would this work cross-platform?
>
> On Fri, Jan 4, 2019 at 8:48 AM Zachary Turner wrote:
>
>> For those kinds of cases, we could use obj2yaml and check in yaml right?
Waiiit a second. Why don't we just teach obj2yaml and yaml2obj to
round-trip minidumps? This way you could run it on a minidump, then
hand-edit it to customize some bits you want to change, then check in the
yaml.
On Fri, Jan 4, 2019 at 10:56 AM Greg Clayton wrote:
>
> On Jan 4, 2019, at 9
The same plugin is being used in both cases, the patch only touches the
nonnative PDB reader. The followup changes to the plugin itself are
necessary because the results were not identical (another reason i want to
de-support ParseDeclsForContext at global acope - having 2 implementations
of the sa
Author: zturner
Date: Mon May 21 12:47:45 2018
New Revision: 332877
URL: http://llvm.org/viewvc/llvm-project?rev=332877&view=rev
Log:
Enable Python API for OpenBSD.
Patch by David Carlier
Modified:
lldb/trunk/scripts/utilsOsType.py
Modified: lldb/trunk/scripts/utilsOsType.py
URL:
http://ll
This change has introduced a dependency from Core -> clang Driver (due to
#include "clang/Driver/Driver.h" in ModuleList.cpp). Can you please try to
find a way to remove this dependency?
On Fri, Mar 2, 2018 at 2:45 PM Phabricator via Phabricator via lldb-commits
wrote:
> This revision was not a
se don't use the driver and now pull it in, would moving the Clang
> API into a different clang library work? Which one?
>
> -- adrian
>
> On May 22, 2018, at 11:28 AM, Zachary Turner wrote:
>
> This change has introduced a dependency from Core -> clang Driver (due t
Yea I don’t think this addresses the problem. We should be able to link
against parts of lldb without a dependency on clang. Since this is about
configuring something related to clang, it seems like it should be isolated
to some part of lldb that interfaces with clang
On Tue, May 22, 2018 at 4:32 P
think about that approach?
On Wed, May 23, 2018 at 8:42 AM Adrian Prantl wrote:
>
>
> > On May 22, 2018, at 6:57 PM, Zachary Turner wrote:
> >
> > Yea I don’t think this addresses the problem. We should be able to link
> against parts of lldb without a dependency on
SystemInitializerFull.cpp, in the
function SystemInitializerFull::Initialize().
On Wed, May 23, 2018 at 9:44 AM Adrian Prantl wrote:
>
>
> On May 23, 2018, at 8:51 AM, Zachary Turner wrote:
>
> There's not really a diagram, because we don't have an exact vision of
>
, May 23, 2018 at 1:25 PM Zachary Turner via Phabricator <
revi...@reviews.llvm.org> wrote:
> zturner added inline comments.
>
>
>
> Comment at: source/Core/ModuleList.cpp:94
>
> - llvm::SmallString<128> path;
> - clang::driver::Dr
Author: zturner
Date: Wed May 23 16:56:09 2018
New Revision: 333143
URL: http://llvm.org/viewvc/llvm-project?rev=333143&view=rev
Log:
Break dependency from Core to ObjectFileJIT.
The only reason this was here was so that Module could have a
function called CreateJITModule which created things in
Author: zturner
Date: Wed May 23 17:11:24 2018
New Revision: 333145
URL: http://llvm.org/viewvc/llvm-project?rev=333145&view=rev
Log:
Add missing include.
Modified:
lldb/trunk/include/lldb/Core/Module.h
Modified: lldb/trunk/include/lldb/Core/Module.h
URL:
http://llvm.org/viewvc/llvm-project
On Wed, May 23, 2018 at 7:04 PM Jim Ingham via Phabricator <
revi...@reviews.llvm.org> wrote:
> jingham added a comment.
>
> I worry when concerns of layering which seem a little artificial to me
> would make us add a shadow class for something that is already well
> expressed as it is. If you pa
+1 I’d like to get rid of all EnumerateXXX with callback functions and
replace with iterator based equivalents. Given that in this case the
iterator version already exists, I definitely think we should try to use it
instead
On Wed, May 30, 2018 at 9:30 AM Pavel Labath via Phabricator <
revi...@revi
Author: zturner
Date: Mon Jun 4 10:41:00 2018
New Revision: 333933
URL: http://llvm.org/viewvc/llvm-project?rev=333933&view=rev
Log:
Remove dependency from Host to clang.
Host depended on clang because HostInfo had a function to get
the directory where clang was installed. We move this over to
Do you just need a pdb, or does it really need to be a vs pdb? lld can
generate high quality pdbs now. So it might be possible to use lld to link
and produce a pdb when you run the test.
Pavel’s suggestion is equally viable, you can dump a pdb to yaml and
convert it back to a pdb at test time.
Th
On Thu, Jun 7, 2018 at 2:53 AM Aleksandr Urakov via Phabricator <
revi...@reviews.llvm.org> wrote:
> aleksandr.urakov added a comment.
>
> Yes, it does really need to be a VS PDB. Function-level linking is a
> linker's feature and I can't found support of this feature in lld, so I
> think module a
As a general rule, lld-link is command line compatible with MSVC and
clang-cl is command line compatible with cl. So, the /order option should
work exactly the same with lld-link as it does with link.
On Thu, Jun 7, 2018 at 7:00 AM Aleksandr Urakov via Phabricator <
revi...@reviews.llvm.org> wrot
101 - 200 of 3010 matches
Mail list logo
