Page MenuHomePhabricator

sylvestre.ledru (Sylvestre Ledru)
User

Projects

User does not belong to any projects.

User Details

User Since
Mar 8 2013, 1:12 AM (411 w, 3 d)

Recent Activity

Today

sylvestre.ledru added a comment to D92759: [lldb] Introduce separate scratch ASTs for debug info types and types imported from C++ modules..

This change doesn't build with gcc 5.3.1.
More details: https://bugs.llvm.org/show_bug.cgi?id=48869

Mon, Jan 25, 1:04 AM · Restricted Project

Wed, Jan 20

sylvestre.ledru added a comment to D94941: Add minor version to libclang.so and libclang-cpp.so SONAME.

I guess this will be clearly mentioned in the release notes :)

Wed, Jan 20, 8:41 AM · Restricted Project
sylvestre.ledru accepted D94941: Add minor version to libclang.so and libclang-cpp.so SONAME.

Let's do it then :)

Wed, Jan 20, 8:41 AM · Restricted Project
sylvestre.ledru added a comment to D94941: Add minor version to libclang.so and libclang-cpp.so SONAME.

Yeah, my question is more "is that enough for distro?"
https://www.debian.org/doc/manuals/maint-guide/advanced.en.html
https://www.gnu.org/software/libtool/manual/html_node/Updating-version-info.html

Wed, Jan 20, 8:25 AM · Restricted Project
sylvestre.ledru added a comment to D94941: Add minor version to libclang.so and libclang-cpp.so SONAME.

I might be wrong but if the ABI is incompatible, are we not supposed to update the SONAME itself?

Wed, Jan 20, 8:14 AM · Restricted Project

Dec 22 2020

sylvestre.ledru added a comment to D92245: -fstack-clash-protection: Return an actual error when used on unsupported OS.

Thanks for the revert!

Dec 22 2020, 3:59 AM · Restricted Project, Restricted Project
sylvestre.ledru committed rG4d59c8fdb955: -fstack-clash-protection: Return an actual error when used on unsupported OS (authored by sylvestre.ledru).
-fstack-clash-protection: Return an actual error when used on unsupported OS
Dec 22 2020, 3:06 AM
sylvestre.ledru closed D92245: -fstack-clash-protection: Return an actual error when used on unsupported OS.
Dec 22 2020, 3:06 AM · Restricted Project, Restricted Project
sylvestre.ledru updated the diff for D92245: -fstack-clash-protection: Return an actual error when used on unsupported OS.

Error only on Windows

Dec 22 2020, 2:27 AM · Restricted Project, Restricted Project

Dec 16 2020

sylvestre.ledru added a comment to D93351: [llvm-shlib] Build backend libraries as loadable modules.

I love the idea. Many thanks for implementing it!
I think it should go in the release notes.

Dec 16 2020, 8:45 AM · Restricted Project

Dec 14 2020

sylvestre.ledru added a comment to D92245: -fstack-clash-protection: Return an actual error when used on unsupported OS.

@serge-sans-paille ok to push this? :)
merci

Dec 14 2020, 2:02 AM · Restricted Project, Restricted Project

Dec 13 2020

sylvestre.ledru added a comment to D93055: [OpenMP] Add time profiling for libomptarget.

I think there is something wrong in this patch
It creates a new library called:
usr/lib/llvm-12/lib/libomptarget.so.12

Dec 13 2020, 12:43 AM · Restricted Project

Dec 11 2020

sylvestre.ledru added a comment to D37979: ClangFormat - Add one space in inline empty function that can be wrapped on a single line. .

@mhq199657 we (Mozilla/Firefox) moved to the Google Coding style. So, we are less interested by this change now.

Dec 11 2020, 12:29 AM

Dec 4 2020

sylvestre.ledru added a comment to D92652: [clang-tidy][docs] Update check options with boolean values instead of non-zero/0/1.

Thanks for doing that :)

Dec 4 2020, 7:24 AM · Restricted Project

Nov 30 2020

sylvestre.ledru updated the diff for D92245: -fstack-clash-protection: Return an actual error when used on unsupported OS.

