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 (431 w, 1 d)

Recent Activity

Apr 22 2021

sylvestre.ledru committed rGd71ee3993fe7: Add support of the next Ubuntu (Ubuntu 21.10 - Impish Idri) (authored by sylvestre.ledru).
Add support of the next Ubuntu (Ubuntu 21.10 - Impish Idri)
Apr 22 2021, 11:38 AM

Apr 19 2021

sylvestre.ledru committed rG485e561f8d53: Try to unbreak the compiler-rt build on s390x (authored by sylvestre.ledru).
Try to unbreak the compiler-rt build on s390x
Apr 19 2021, 4:19 AM

Apr 18 2021

sylvestre.ledru added a comment to D98926: [sanitizer] Simplify GetTls with dl_iterate_phdr on Linux and use it on musl/FreeBSD.

@MaskRay
Seems that it caused
https://bugs.llvm.org/show_bug.cgi?id=50017
cannot link on s390x

Apr 18 2021, 3:02 AM · Restricted Project

Apr 15 2021

sylvestre.ledru committed rG56cb214b38a7: add test case for ignoring -flto=auto and -flto=jobserver (authored by doko).
add test case for ignoring -flto=auto and -flto=jobserver
Apr 15 2021, 3:20 AM
sylvestre.ledru closed D100484: add test case for ignoring -flto=auto and -flto=jobserver.
Apr 15 2021, 3:20 AM · Restricted Project

Apr 6 2021

sylvestre.ledru added a comment to D99996: [Driver] Drop $DEFAULT_TRIPLE-$name as a fallback program name.

To understand better, we will be fixing a bug from 2015, correct?

Apr 6 2021, 11:45 PM · Restricted Project

Apr 5 2021

sylvestre.ledru committed rG162848654842: ignore -flto= options recognized by GCC (authored by sylvestre.ledru).
ignore -flto= options recognized by GCC
Apr 5 2021, 2:55 AM
sylvestre.ledru closed D99501: ignore -flto= options recognized by GCC.
Apr 5 2021, 2:55 AM · Restricted Project

Apr 4 2021

sylvestre.ledru abandoned D77983: clang-tidy doc: add a note for every checker with an autofix.
Apr 4 2021, 9:25 AM · Restricted Project, Restricted Project
sylvestre.ledru added a reviewer for D60380: Also document -arch as -arch is mac specific: serge-sans-paille.
Apr 4 2021, 9:25 AM · Restricted Project
sylvestre.ledru added a reviewer for D53014: Add CMAKE_BUILD_TYPE to the list of BOOTSTRAP_DEFAULT_PASSTHROUGH variables: serge-sans-paille.
Apr 4 2021, 9:24 AM
sylvestre.ledru accepted D95114: HowToReleaseLLVM: Add annual release schedule template.

I am not opposed to this. I am just afraid that we will miss all the deadlines :)

Apr 4 2021, 9:22 AM · Restricted Project
sylvestre.ledru accepted D99081: [clang-tidy] ensure run-clang-tidy reports children killed by signals.

OK, no worries.
I don't think it will regress the situation anyway

Apr 4 2021, 9:21 AM · Restricted Project
sylvestre.ledru accepted D99852: [Driver] Detect libstdc++ include paths for native gcc (-m32 and -m64) on Debian i386.

Many thanks. it fixed my issues!

Apr 4 2021, 9:21 AM · Restricted Project
sylvestre.ledru updated subscribers of D99852: [Driver] Detect libstdc++ include paths for native gcc (-m32 and -m64) on Debian i386.
Apr 4 2021, 9:21 AM · Restricted Project
sylvestre.ledru accepted D91598: Allow packager to provide a default configuration file for clang.

It would be great to:

Apr 4 2021, 9:18 AM

Apr 3 2021

sylvestre.ledru added a comment to rG13a633b438b6: [gcov] Delete CC1 option -coverage-no-function-names-in-data.

Thanks for taking the time to reply

Apr 3 2021, 2:37 PM
sylvestre.ledru added a comment to rG13a633b438b6: [gcov] Delete CC1 option -coverage-no-function-names-in-data.

Also, please avoid using terms like madness, awful patch, etc when talking about the work of others...

Apr 3 2021, 11:48 AM
sylvestre.ledru added a comment to rG13a633b438b6: [gcov] Delete CC1 option -coverage-no-function-names-in-data.

That said, I think most stuff has been done with gcov. So new development may be rare.

Apr 3 2021, 9:34 AM

Mar 22 2021

sylvestre.ledru added a comment to D99081: [clang-tidy] ensure run-clang-tidy reports children killed by signals.

Do we have tests for this?
If so, it would be nice to add some :)

Mar 22 2021, 8:35 AM · Restricted Project

Mar 12 2021

sylvestre.ledru added a comment to rG13a633b438b6: [gcov] Delete CC1 option -coverage-no-function-names-in-data.

To help with turnaround time, we, at Mozilla, are happy to quickly review your patches

Mar 12 2021, 1:29 PM

Feb 2 2021

sylvestre.ledru added a comment to D95769: [OpenMP][Libomptarget] Remove possible harmful copy constructor call for RTLsTy.

I confirm that it fixed the problem - still occurring on 12

Feb 2 2021, 8:26 AM · Restricted Project

Jan 31 2021

sylvestre.ledru added a comment to D93293: [OpenMP][Libomptarget] Introduce changes to support remote plugin.

@atmnpatel
Seems that this change caused:
https://bugs.llvm.org/show_bug.cgi?id=48973

Jan 31 2021, 5:35 AM · Restricted Project

Jan 25 2021

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

Jan 25 2021, 1:04 AM · Restricted Project

Jan 20 2021

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 :)

Jan 20 2021, 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 :)

Jan 20 2021, 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

Jan 20 2021, 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?

Jan 20 2021, 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