Author: Sylvestre Ledru
Date: 2024-12-11T00:00:44+01:00
New Revision: a2fb70523ac310af6d8a4d9663dfe7c9cd370c53
URL:
https://github.com/llvm/llvm-project/commit/a2fb70523ac310af6d8a4d9663dfe7c9cd370c53
DIFF:
https://github.com/llvm/llvm-project/commit/a2fb70523ac310af6d8a4d9663dfe7c9cd370c53
sylvestre wrote:
it also caused https://github.com/llvm/llvm-project/issues/119467
I am sorry but I pushed a revert of this PR
9a9c1d4a6155a96ce9be494cec7e25731d36b33e
and your followup patches:
e2bb47443d2e5c022c7851dd6029e3869fc8835c
0be33484853557bc0fd9dfb94e0b6c15dda136ce
d300337e93da4ed96b
sylvestre wrote:
@Jlalond it breaks the build on i386 on:
https://github.com/llvm/llvm-project/issues/117327
https://github.com/llvm/llvm-project/pull/110065
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
https://github.com/sylvestre edited
https://github.com/llvm/llvm-project/pull/108719
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
sylvestre wrote:
Seems that it caused https://github.com/llvm/llvm-project/issues/73580 no?
https://github.com/llvm/llvm-project/pull/68845
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-
sylvestre.ledru added a comment.
Maybe add it to the release notes?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137337/new/
https://reviews.llvm.org/D137337
___
lldb-commits mailing list
lldb-commits@l
sylvestre.ledru added a comment.
i hope not too many people rely on it.
LGTM but please add it to the release notes
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125860/new/
https://reviews.llvm.org/D125860
___
lldb-commits mailing list
lldb-
sylvestre.ledru added a comment.
Log:
https://llvm-jenkins.debian.net/job/llvm-toolchain-impish-14-binaries/12/architecture=amd64,distribution=impish,label=amd64/console
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119186/new/
https://reviews.llvm
sylvestre.ledru added a comment.
Debian is still broken with this patch on -14:
lib/liblldbPluginProcessGDBRemote.a(GDBRemoteCommunication.cpp.o):GDBRemoteCommunication.cpp:function
lldb_private::process_gdb_remote::GDBRemoteCommunication::DecompressPacket():
error: undefined reference to 'i
sylvestre.ledru added a comment.
Seems that it broke apt.llvm.org on ubuntu bionic with Python 3.6:
I reported this issue: https://github.com/llvm/llvm-project/issues/53291
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114722/new/
https://reviews.l
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG427f6e11e68a: [lldb][docs] Update instructions for building
libc++ (authored by torarnv, committed by sylvestre.ledru).
Repository:
rG LLVM Github
Author: Tor Arne Vestbø
Date: 2021-12-16T16:50:42+01:00
New Revision: 427f6e11e68a2c57c12fe9049a939752d76860be
URL:
https://github.com/llvm/llvm-project/commit/427f6e11e68a2c57c12fe9049a939752d76860be
DIFF:
https://github.com/llvm/llvm-project/commit/427f6e11e68a2c57c12fe9049a939752d76860be.dif
sylvestre.ledru added a comment.
I will push it for you. What do you want to use as full name + email ?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115877/new/
https://reviews.llvm.org/D115877
___
lldb
sylvestre.ledru accepted this revision.
sylvestre.ledru added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115877/new/
https://reviews.llvm.org/D115877
sylvestre.ledru added a comment.
> Maybe it would be better to disable the binary completely (in cmake) for
> unsupported architectures instead?
+1, having an error message like "lldb isn't supported on arch foo" would save
hours of work :)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org
Author: Kirill Shmakov
Date: 2021-08-21T12:24:29+02:00
New Revision: 2cc1198e36d042f9a5cd8d66773d7e85d1d0c62f
URL:
https://github.com/llvm/llvm-project/commit/2cc1198e36d042f9a5cd8d66773d7e85d1d0c62f
DIFF:
https://github.com/llvm/llvm-project/commit/2cc1198e36d042f9a5cd8d66773d7e85d1d0c62f.diff
sylvestre.ledru added a comment.
This change doesn't build with gcc 5.3.1.
More details: https://bugs.llvm.org/show_bug.cgi?id=48869
It would be great if you have could have a look! thanks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92759/new/
h
This revision was automatically updated to reflect the committed changes.
Closed by commit rG43e451f9f316: Fix gendered documentation (authored by
pedro.gonnet, committed by sylvestre.ledru).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78807/new/
Author: Pedro Gonnet
Date: 2020-10-31T12:17:19+01:00
New Revision: 43e451f9f316fd224d88899d45313a73ff11291e
URL:
https://github.com/llvm/llvm-project/commit/43e451f9f316fd224d88899d45313a73ff11291e
DIFF:
https://github.com/llvm/llvm-project/commit/43e451f9f316fd224d88899d45313a73ff11291e.diff
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG0784e17f1b4a: Remove .svn from exclude list as we moved to
git (authored by sylvestre.ledru).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE L
Author: Sylvestre Ledru
Date: 2020-10-21T16:09:21+02:00
New Revision: 0784e17f1b4ac6b613ebf1fb1fb9b0dc9d0776ec
URL:
https://github.com/llvm/llvm-project/commit/0784e17f1b4ac6b613ebf1fb1fb9b0dc9d0776ec
DIFF:
https://github.com/llvm/llvm-project/commit
sylvestre.ledru added a comment.
ed if you want to approve it ;)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89859/new/
https://reviews.llvm.org/D89859
___
lldb-commits mailing list
lldb-commits@lists.
sylvestre.ledru created this revision.
sylvestre.ledru added a reviewer: serge-sans-paille.
Herald added subscribers: llvm-commits, lldb-commits, cfe-commits, mgorny.
Herald added a reviewer: bollu.
Herald added projects: clang, LLDB, LLVM.
sylvestre.ledru requested review of this revision.
Herald
sylvestre.ledru accepted this revision.
sylvestre.ledru added a comment.
This revision is now accepted and ready to land.
thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78807/new/
https://reviews.llvm.org/D78807
__
sylvestre.ledru added inline comments.
Comment at: lldb/source/Utility/XcodeSDK.cpp:72
+ llvm::VersionTuple version = ParseSDKVersion(input);
+ return {sdk, version};
+}
With older version of the libcstdc++ (some Ubuntu LTS), it fails with:
```
error: chosen
Author: Sylvestre Ledru
Date: 2019-11-16T14:47:20+01:00
New Revision: 9b40a7f3bf7198dd64403cbd83478087e72f994b
URL:
https://github.com/llvm/llvm-project/commit/9b40a7f3bf7198dd64403cbd83478087e72f994b
DIFF:
https://github.com/llvm/llvm-project/commit
Author: sylvestre
Date: Sat Oct 5 01:22:40 2019
New Revision: 373819
URL: http://llvm.org/viewvc/llvm-project?rev=373819&view=rev
Log:
Do not install lit-cpuid
Summary:
AFAIK, lit-cpuid is used by the tests.
Installing it causes LLVMExports*.cmake files to depend
on this program.
It causes some
sylvestre.ledru added a comment.
Sure, happy to try, do you have some docs on this? Thanks :)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68537/new/
https://reviews.llvm.org/D68537
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
sylvestre.ledru created this revision.
sylvestre.ledru added a reviewer: mgorny.
Herald added a subscriber: delcypher.
AFAIK, lit-cpuid is used by the tests.
Installing it causes LLVMExports*.cmake files to depend
on this program.
It causes some serious packaging issues as it would means
that llv
sylvestre.ledru added a comment.
@labath
I am building with AVR as experimental target and this change probably broke
the build.
In file included from
/home/sylvestre/dev/debian/pkg-llvm/llvm-toolchain/branches/llvm-toolchain-snapshot_10~svn372978/tools/lldb/source/API/SystemInitializerFull.
Author: sylvestre
Date: Thu Aug 29 00:19:14 2019
New Revision: 370302
URL: http://llvm.org/viewvc/llvm-project?rev=370302&view=rev
Log:
fix the bugzilla url
Modified:
lldb/trunk/docs/lldb.1
Modified: lldb/trunk/docs/lldb.1
URL:
http://llvm.org/viewvc/llvm-project/lldb/trunk/docs/lldb.1?rev=
Author: sylvestre
Date: Thu Aug 29 00:20:08 2019
New Revision: 370303
URL: http://llvm.org/viewvc/llvm-project?rev=370303&view=rev
Log:
use https for llvm.org in the doc
Modified:
lldb/trunk/docs/index.rst
lldb/trunk/docs/resources/build.rst
lldb/trunk/docs/resources/source.rst
Modif
Author: sylvestre
Date: Thu Aug 29 00:14:09 2019
New Revision: 370301
URL: http://llvm.org/viewvc/llvm-project?rev=370301&view=rev
Log:
Provide a real link to the doc and remove old information
Modified:
lldb/trunk/docs/resources/download.rst
Modified: lldb/trunk/docs/resources/download.rst
sylvestre.ledru added a comment.
On i386
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55434/new/
https://reviews.llvm.org/D55434
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
sylvestre.ledru added a comment.
@labath This broke lldb on Debian stable:
In file included from
/build/llvm-toolchain-snapshot-8~svn350764/tools/lldb/source/Utility/DataExtractor.cpp:10:
/build/llvm-toolchain-snapshot-8~svn350764/tools/lldb/include/lldb/Utility/DataExtractor.h:1099:29:
er
sylvestre.ledru added a comment.
This fixed the issue, thanks!
https://reviews.llvm.org/D39246
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: sylvestre
Date: Mon Jun 19 01:57:54 2017
New Revision: 305664
URL: http://llvm.org/viewvc/llvm-project?rev=305664&view=rev
Log:
Fix a -Wmisleading-indentation warning
Modified:
lldb/trunk/source/Commands/CommandObjectFrame.cpp
Modified: lldb/trunk/source/Commands/CommandObjectFrame.
Author: sylvestre
Date: Thu Feb 16 12:45:27 2017
New Revision: 295352
URL: http://llvm.org/viewvc/llvm-project?rev=295352&view=rev
Log:
Reindent the code to avoid an error with gcc:
error: this ‘else’ clause does not guard... [-Werror=misleading-indentation]
Modified:
lldb/trunk/source/Host/
sylvestre.ledru added a comment.
Like llvm or lldb in general, just integrate it into liblldb?
By the way, liblldb-intel-mpxtable.so-5.0 is incorrect should be
liblldb-intel-mpxtable.so.5.0
Repository:
rL LLVM
https://reviews.llvm.org/D29078
__
sylvestre.ledru added a comment.
This change is creating a new library
usr/lib/llvm-5.0/bin/liblldb-intel-mpxtable.so on GNU/Linux 64b
That sounds pretty specific, is that really something that we want to do?
Repository:
rL LLVM
https://reviews.llvm.org/D29078
sylvestre.ledru abandoned this revision.
sylvestre.ledru added a comment.
Thanks!
https://reviews.llvm.org/D28155
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
sylvestre.ledru added a comment.
Probably, I did it the easy way
I don't know what ARG_GENERATE_INSTALL in cmake/modules/AddLLDB.cmake is for
https://reviews.llvm.org/D28155
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llv
sylvestre.ledru created this revision.
sylvestre.ledru added a reviewer: beanz.
sylvestre.ledru added a subscriber: LLDB.
Herald added a subscriber: mgorny.
For some reasons, the usage of add_lldb_tool instead of add_lldb_executable
causes these two binaries not be installed.
This patch fixes the
Author: sylvestre
Date: Sat Dec 17 07:04:35 2016
New Revision: 290045
URL: http://llvm.org/viewvc/llvm-project?rev=290045&view=rev
Log:
Fix the cmake declaration syntax
Modified:
lldb/trunk/scripts/Python/modules/CMakeLists.txt
Modified: lldb/trunk/scripts/Python/modules/CMakeLists.txt
URL:
Author: sylvestre
Date: Sat Dec 17 06:48:06 2016
New Revision: 290044
URL: http://llvm.org/viewvc/llvm-project?rev=290044&view=rev
Log:
Support of lldb on Kfreebsd
Summary: Patch by Pino Toscano. Reported in http://bugs.debian.org/835665
Reviewers: tfiala, emaste
Subscribers: beanz, mgorny, em
sylvestre.ledru added a comment.
Can we land this patch now? Thanks
Repository:
rL LLVM
https://reviews.llvm.org/D24863
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
sylvestre.ledru updated this revision to Diff 77714.
https://reviews.llvm.org/D23977
Files:
cmake/LLDBDependencies.cmake
cmake/modules/LLDBConfig.cmake
scripts/Python/modules/CMakeLists.txt
scripts/utilsOsType.py
Index: scripts/utilsOsType.py
sylvestre.ledru updated this revision to Diff 72758.
sylvestre.ledru added a comment.
Herald added subscribers: mgorny, beanz.
Updated by Pino!
https://reviews.llvm.org/D23977
Files:
cmake/LLDBDependencies.cmake
cmake/modules/LLDBConfig.cmake
scripts/Python/modules/CMakeLists.txt
script
Author: sylvestre
Date: Wed Aug 31 02:16:56 2016
New Revision: 280200
URL: http://llvm.org/viewvc/llvm-project?rev=280200&view=rev
Log:
Update the Linux code to reflect the changes done by zturner in r280139
Modified:
lldb/trunk/source/Plugins/Process/Linux/NativeProcessLinux.cpp
Modified: l
sylvestre.ledru created this revision.
sylvestre.ledru added a reviewer: tfiala.
sylvestre.ledru added a subscriber: LLDB.
sylvestre.ledru added a project: LLDB.
Herald added subscribers: srhines, danalbert, tberghammer.
Patch by Pino Toscano. Reported in http://bugs.debian.org/835665
https://rev
Author: sylvestre
Date: Sun Nov 8 16:30:34 2015
New Revision: 252429
URL: http://llvm.org/viewvc/llvm-project?rev=252429&view=rev
Log:
autotools: Also link the new go libraries
Modified:
lldb/trunk/lib/Makefile
lldb/trunk/source/Plugins/Makefile
Modified: lldb/trunk/lib/Makefile
URL:
h
Author: sylvestre
Date: Thu Oct 8 06:17:01 2015
New Revision: 249681
URL: http://llvm.org/viewvc/llvm-project?rev=249681&view=rev
Log:
Fix the undefined symbol issue with go linking with autotools: build the
LanguageRuntime for Go + link against it
Modified:
lldb/trunk/lib/Makefile
lldb
Author: sylvestre
Date: Wed Oct 7 13:26:29 2015
New Revision: 249585
URL: http://llvm.org/viewvc/llvm-project?rev=249585&view=rev
Log:
Revert commit -r249558 as it is not fixing the issue :/
Modified:
lldb/trunk/lib/Makefile
Modified: lldb/trunk/lib/Makefile
URL:
http://llvm.org/viewvc/llv
Author: sylvestre
Date: Wed Oct 7 11:13:10 2015
New Revision: 249558
URL: http://llvm.org/viewvc/llvm-project?rev=249558&view=rev
Log:
Fix a linking issue on lib lldb linking with the new Go support
Modified:
lldb/trunk/lib/Makefile
Modified: lldb/trunk/lib/Makefile
URL:
http://llvm.org/vi
Author: sylvestre
Date: Fri Aug 28 07:24:07 2015
New Revision: 246294
URL: http://llvm.org/viewvc/llvm-project?rev=246294&view=rev
Log:
Avoid usage of F_DUPFD_CLOEXEC where not available (e.g. kfreebsd*)
Summary:
kfreebsd doesn't have F_DUPFD_CLOEXEC, so use it conditionally.
Author: Emilio Pozu
Author: sylvestre
Date: Wed Aug 5 07:46:53 2015
New Revision: 244039
URL: http://llvm.org/viewvc/llvm-project?rev=244039&view=rev
Log:
Bug 24332 - Fix the build of lldb under GNU/Linux when using autotools
Reviewers: loladiro, jingham, emaste, zturner, brucem
Subscribers: tberghammer, danalbert
56 matches
Mail list logo