extend the test (thanks serge)

Nov 30 2020, 9:29 AM · Restricted Project, Restricted Project
sylvestre.ledru updated the diff for D92245: -fstack-clash-protection: Return an actual error when used on unsupported OS.

Fix the error message

Nov 30 2020, 2:07 AM · Restricted Project, Restricted Project

Nov 28 2020

sylvestre.ledru updated the diff for D92245: -fstack-clash-protection: Return an actual error when used on unsupported OS.

clang-format

Nov 28 2020, 5:42 AM · Restricted Project, Restricted Project
sylvestre.ledru accepted D91799: [clang-format] State where clang-format-diff.py should be run from.
Nov 28 2020, 5:39 AM · Restricted Project

Nov 27 2020

sylvestre.ledru requested review of D92245: -fstack-clash-protection: Return an actual error when used on unsupported OS.
Nov 27 2020, 12:51 PM · Restricted Project, Restricted Project
sylvestre.ledru committed rGfa39b9eab1fe: TargetProcessControl.cpp - Remove warning: extra ‘;’ (authored by sylvestre.ledru).
TargetProcessControl.cpp - Remove warning: extra ‘;’
Nov 27 2020, 9:20 AM

Nov 18 2020

sylvestre.ledru added a comment to D85600: [clang-format] use spaces for alignment of binary/ternary expressions with UT_AlignWithSpaces.

I am not 100% that it is thanks to this patch but reformatting Firefox code with clang-format 11 significantly improves the readability of the usage
of ternary operators. See: https://phabricator.services.mozilla.com/D90795

Nov 18 2020, 12:55 AM · Restricted Project, Restricted Project, Restricted Project

Nov 17 2020

sylvestre.ledru added a comment to D91598: Allow packager to provide a default configuration file for clang.

@sylvestre.ledru Can you drop the picture?

Nov 17 2020, 9:04 AM
sylvestre.ledru added a comment to D91598: Allow packager to provide a default configuration file for clang.

Great idea. Do you have an example of what the configuration could look like on a distro?
Merci!

Nov 17 2020, 3:22 AM

Nov 7 2020

sylvestre.ledru accepted D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy.

Sounds great. You might want to ask feedback to other reviews too!

Nov 7 2020, 6:35 AM · Restricted Project, Restricted Project
sylvestre.ledru requested changes to D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy.

Thanks.
Could you please also update the release notes for 12?

Nov 7 2020, 4:41 AM · Restricted Project, Restricted Project
sylvestre.ledru added a comment to D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy.

