chaoren added a comment.
Cool. Would it be a good idea to only use LLVM's demangler? We can get rid of
all the messy ifdef business, and I assume the LLVM demangler would be better
maintained and more up to date than any system demangler.
https://reviews.llvm.org/D24293
Author: chaoren
Date: Mon Jul 18 16:11:43 2016
New Revision: 275914
URL: http://llvm.org/viewvc/llvm-project?rev=275914&view=rev
Log:
Add missing headers after header cleanup in r275882.
Modified:
lldb/trunk/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.cpp
Modified:
lldb/tru
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270493: Make sure TestRedefinitionsInInlines.py actually
inlines. (authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D20540?vs=58153&id=58156#toc
Repository:
rL LLVM
http://revi
Author: chaoren
Date: Mon May 23 16:44:34 2016
New Revision: 270493
URL: http://llvm.org/viewvc/llvm-project?rev=270493&view=rev
Log:
Make sure TestRedefinitionsInInlines.py actually inlines.
Reviewers: spyffe
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D20540
Modi
chaoren created this revision.
chaoren added a reviewer: spyffe.
chaoren added a subscriber: lldb-commits.
http://reviews.llvm.org/D20540
Files:
packages/Python/lldbsuite/test/lang/c/inlines/main.c
Index: packages/Python/lldbsuite/test/lang/c/inlines/main.c
chaoren added a subscriber: chaoren.
chaoren added a comment.
We don't test host-only Windows. It's passing Windows to Android though.
What's the reason that it fails on Windows?
Repository:
rL LLVM
http://reviews.llvm.org/D19751
___
lldb-commits
We don't test host-only Windows. It's passing Windows to Android though.
What's the reason that it fails on Windows?
On Wed, May 4, 2016 at 2:31 PM, Adrian McCarthy wrote:
> amccarth added a subscriber: amccarth.
> amccarth added a comment.
>
> Chaoren: Did this completely fix the test for you?
Author: chaoren
Date: Fri Apr 29 18:34:44 2016
New Revision: 268135
URL: http://llvm.org/viewvc/llvm-project?rev=268135&view=rev
Log:
Fix TestEnumTypes.py for 32 bit platforms.
Reviewers: clayborg
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D19751
Modified:
lld
This revision was automatically updated to reflect the committed changes.
Closed by commit rL268135: Fix TestEnumTypes.py for 32 bit platforms. (authored
by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D19751?vs=55685&id=55686#toc
Repository:
rL LLVM
http://reviews.llvm.org/D1
chaoren created this revision.
chaoren added a reviewer: clayborg.
chaoren added a subscriber: lldb-commits.
http://reviews.llvm.org/D19751
Files:
packages/Python/lldbsuite/test/lang/c/enum_types/TestEnumTypes.py
Index: packages/Python/lldbsuite/test/lang/c/enum_types/TestEnumTypes.py
+self.expect('frame variable f.op', DATA_TYPES_DISPLAYED_CORRECTLY,
substrs = ['ops *', 'f.op', '*0x*'])
+self.expect('frame variable *f.op', DATA_TYPES_DISPLAYED_CORRECTLY,
substrs = ['ops', '*f.op', ''])
+self.expect('expr f.op', DATA_TYPES_DISPLAYED_CORREC
This revision was automatically updated to reflect the committed changes.
Closed by commit rL267923: XFail TestBitfields.py Python API tests. (authored
by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D19680?vs=55464&id=55465#toc
Repository:
rL LLVM
http://reviews.llvm.org/D196
chaoren updated this revision to Diff 55464.
chaoren added a comment.
Clickable links.
http://reviews.llvm.org/D19680
Files:
packages/Python/lldbsuite/test/lang/c/bitfields/TestBitfields.py
Index: packages/Python/lldbsuite/test/lang/c/bitfields/TestBitfields.py
==
Author: chaoren
Date: Thu Apr 28 14:40:19 2016
New Revision: 267923
URL: http://llvm.org/viewvc/llvm-project?rev=267923&view=rev
Log:
XFail TestBitfields.py Python API tests.
Summary:
Started failing after rL267895.
Possibly related to http://llvm.org/pr27510.
Reviewers: labath, tfiala
Subscrib
chaoren created this revision.
chaoren added reviewers: labath, tfiala.
chaoren added a subscriber: lldb-commits.
Started failing after r267895.
Possibly related to llvm.org/pr27510.
http://reviews.llvm.org/D19680
Files:
packages/Python/lldbsuite/test/lang/c/bitfields/TestBitfields.py
Index:
n wrote:
> > Ah, I see. Is that a known clang bug?
> >
> > On Mon, Apr 25, 2016 at 4:07 PM, Siva Chandra
> > wrote:
> >>
> >> AFAICT, happens only with Clang; Using g_static_var, say as "return
> >> g_static_var - 123;", fixes for me.
>
This revision was automatically updated to reflect the committed changes.
Closed by commit rL267492: Fix TestGetVariables.py. (authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D19511?vs=54940&id=54944#toc
Repository:
rL LLVM
http://reviews.llvm.org/D19511
Files:
lld
Author: chaoren
Date: Mon Apr 25 18:29:53 2016
New Revision: 267492
URL: http://llvm.org/viewvc/llvm-project?rev=267492&view=rev
Log:
Fix TestGetVariables.py.
Reviewers: sivachandra, clayborg
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D19511
Modified:
lldb/tr
chaoren created this revision.
chaoren added reviewers: sivachandra, clayborg.
chaoren added a subscriber: lldb-commits.
http://reviews.llvm.org/D19511
Files:
packages/Python/lldbsuite/test/python_api/frame/get-variables/TestGetVariables.py
packages/Python/lldbsuite/test/python_api/frame/get
Ah, I see. Is that a known clang bug?
On Mon, Apr 25, 2016 at 4:07 PM, Siva Chandra
wrote:
> AFAICT, happens only with Clang; Using g_static_var, say as "return
> g_static_var - 123;", fixes for me.
>
> On Mon, Apr 25, 2016 at 3:50 PM, Chaoren Lin via lldb-commits
> w
Is g_global_var necessarily static?
On Linux, we're only seeing 2 static variables when your test expects 3.
> static_names = ['static_var', 'g_global_var', 'static_var']
I'm guessing g_global_var isn't treated as static.
On Mon, Apr 25, 2016 at 2:54 PM, Greg Clayton via lldb-commits <
lldb-co
Author: chaoren
Date: Mon Apr 18 20:09:37 2016
New Revision: 266702
URL: http://llvm.org/viewvc/llvm-project?rev=266702&view=rev
Log:
Fix Windows build.
Modified:
lldb/trunk/source/Host/windows/Windows.cpp
Modified: lldb/trunk/source/Host/windows/Windows.cpp
URL:
http://llvm.org/viewvc/llvm
chaoren added a comment.
In http://reviews.llvm.org/D18481#383694, @zturner wrote:
> This seems very strange to me. What if someone specifies those flags on the
> command line in the makefile instead of via some environment variables?
I don't think there's anything we can do about that unless
chaoren accepted this revision.
chaoren added a comment.
This revision is now accepted and ready to land.
Seems legit.
http://reviews.llvm.org/D17402
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/lis
issue
>>> showed up. However, that bot as been broken since this change went in.
>>>
>>> I reverted it in r262156.
>>>
>>> Feel free to reapply if you have a suggested fix for the test failure
>>> introduced.
>>>
>>> Thanks!
>
gt; I reverted it in r262156.
>
> Feel free to reapply if you have a suggested fix for the test failure
> introduced.
>
> Thanks!
>
> -Todd
>
> On Fri, Feb 26, 2016 at 12:08 PM, Chaoren Lin via lldb-commits <
> lldb-commits@lists.llvm.org> wrote:
>
>> Hmm
This revision was automatically updated to reflect the committed changes.
Closed by commit rL262081: Register value is not necessarily scalar. (authored
by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D17658?vs=49237&id=49238#toc
Repository:
rL LLVM
http://reviews.llvm.org/D17
Author: chaoren
Date: Fri Feb 26 16:12:35 2016
New Revision: 262081
URL: http://llvm.org/viewvc/llvm-project?rev=262081&view=rev
Log:
Register value is not necessarily scalar.
Reviewers: aidan.dodds, mamai
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D17658
Modified
chaoren updated this revision to Diff 49237.
chaoren added a comment.
Check return value of GetScalarValue directly.
http://reviews.llvm.org/D17658
Files:
source/Expression/Materializer.cpp
Index: source/Expression/Materializer.cpp
chaoren created this revision.
chaoren added reviewers: aidan.dodds, mamai.
chaoren added a subscriber: lldb-commits.
http://reviews.llvm.org/D17658
Files:
source/Expression/Materializer.cpp
Index: source/Expression/Materializer.cpp
=
Hmm. Weird. That assert is failing on Linux:
http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/11833
On Fri, Feb 26, 2016 at 9:40 AM, Aidan Dodds via lldb-commits <
lldb-commits@lists.llvm.org> wrote:
> Author: aidandodds
> Date: Fri Feb 26 11:40:50 2016
> New Revision: 2620
It's okay. I think I fixed it with http://reviews.llvm.org/D17634
On Thu, Feb 25, 2016 at 6:55 PM, Jason Molenda wrote:
> Jim just headed home - if you don't mind, please revert the commit. He'll
> look into it tomorrow morning.
>
> J
>
>
> > On Feb 25,
This revision was automatically updated to reflect the committed changes.
Closed by commit rL261969: Clear alias argument vector for 'p' alias. (authored
by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D17634?vs=49147&id=49148#toc
Repository:
rL LLVM
http://reviews.llvm.org/D1
Author: chaoren
Date: Thu Feb 25 21:36:27 2016
New Revision: 261969
URL: http://llvm.org/viewvc/llvm-project?rev=261969&view=rev
Log:
Clear alias argument vector for 'p' alias.
Summary: This fixes the 'p' command which should be aliased to 'expresion --'.
Reviewers: jingham
Subscribers: lldb-co
chaoren created this revision.
chaoren added a reviewer: jingham.
chaoren added a subscriber: lldb-commits.
This fixes the 'p' command which should be aliased to 'expresion --'.
http://reviews.llvm.org/D17634
Files:
source/Interpreter/CommandInterpreter.cpp
Index: source/Interpreter/CommandIn
Hi Jim,
This broke a lot of our tests on Linux. E.g., "p foo" doesn't work.
http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/11803
Since the breakage is pretty severe, mind if I revert this first until the
problem can be fixed?
On Thu, Feb 25, 2016 at 5:42 PM Jim Ingham vi
Author: chaoren
Date: Tue Feb 23 21:15:21 2016
New Revision: 261718
URL: http://llvm.org/viewvc/llvm-project?rev=261718&view=rev
Log:
Fix TestCStrings for Linux with i386 inferiors.
Summary: Temporarily revert part of r261704.
Reviewers: spyffe
Subscribers: lldb-commits
Differential Revision:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL261718: Fix TestCStrings for Linux with i386 inferiors.
(authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D17557?vs=48867&id=48878#toc
Repository:
rL LLVM
http://reviews.llvm.o
chaoren updated this revision to Diff 48867.
chaoren added a comment.
Use target instead of sc.target_sp.get().
http://reviews.llvm.org/D17557
Files:
source/Expression/IRExecutionUnit.cpp
Index: source/Expression/IRExecutionUnit.cpp
===
chaoren created this revision.
chaoren added a reviewer: spyffe.
chaoren added a subscriber: lldb-commits.
Temporarily revert part of r261704.
http://reviews.llvm.org/D17557
Files:
source/Expression/IRExecutionUnit.cpp
Index: source/Expression/IRExecutionUnit.cpp
=
Hi Sean,
On Linux with i386 inferiors,
range.GetBaseAddress().GetCallableLoadAddress(target);
returns the wrong load address for strlen, while
candidate_sc.symbol->ResolveCallableAddress(*target);
returns the correct load address.
This is causing TestCStrings to fail on our bot:
http://lab.ll
Author: chaoren
Date: Thu Jan 28 21:48:08 2016
New Revision: 259153
URL: http://llvm.org/viewvc/llvm-project?rev=259153&view=rev
Log:
Fix TestDataFormatterScript for Linux.
Summary:
m_function_name will contain a dummy name for the auto-generated function from
the python script on Linux. Check fo
This revision was automatically updated to reflect the committed changes.
Closed by commit rL259153: Fix TestDataFormatterScript for Linux. (authored by
chaoren).
Changed prior to commit:
http://reviews.llvm.org/D16703?vs=46330&id=46340#toc
Repository:
rL LLVM
http://reviews.llvm.org/D16703
chaoren created this revision.
chaoren added a reviewer: granata.enrico.
chaoren added a subscriber: lldb-commits.
m_function_name will contain a dummy name for the auto-generated function from
the python script on Linux. Check for script name first.
http://reviews.llvm.org/D16703
Files:
sourc
Author: chaoren
Date: Thu Jan 28 14:25:46 2016
New Revision: 259086
URL: http://llvm.org/viewvc/llvm-project?rev=259086&view=rev
Log:
Fix build after rL259070.
Modified:
lldb/trunk/source/Plugins/ExpressionParser/Clang/ClangASTSource.cpp
Modified: lldb/trunk/source/Plugins/ExpressionParser/C
This revision was automatically updated to reflect the committed changes.
Closed by commit rL252391: Fix TestThreadSpecificBreakpoint.py on Linux after
rL252355. (authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D14472?vs=39615&id=39616#toc
Repository:
rL LLVM
http://r
Author: chaoren
Date: Fri Nov 6 20:16:31 2015
New Revision: 252391
URL: http://llvm.org/viewvc/llvm-project?rev=252391&view=rev
Log:
Fix TestThreadSpecificBreakpoint.py on Linux after rL252355.
Summary:
On Linux, if a thread-specific conditional breakpoint was hit, it
won't necessarily be the th
chaoren created this revision.
chaoren added reviewers: sivachandra, jingham.
chaoren added a subscriber: lldb-commits.
On Linux, if a thread-specific conditional breakpoint was hit, it
won't necessarily be the thread that hit the breakpoint itself that
evaluates the conditional expression, so the
Author: chaoren
Date: Fri Nov 6 16:30:30 2015
New Revision: 252353
URL: http://llvm.org/viewvc/llvm-project?rev=252353&view=rev
Log:
Fix Linux tests after r252348.
Modified:
lldb/trunk/packages/Python/lldbsuite/test/dotest.py
Modified: lldb/trunk/packages/Python/lldbsuite/test/dotest.py
URL
Author: chaoren
Date: Thu Nov 5 17:19:27 2015
New Revision: 252230
URL: http://llvm.org/viewvc/llvm-project?rev=252230&view=rev
Log:
Another import fix for OS X.
Modified:
lldb/trunk/packages/Python/lldbsuite/test/dotest.py
Modified: lldb/trunk/packages/Python/lldbsuite/test/dotest.py
URL:
rL252218 should hopefully fix it.
On Thu, Nov 5, 2015 at 1:53 PM, Chaoren Lin wrote:
> Hi, I think this changed caused all of the OS X tests to fail:
>
>
> http://lab.llvm.org:8011/builders/lldb-x86_64-darwin-13.4/builds/6969/steps/test1/logs/stdio
>
> with this error:
>
> Traceback (most recent
Author: chaoren
Date: Thu Nov 5 16:00:47 2015
New Revision: 252218
URL: http://llvm.org/viewvc/llvm-project?rev=252218&view=rev
Log:
Fix OS X tests.
Modified:
lldb/trunk/packages/Python/lldbsuite/test/dotest.py
Modified: lldb/trunk/packages/Python/lldbsuite/test/dotest.py
URL:
http://llvm.
Hi, I think this changed caused all of the OS X tests to fail:
http://lab.llvm.org:8011/builders/lldb-x86_64-darwin-13.4/builds/6969/steps/test1/logs/stdio
with this error:
Traceback (most recent call last):
File "/Users/lldb_build/lldbSlave/buildDir/scripts/../lldb/test/dotest.py",
line 7, in
Author: chaoren
Date: Thu Nov 5 14:45:29 2015
New Revision: 252205
URL: http://llvm.org/viewvc/llvm-project?rev=252205&view=rev
Log:
Completely avoid building Apple simulator on non-Darwin platforms.
Summary:
This is a resubmission of r252179, but correctly ignores the source
files for other pla
This revision was automatically updated to reflect the committed changes.
Closed by commit rL252205: Completely avoid building Apple simulator on
non-Darwin platforms. (authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D14389?vs=39394&id=39402#toc
Repository:
rL LLVM
ht
chaoren added a comment.
I haven't seen anything else that uses LLVM_OPTIONAL_SOURCES, but it's
conceivable that there might be items in the list already, and I would like to
avoid doing anything to it besides just appending.
http://reviews.llvm.org/D14389
__
chaoren added a comment.
It's not actually CMake complaining but the LLVM cmake scripts.
http://reviews.llvm.org/D14389
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
chaoren added a comment.
LLVM_OPTIONAL_SOURCES should avoid that.
http://reviews.llvm.org/D14389
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
chaoren created this revision.
chaoren added reviewers: granata.enrico, tberghammer, zturner, jingham.
chaoren added a subscriber: lldb-commits.
This is a resubmission of r252179, but correctly ignores the source
files for other platforms.
http://reviews.llvm.org/D14389
Files:
source/Initializ
Author: chaoren
Date: Thu Nov 5 12:49:02 2015
New Revision: 252182
URL: http://llvm.org/viewvc/llvm-project?rev=252182&view=rev
Log:
Revert "Do not build any of the simulator material on non-Darwin"
This breaks the cmake build on all non-darwin platforms.
This reverts commit ff6cc332dd6654620e3
Author: chaoren
Date: Thu Nov 5 12:49:03 2015
New Revision: 252183
URL: http://llvm.org/viewvc/llvm-project?rev=252183&view=rev
Log:
Use lldb::pid_t instead of pid_t.
Reviewers: granata.enrico
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D14388
Modified:
lldb/
This revision was automatically updated to reflect the committed changes.
Closed by commit rL252183: Use lldb::pid_t instead of pid_t. (authored by
chaoren).
Changed prior to commit:
http://reviews.llvm.org/D14388?vs=39384&id=39387#toc
Repository:
rL LLVM
http://reviews.llvm.org/D14388
Fil
chaoren created this revision.
chaoren added a reviewer: granata.enrico.
chaoren added a subscriber: lldb-commits.
http://reviews.llvm.org/D14388
Files:
source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSupport.h
source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSu
Author: chaoren
Date: Wed Nov 4 21:30:45 2015
New Revision: 252143
URL: http://llvm.org/viewvc/llvm-project?rev=252143&view=rev
Log:
Fix build for platforms without ObjC++.
Reviewers: jingham, jasonmolenda
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D14365
Modifie
This revision was automatically updated to reflect the committed changes.
Closed by commit rL252143: Fix build for platforms without ObjC++. (authored by
chaoren).
Changed prior to commit:
http://reviews.llvm.org/D14365?vs=39305&id=39309#toc
Repository:
rL LLVM
http://reviews.llvm.org/D1436
chaoren created this revision.
chaoren added reviewers: jingham, jasonmolenda.
chaoren added a subscriber: lldb-commits.
http://reviews.llvm.org/D14365
Files:
source/Plugins/Platform/MacOSX/CMakeLists.txt
source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSupport.mm
Index:
sour
t;> Jim
> > >>
> > >>> On Nov 4, 2015, at 5:43 PM, Jason Molenda
> wrote:
> > >>>
> > >>> We do the same thing in other MacOSX platform files where they
> depend on Apple specific libraries.
> > >>>
> > >>
Author: chaoren
Date: Wed Nov 4 20:17:21 2015
New Revision: 252133
URL: http://llvm.org/viewvc/llvm-project?rev=252133&view=rev
Log:
Fix TestGoFormatters.py.
Modified:
lldb/trunk/packages/Python/lldbsuite/test/lang/go/formatters/TestGoFormatters.py
Modified:
lldb/trunk/packages/Python/lld
gt; On Nov 4, 2015, at 5:39 PM, Jim Ingham via lldb-commits <
> lldb-commits@lists.llvm.org> wrote:
> >>>>
> >>>> Why are we trying to build this at all on Linux? Seems odd to build
> an empty file rather than just not build it.
> >>>>
> >&
this at all on Linux? Seems odd to build
> an empty file rather than just not build it.
> >>>
> >>> Jim
> >>>
> >>>> On Nov 4, 2015, at 5:23 PM, Chaoren Lin via lldb-commits <
> lldb-commits@lists.llvm.org> wrote:
> >>&g
chaoren added a comment.
Yeah, it seems like it's been passing for gcc-4.9 for a while now. Thanks
for checking!
Repository:
rL LLVM
http://reviews.llvm.org/D12115
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cg
Yeah, it seems like it's been passing for gcc-4.9 for a while now. Thanks
for checking!
On Wed, Nov 4, 2015 at 5:15 PM, Dawn Perchik wrote:
> dawn marked an inline comment as done.
> dawn added a comment.
>
> In http://reviews.llvm.org/D12115#248069, @chaoren wrote:
>
> > Could you please take a
Author: chaoren
Date: Wed Nov 4 19:23:19 2015
New Revision: 252124
URL: http://llvm.org/viewvc/llvm-project?rev=252124&view=rev
Log:
Fix build on Linux.
Modified:
lldb/trunk/source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSupport.mm
Modified:
lldb/trunk/source/Plugins/Plat
This revision was automatically updated to reflect the committed changes.
Closed by commit rL251589: Correctly include LLVM_EXTERNAL_CLANG_SOURCE_DIR.
(authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D14166?vs=38700&id=38702#toc
Repository:
rL LLVM
http://reviews.llvm
Author: chaoren
Date: Wed Oct 28 19:48:38 2015
New Revision: 251589
URL: http://llvm.org/viewvc/llvm-project?rev=251589&view=rev
Log:
Correctly include LLVM_EXTERNAL_CLANG_SOURCE_DIR.
Reviewers: sas, ovyalov
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D14166
Modifi
chaoren resigned from this revision.
chaoren removed a reviewer: chaoren.
chaoren added a comment.
Seems legit. But I've never built standalone, so I probably shouldn't be a
reviewer for this.
http://reviews.llvm.org/D13819
___
lldb-commits mailing
chaoren created this revision.
chaoren added reviewers: sas, ovyalov.
chaoren added a subscriber: lldb-commits.
http://reviews.llvm.org/D14166
Files:
cmake/modules/LLDBConfig.cmake
Index: cmake/modules/LLDBConfig.cmake
===
--- cma
This revision was automatically updated to reflect the committed changes.
Closed by commit rL251439: Use accept4 workaround for MIPS Android build.
(authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D14127?vs=38581&id=38582#toc
Repository:
rL LLVM
http://reviews.llvm.or
Author: chaoren
Date: Tue Oct 27 14:17:35 2015
New Revision: 251439
URL: http://llvm.org/viewvc/llvm-project?rev=251439&view=rev
Log:
Use accept4 workaround for MIPS Android build.
Summary: Similar to http://reviews.llvm.org/rL242319, which was for ARM.
Reviewers: chying, ovyalov
Subscribers: a
chaoren created this revision.
chaoren added reviewers: chying, ovyalov.
chaoren added a subscriber: lldb-commits.
Herald added subscribers: srhines, danalbert, tberghammer, aemerson.
Similar to http://reviews.llvm.org/rL242319, which was for ARM.
http://reviews.llvm.org/D14127
Files:
cmake/pl
Author: chaoren
Date: Tue Oct 6 19:01:06 2015
New Revision: 249486
URL: http://llvm.org/viewvc/llvm-project?rev=249486&view=rev
Log:
Fix Android build.
Modified:
lldb/trunk/CMakeLists.txt
lldb/trunk/cmake/platforms/Android.cmake
lldb/trunk/source/CMakeLists.txt
Modified: lldb/trunk/
Does timeout not work?
On Tue, Sep 29, 2015 at 4:06 PM, Todd Fiala via lldb-commits <
lldb-commits@lists.llvm.org> wrote:
> Author: tfiala
> Date: Tue Sep 29 18:06:56 2015
> New Revision: 248846
>
> URL: http://llvm.org/viewvc/llvm-project?rev=248846&view=rev
> Log:
> Skipping TestAttachDenied.py
Author: chaoren
Date: Tue Sep 22 12:37:34 2015
New Revision: 248301
URL: http://llvm.org/viewvc/llvm-project?rev=248301&view=rev
Log:
Update TestChangeProcessGroup to remove obsolete workaround.
Expression evaluation in syscalls should work now.
Reviewers: labath
Subscribers: lldb-commits
Diff
This revision was automatically updated to reflect the committed changes.
Closed by commit rL248301: Update TestChangeProcessGroup to remove obsolete
workaround. (authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D12976?vs=35304&id=35391#toc
Repository:
rL LLVM
http://r
chaoren updated this revision to Diff 35304.
chaoren added a comment.
Remove existing workaround.
http://reviews.llvm.org/D12976
Files:
test/functionalities/process_group/TestChangeProcessGroup.py
Index: test/functionalities/process_group/TestChangeProcessGroup.py
===
chaoren added a comment.
Do we still need the original stepInstruction?
http://reviews.llvm.org/D12976
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: chaoren
Date: Fri Sep 18 19:57:11 2015
New Revision: 248075
URL: http://llvm.org/viewvc/llvm-project?rev=248075&view=rev
Log:
Fix Linux bot.
Modified:
lldb/trunk/test/dosep.py
Modified: lldb/trunk/test/dosep.py
URL:
http://llvm.org/viewvc/llvm-project/lldb/trunk/test/dosep.py?rev=24
chaoren added a comment.
I thought this was supposed to be fixed, but TestChangeProcessGroup,
TestExpressionInSyscall, and TestRegisters just started failing for me recently
because of this.
http://reviews.llvm.org/D12976
___
lldb-commits mailing
chaoren updated this revision to Diff 35113.
chaoren added a comment.
Remove old line.
http://reviews.llvm.org/D12976
Files:
test/functionalities/process_group/TestChangeProcessGroup.py
Index: test/functionalities/process_group/TestChangeProcessGroup.py
==
chaoren created this revision.
chaoren added a reviewer: labath.
chaoren added a subscriber: lldb-commits.
http://reviews.llvm.org/D12976
Files:
test/functionalities/process_group/TestChangeProcessGroup.py
Index: test/functionalities/process_group/TestChangeProcessGroup.py
Author: chaoren
Date: Thu Sep 17 13:38:55 2015
New Revision: 247914
URL: http://llvm.org/viewvc/llvm-project?rev=247914&view=rev
Log:
Fix TestMiSymbol for gcc-4.9 test.
Modified:
lldb/trunk/test/tools/lldb-mi/symbol/TestMiSymbol.py
Modified: lldb/trunk/test/tools/lldb-mi/symbol/TestMiSymbol.
chaoren added a subscriber: chaoren.
chaoren added a comment.
Could you please take a look at these issues?
TestMiSymbol started to fail on Linux with gcc-4.9:
http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/6387
Comment at: lldb/trunk/tools/lldb-mi/MIUt
Author: chaoren
Date: Wed Sep 16 18:36:41 2015
New Revision: 247863
URL: http://llvm.org/viewvc/llvm-project?rev=247863&view=rev
Log:
XFAIL TestCppNsImport for gcc-4.9 inclusive.
Modified:
lldb/trunk/test/lang/cpp/nsimport/TestCppNsImport.py
Modified: lldb/trunk/test/lang/cpp/nsimport/TestCp
This revision was automatically updated to reflect the committed changes.
Closed by commit rL247857: Fix Makefile for Windows to Android tests. (authored
by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D12909?vs=34905&id=34929#toc
Repository:
rL LLVM
http://reviews.llvm.org/D1
Author: chaoren
Date: Wed Sep 16 16:51:51 2015
New Revision: 247857
URL: http://llvm.org/viewvc/llvm-project?rev=247857&view=rev
Log:
Fix Makefile for Windows to Android tests.
Reviewers: zturner
Subscribers: tberghammer, lldb-commits, danalbert
Differential Revision: http://reviews.llvm.org/D1
Author: chaoren
Date: Wed Sep 16 16:32:48 2015
New Revision: 247855
URL: http://llvm.org/viewvc/llvm-project?rev=247855&view=rev
Log:
XFAIL TestCppNsImport for gcc-4.9.
Works for gcc-4.8. A bug in gcc perhaps.
Modified:
lldb/trunk/test/lang/cpp/nsimport/TestCppNsImport.py
Modified: lldb/tru
chaoren created this revision.
chaoren added a reviewer: zturner.
chaoren added a subscriber: lldb-commits.
Herald added subscribers: danalbert, tberghammer.
http://reviews.llvm.org/D12909
Files:
test/make/Makefile.rules
Index: test/make/Makefile.rules
=
Author: chaoren
Date: Tue Sep 15 20:20:34 2015
New Revision: 247766
URL: http://llvm.org/viewvc/llvm-project?rev=247766&view=rev
Log:
Fix off-by-one size check.
Modified:
lldb/trunk/source/Expression/IRInterpreter.cpp
Modified: lldb/trunk/source/Expression/IRInterpreter.cpp
URL:
http://llvm
Author: chaoren
Date: Mon Sep 14 19:52:00 2015
New Revision: 247648
URL: http://llvm.org/viewvc/llvm-project?rev=247648&view=rev
Log:
Revert "Fixup XFAIL marking on TestConstVariables for clang version"
Summary:
Linux with ToT clang (3.8) fails.
This reverts commit r247633.
Reviewers: tfiala
S
This revision was automatically updated to reflect the committed changes.
Closed by commit rL247648: Revert "Fixup XFAIL marking on TestConstVariables
for clang version" (authored by chaoren).
Changed prior to commit:
http://reviews.llvm.org/D12868?vs=34764&id=34766#toc
Repository:
rL LLVM
1 - 100 of 164 matches
Mail list logo