FWIW, I am doing this change already in Debian & Ubuntu ( https://salsa.debian.org/pkg-llvm-team/llvm-toolchain/-/blob/10/debian/clang-tidy-X.Y.links.in#L2 )

Nov 7 2020, 12:46 AM · Restricted Project, Restricted Project

Oct 31 2020

sylvestre.ledru committed rG43e451f9f316: Fix gendered documentation (authored by pedro.gonnet).
Fix gendered documentation
Oct 31 2020, 4:17 AM
sylvestre.ledru closed D78807: Fix gendered documentation.
Oct 31 2020, 4:17 AM · Restricted Project, Restricted Project
sylvestre.ledru resigned from D23975: Support of kfreebsd & GNU Hurd in llvm-shlib.

I think it is now fixed

Oct 31 2020, 4:07 AM
sylvestre.ledru resigned from D34100: fix the bug of opening the file with the bytes can't be decoded to 'utf-8'.

Sorry, I am not a good reviewer for this file

Oct 31 2020, 4:07 AM · Restricted Project, Restricted Project
sylvestre.ledru accepted D58118: Fix llvm-config behavior when LLVM_TOOLS_INSTALL_DIR has a depth greater than one.
Oct 31 2020, 4:06 AM · Restricted Project
sylvestre.ledru added a comment to D58118: Fix llvm-config behavior when LLVM_TOOLS_INSTALL_DIR has a depth greater than one.

I guess it isn't possible to write a test for this?

Oct 31 2020, 4:06 AM · Restricted Project
sylvestre.ledru accepted D88084: [clang-format] Changed default styles BraceWrappping bool table to directly use variables.

Much better

Oct 31 2020, 4:05 AM · Restricted Project, Restricted Project
sylvestre.ledru accepted D89481: [scan-build] Fix clang++ pathname again.

Looks good to me

Oct 31 2020, 4:04 AM · Restricted Project
sylvestre.ledru resigned from D89974: [driver][CUDA] Use CMake's FindCUDA as default --cuda-path..
Oct 31 2020, 4:03 AM · Restricted Project, Restricted Project

Oct 29 2020

sylvestre.ledru committed rG5d1eae7d23e3: Add support of the next Ubuntu (Ubuntu 21.04 - Hirsute Hippo) (authored by sylvestre.ledru).
Add support of the next Ubuntu (Ubuntu 21.04 - Hirsute Hippo)
Oct 29 2020, 11:12 AM

Oct 21 2020

sylvestre.ledru added a comment to D88922: [CMake] Avoid accidental C++ standard library dependency in sanitizers.

I can reproduce the build issue on apt.llvm.org
Full log:
https://llvm-jenkins.debian.net/job/llvm-toolchain-xenial-binaries/architecture=amd64,distribution=xenial/2554/consoleFull

Oct 21 2020, 12:48 PM · Restricted Project
sylvestre.ledru committed rG0784e17f1b4a: Remove .svn from exclude list as we moved to git (authored by sylvestre.ledru).
Remove .svn from exclude list as we moved to git
Oct 21 2020, 7:09 AM
sylvestre.ledru closed D89859: Remove .svn from exclude list as we moved to git.
Oct 21 2020, 7:09 AM · Restricted Project, Restricted Project, Restricted Project
sylvestre.ledru added a comment to D89859: Remove .svn from exclude list as we moved to git.

ed if you want to approve it ;)

Oct 21 2020, 7:03 AM · Restricted Project, Restricted Project, Restricted Project
sylvestre.ledru added a reviewer for D89859: Remove .svn from exclude list as we moved to git: emaste.
Oct 21 2020, 7:03 AM · Restricted Project, Restricted Project, Restricted Project
sylvestre.ledru requested review of D89859: Remove .svn from exclude list as we moved to git.
Oct 21 2020, 2:05 AM · Restricted Project, Restricted Project, Restricted Project

Oct 17 2020

sylvestre.ledru added a reverting change for rGb2eb43931757: [clang-format] Fix AlignConsecutive on PP blocks: rGb9e789447f14: Revert "[clang-format] Fix AlignConsecutive on PP blocks".
Oct 17 2020, 10:53 AM
sylvestre.ledru committed rGb9e789447f14: Revert "[clang-format] Fix AlignConsecutive on PP blocks" (authored by sylvestre.ledru).
Revert "[clang-format] Fix AlignConsecutive on PP blocks"
Oct 17 2020, 10:53 AM
sylvestre.ledru added a reverting change for D79388: [clang-format] Fix AlignConsecutive on PP blocks: rGb9e789447f14: Revert "[clang-format] Fix AlignConsecutive on PP blocks".
Oct 17 2020, 10:53 AM · Restricted Project, Restricted Project
sylvestre.ledru closed D89464: Revert "[clang-format] Fix AlignConsecutive on PP blocks".
Oct 17 2020, 10:53 AM · Restricted Project

Oct 16 2020

sylvestre.ledru updated the diff for D89464: Revert "[clang-format] Fix AlignConsecutive on PP blocks".

Add the test which was failing + add the previous tests (but disabled)

Oct 16 2020, 8:41 AM · Restricted Project

Oct 15 2020

sylvestre.ledru added inline comments to D89464: Revert "[clang-format] Fix AlignConsecutive on PP blocks".
Oct 15 2020, 8:03 AM · Restricted Project
sylvestre.ledru added reviewers for D89464: Revert "[clang-format] Fix AlignConsecutive on PP blocks": JakeMerdichAMD, MyDeveloperDay.
Oct 15 2020, 8:03 AM · Restricted Project
sylvestre.ledru added a reverting change for rGb2eb43931757: [clang-format] Fix AlignConsecutive on PP blocks: D89464: Revert "[clang-format] Fix AlignConsecutive on PP blocks".
Oct 15 2020, 7:08 AM
sylvestre.ledru requested review of D89464: Revert "[clang-format] Fix AlignConsecutive on PP blocks".
Oct 15 2020, 7:08 AM · Restricted Project
sylvestre.ledru added a reverting change for D79388: [clang-format] Fix AlignConsecutive on PP blocks: D89464: Revert "[clang-format] Fix AlignConsecutive on PP blocks".
Oct 15 2020, 7:08 AM · Restricted Project, Restricted Project

Oct 13 2020

sylvestre.ledru committed rG002968a32046: [clang-tidy] Add an example for misc-unused-alias-decls (authored by sylvestre.ledru).
[clang-tidy] Add an example for misc-unused-alias-decls
Oct 13 2020, 4:57 AM
sylvestre.ledru closed D89270: [clang-tidy] Add an example for misc-unused-alias-decls.
Oct 13 2020, 4:56 AM · Restricted Project

Oct 12 2020

sylvestre.ledru requested review of D89270: [clang-tidy] Add an example for misc-unused-alias-decls.
Oct 12 2020, 2:36 PM · Restricted Project
sylvestre.ledru added a comment to D79388: [clang-format] Fix AlignConsecutive on PP blocks.

@JakeMerdichAMD as the regression caused by this change is way more significant than the initial bug, I am planning to revert this patch in the next few days (both in master and 11).
Please let me know if you have any objection.

Oct 12 2020, 10:04 AM · Restricted Project, Restricted Project

Oct 8 2020

sylvestre.ledru added a comment to D79388: [clang-format] Fix AlignConsecutive on PP blocks.

@JakeMerdichAMD @MyDeveloperDay Do you know if the regression has been fixed?
thanks

Oct 8 2020, 3:52 AM · Restricted Project, Restricted Project

Oct 7 2020

sylvestre.ledru accepted D88990: Update documentation on stage3 cmake build.
Oct 7 2020, 11:04 AM · Restricted Project, Restricted Project

Sep 21 2020

sylvestre.ledru added a comment to D75791: [clang-format] Added new option IndentExternBlock.

@MarcusJohnson91 I know it is confusing but we don't use the Mozilla coding style. We are using the Google style.

Sep 21 2020, 11:52 PM · Restricted Project, Restricted Project, Restricted Project

Sep 20 2020

sylvestre.ledru added a comment to D75791: [clang-format] Added new option IndentExternBlock.

Any chance this changes could have caused this regression https://bugs.llvm.org/show_bug.cgi?id=47589 ?

Sep 20 2020, 3:24 AM · Restricted Project, Restricted Project, Restricted Project
sylvestre.ledru added a comment to D81272: [clang-tidy] New check `bugprone-redundant-branch-condition`.
Sep 20 2020, 3:11 AM · Restricted Project, Restricted Project

Sep 17 2020

sylvestre.ledru added a comment to D87758: Provide a hook to customize missing library error handling.

I am sure you are planning to do it but please don't forget the release notes and the documentation.

Sep 17 2020, 5:34 AM · Restricted Project

Sep 16 2020

sylvestre.ledru added a comment to D81272: [clang-tidy] New check `bugprone-redundant-branch-condition`.

Thanks for this checker. FYI, it found (at least) 3 defects in Firefox code (fixed with the autofix):
https://hg.mozilla.org/mozilla-central/rev/651e68f628d0

Sep 16 2020, 3:26 AM · Restricted Project, Restricted Project

Sep 9 2020

sylvestre.ledru added a comment to D87187: [Driver] Perform Linux distribution detection just once.

I guess it isn't possible to write tests for this?

Sep 9 2020, 11:30 PM · Restricted Project

Jul 8 2020

sylvestre.ledru committed rGbbea4d5e6b82: clang: Don't show a trailing space with --version when not built from the repo (authored by sylvestre.ledru).
clang: Don't show a trailing space with --version when not built from the repo
Jul 8 2020, 5:02 AM
sylvestre.ledru closed D83386: clang: Don't show a trailing space with --version when not built from the repo.
Jul 8 2020, 5:02 AM · Restricted Project
Herald added a project to D83386: clang: Don't show a trailing space with --version when not built from the repo: Restricted Project.
Jul 8 2020, 4:46 AM · Restricted Project

Jun 22 2020

sylvestre.ledru accepted D81932: [clang-tidy] Improved accuracy of check list updater script.

Excellent, thanks :)

Jun 22 2020, 1:01 AM · Restricted Project

Jun 19 2020

sylvestre.ledru added a comment to D75284: [Sphinx] Adjust for source_parsers deprecation in Sphinx 3.0 .

it breaks the build on Debian stable (and probably some Ubuntu LTS). With sphinx 1.8.4
Could you please revert the change ?

Jun 19 2020, 6:59 AM · Restricted Project

Jun 18 2020

sylvestre.ledru added a comment to D76012: [OpenMP][Tool] Header-only multiplexing of OMPT tools.

@protze.joachim @jdoerfert ompt-multiplex.h is installed by default in /usr/include/
others opemp headers are installed in /usr/include/openmp/

Jun 18 2020, 4:18 AM · Restricted Project
sylvestre.ledru added a comment to D76012: [OpenMP][Tool] Header-only multiplexing of OMPT tools.

maybe this improvement should be added to the release notes too ?

Jun 18 2020, 4:18 AM · Restricted Project
sylvestre.ledru resigned from D66390: [PATCH] [llvm] [Support] Update license on ConvertUTF.* per upstream license change.

I would like to see that happening but I am not a lawyer and important enough person in the project to make such call.

Jun 18 2020, 4:18 AM · Restricted Project
sylvestre.ledru accepted D78867: [docs] avoid 'arc land' command.

works for me
I like "arc land" but I am not opposed to changing the doc

Jun 18 2020, 3:46 AM · Restricted Project
sylvestre.ledru resigned from D34083: fix crash on thumb backend, only visit first and last operands of GEP on ConstantHoisting.
Jun 18 2020, 3:46 AM · Restricted Project
sylvestre.ledru accepted D75284: [Sphinx] Adjust for source_parsers deprecation in Sphinx 3.0 .

thanks!

Jun 18 2020, 3:46 AM · Restricted Project

Jun 4 2020

sylvestre.ledru added a comment to D79895: Add a new warning to warn when passing uninitialized variables as const reference parameters to a function.

@zequanwu could you please update the release notes? thanks
https://github.com/llvm/llvm-project/blob/master/clang/docs/ReleaseNotes.rst

Jun 4 2020, 8:11 AM · Restricted Project

May 29 2020

sylvestre.ledru committed rGa3418631e8aa: libclc: update website url (authored by davidak <davidak@users.noreply.github.com>).
libclc: update website url
May 29 2020, 12:30 AM
sylvestre.ledru committed rGca467542eecf: [CMake] Pass CLANG_VENDOR variables into later stages (authored by sylvestre.ledru).
[CMake] Pass CLANG_VENDOR variables into later stages
May 29 2020, 12:30 AM

May 28 2020

sylvestre.ledru added inline comments to D80627: [clang-format] Create a python documentation tool to generate a summary of the clang-format status for the whole of the LLVM project.
May 28 2020, 6:30 AM · Restricted Project, Restricted Project
sylvestre.ledru added a comment to D80627: [clang-format] Create a python documentation tool to generate a summary of the clang-format status for the whole of the LLVM project.

Well done!

May 28 2020, 5:57 AM · Restricted Project, Restricted Project

May 18 2020

sylvestre.ledru committed rGdbb034947cc1: Add support of the next Ubuntu (Ubuntu 20.10 - Groovy Gorilla) (authored by sylvestre.ledru).
Add support of the next Ubuntu (Ubuntu 20.10 - Groovy Gorilla)
May 18 2020, 11:57 PM

May 13 2020

sylvestre.ledru updated subscribers of rG13a633b438b6: [gcov] Delete CC1 option -coverage-no-function-names-in-data.

Can you point to me how Firefox uses -Xclang -coverage-no-function-names-in-data? Debian code search does not reveal anything.

May 13 2020, 10:17 AM
sylvestre.ledru updated subscribers of rG13a633b438b6: [gcov] Delete CC1 option -coverage-no-function-names-in-data.
May 13 2020, 4:48 AM
sylvestre.ledru added a comment to rG13a633b438b6: [gcov] Delete CC1 option -coverage-no-function-names-in-data.

@MaskRay Please don't land such patch without review.

May 13 2020, 4:48 AM

May 10 2020

sylvestre.ledru added a comment to D78052: add_new_check.py: Update of the template to add an autofix section.

has been reverted

May 10 2020, 4:47 AM · Restricted Project
sylvestre.ledru reopened D78052: add_new_check.py: Update of the template to add an autofix section.
May 10 2020, 4:47 AM · Restricted Project
sylvestre.ledru reopened D77983: clang-tidy doc: add a note for every checker with an autofix.

has been reverted

May 10 2020, 4:47 AM · Restricted Project
sylvestre.ledru committed rGa832fc4b0c1e: Revert two patches, not ready to be shared (authored by sylvestre.ledru).
Revert two patches, not ready to be shared
May 10 2020, 2:40 AM
sylvestre.ledru added a reverting change for rGdc0f79ea5b99: clang-tidy doc: add a note for checkers with an autofix: rGa832fc4b0c1e: Revert two patches, not ready to be shared.
May 10 2020, 2:40 AM
sylvestre.ledru added a reverting change for rGf97f92e5b0fd: add_new_check.py: Update of the template to add an autofix section: rGa832fc4b0c1e: Revert two patches, not ready to be shared.
May 10 2020, 2:40 AM
sylvestre.ledru committed rGb0828135dbe3: clang-tidy doc: Fix the syntax to use rst list type (authored by sylvestre.ledru).
clang-tidy doc: Fix the syntax to use rst list type
May 10 2020, 2:39 AM
sylvestre.ledru committed rGf97f92e5b0fd: add_new_check.py: Update of the template to add an autofix section (authored by sylvestre.ledru).
add_new_check.py: Update of the template to add an autofix section
May 10 2020, 2:39 AM
sylvestre.ledru committed rGdc0f79ea5b99: clang-tidy doc: add a note for checkers with an autofix (authored by sylvestre.ledru).
clang-tidy doc: add a note for checkers with an autofix
May 10 2020, 2:39 AM
sylvestre.ledru closed D77983: clang-tidy doc: add a note for every checker with an autofix.
May 10 2020, 2:39 AM · Restricted Project
sylvestre.ledru closed D78052: add_new_check.py: Update of the template to add an autofix section.
May 10 2020, 2:39 AM · Restricted Project
sylvestre.ledru added a comment to D78052: add_new_check.py: Update of the template to add an autofix section.

@alexfh ping ? :)

May 10 2020, 2:39 AM · Restricted Project
sylvestre.ledru added a comment to D77983: clang-tidy doc: add a note for every checker with an autofix.

@alexfh ping

May 10 2020, 2:39 AM · Restricted Project

May 1 2020

sylvestre.ledru committed rG68e89c5b9603: [Gnu toolchain] Move GCC multilib/multiarch paths support from Linux to Gnu Add… (authored by sthibaul).
[Gnu toolchain] Move GCC multilib/multiarch paths support from Linux to Gnu Add…
May 1 2020, 3:37 AM
sylvestre.ledru committed rGc298e5a02292: [Gnu toolchain] Move GCC multilib/multiarch paths support from Linux to Gnu (authored by sthibaul).
[Gnu toolchain] Move GCC multilib/multiarch paths support from Linux to Gnu
May 1 2020, 3:13 AM

Apr 28 2020

sylvestre.ledru added a comment to D78867: [docs] avoid 'arc land' command.

are you sure? D<revision> worked for me in the past

Apr 28 2020, 1:34 AM · Restricted Project