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 (306 w, 6 d)

Recent Activity

Yesterday

sylvestre.ledru added a comment to D56803: clang -dumpversion returns 4.2.1 for legacy reason, update it.

@ddunbar is that ok with you? Thanks
See
https://bugs.llvm.org/show_bug.cgi?id=38836
https://bugs.launchpad.net/ubuntu/+source/llvm-defaults/+bug/1810860

Wed, Jan 23, 2:03 PM

Mon, Jan 21

sylvestre.ledru added a comment to D56946: [Documentation] Use HTTPS whenever possible in Clang.

Did you test all url?

Mon, Jan 21, 5:49 AM · Restricted Project
sylvestre.ledru added a comment to D54472: Disable invalid isPodLike<> specialization.

@serge-sans-paille
This (probably) broke LLVM on Debian jessie. It fails with:

Mon, Jan 21, 4:49 AM

Thu, Jan 17

sylvestre.ledru resigned from D24863: Keep dependencies separated between static and dynamic libraries. Fix for bug #28127. .
Thu, Jan 17, 5:42 AM · Restricted Project

Wed, Jan 16

sylvestre.ledru abandoned D23976: Disable the common parts for the sanitizer tools on kfreebsd.

fixed I think

Wed, Jan 16, 11:53 PM
sylvestre.ledru added a reviewer for D53014: Add CMAKE_BUILD_TYPE to the list of BOOTSTRAP_DEFAULT_PASSTHROUGH variables: beanz.
Wed, Jan 16, 11:51 PM
sylvestre.ledru created D56803: clang -dumpversion returns 4.2.1 for legacy reason, update it.
Wed, Jan 16, 1:05 PM

Thu, Jan 10

sylvestre.ledru added a comment to D55434: ObjectFileBreakpad: Implement sections.

On i386

Thu, Jan 10, 7:56 AM
sylvestre.ledru added a comment to D55434: ObjectFileBreakpad: Implement sections.

@labath This broke lldb on Debian stable:

Thu, Jan 10, 7:36 AM

Dec 13 2018

sylvestre.ledru updated the summary of D55382: Make -Wstring-plus-int warns even if when the result is not out of bounds.
Dec 13 2018, 8:06 AM

Dec 10 2018

sylvestre.ledru added a comment to D54677: [hurd] Fix unconditional use of PATH_MAX.

@EricWF is it ok to land? Thanks

Dec 10 2018, 2:19 AM

Dec 7 2018

sylvestre.ledru accepted D55391: Don't add unnecessary compiler flags to llvm-config output.

Much better. thanks!

Dec 7 2018, 7:22 AM
sylvestre.ledru abandoned D55174: Fix bug 8220 - llvm-config: Only keep flags starting by -I, -D & -std for --cflags, --cxxflags & --ldflags.
Dec 7 2018, 7:22 AM

Dec 6 2018

sylvestre.ledru added a comment to D55174: Fix bug 8220 - llvm-config: Only keep flags starting by -I, -D & -std for --cflags, --cxxflags & --ldflags.

@tstellar will you accept this patch if I remove -std?
thanks

Dec 6 2018, 1:45 PM
sylvestre.ledru added a comment to D55174: Fix bug 8220 - llvm-config: Only keep flags starting by -I, -D & -std for --cflags, --cxxflags & --ldflags.

And should I update the release notes?

Dec 6 2018, 1:45 PM

Dec 2 2018

sylvestre.ledru added a comment to D55174: Fix bug 8220 - llvm-config: Only keep flags starting by -I, -D & -std for --cflags, --cxxflags & --ldflags.

Other potential options:

Dec 2 2018, 6:45 AM
sylvestre.ledru created D55174: Fix bug 8220 - llvm-config: Only keep flags starting by -I, -D & -std for --cflags, --cxxflags & --ldflags.
Dec 2 2018, 6:41 AM

Nov 26 2018

sylvestre.ledru updated the summary of D53807: Create a diagnostic group for warn_call_to_pure_virtual_member_function_from_ctor_dtor, so it can be turned into an error using Werror.
Nov 26 2018, 6:21 AM

Nov 7 2018

sylvestre.ledru added a comment to D52034: [Clang] Add options -fprofile-filter-files and -fprofile-exclude-files to filter the files to instrument with gcov.

Can be done in a new commit but I think this should be mentioned in the clang release notes.

Nov 7 2018, 11:54 PM

Nov 1 2018

sylvestre.ledru resigned from D52173: Python bindings TypeError in reparse method .
Nov 1 2018, 4:51 AM · Restricted Project
sylvestre.ledru added a comment to D34329: [clang-diff] Initial implementation..

@johannes Someone asked for that in Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=907269

Nov 1 2018, 4:04 AM

Oct 29 2018

sylvestre.ledru added a comment to D53807: Create a diagnostic group for warn_call_to_pure_virtual_member_function_from_ctor_dtor, so it can be turned into an error using Werror.

Please add some tests

Oct 29 2018, 3:15 AM

Oct 23 2018

sylvestre.ledru updated the diff for D53520: Update the example of BS_Stroustrup to match what is done by clang-format.

Remove the desc update

Oct 23 2018, 12:10 PM

Oct 22 2018

sylvestre.ledru created D53520: Update the example of BS_Stroustrup to match what is done by clang-format.
Oct 22 2018, 12:22 PM

Oct 20 2018

sylvestre.ledru updated the diff for D53409: Add support for GNU Hurd in Path.inc and other places.

Remove the DwarfLinker.cpp declaration (useless)

Oct 20 2018, 2:31 PM
sylvestre.ledru added a comment to D53409: Add support for GNU Hurd in Path.inc and other places.

Debian still proposes a GNU Hurd port of this distro. As LLVM is now key, the porters have been trying to get llvm ported.
This fixes a bunch of issues and make llvm work correctly on gnu hurd.

Oct 20 2018, 2:05 PM

Oct 18 2018

sylvestre.ledru created D53410: Add missing PATH_MAX for GNU Hurd support.
Oct 18 2018, 1:28 PM
sylvestre.ledru retitled D53409: Add support for GNU Hurd in Path.inc and other places from Add support for GNU Hurd in Path.inc to Add support for GNU Hurd in Path.inc and other places.
Oct 18 2018, 1:24 PM
sylvestre.ledru updated the diff for D53409: Add support for GNU Hurd in Path.inc and other places.

add path_max def

Oct 18 2018, 1:23 PM
sylvestre.ledru created D53409: Add support for GNU Hurd in Path.inc and other places.
Oct 18 2018, 1:18 PM

Oct 9 2018

sylvestre.ledru added a comment to D52173: Python bindings TypeError in reparse method .

I am sorry but I am not the right reviewer for that. Sorry!

Oct 9 2018, 11:50 PM · Restricted Project
sylvestre.ledru updated the diff for D53018: Document the behavior of option passing when using -DCLANG_ENABLE_BOOTSTRAP=On Also document -DCLANG_BOOTSTRAP_PASSTHROUGH.

Fix a typo

Oct 9 2018, 1:38 AM
sylvestre.ledru created D53018: Document the behavior of option passing when using -DCLANG_ENABLE_BOOTSTRAP=On Also document -DCLANG_BOOTSTRAP_PASSTHROUGH.
Oct 9 2018, 1:32 AM
sylvestre.ledru created D53014: Add CMAKE_BUILD_TYPE to the list of BOOTSTRAP_DEFAULT_PASSTHROUGH variables.
Oct 9 2018, 1:21 AM

Sep 29 2018

sylvestre.ledru added a comment to D49017: Replace "nullptr-terminated" C-arrays of OptionValueEnumeration with safer llvm::ArrayRef.

This broke the apt.llvm.org CI on Debian Jessie and Ubuntu trusty (at least).
I reported https://bugs.llvm.org/show_bug.cgi?id=39131

Sep 29 2018, 10:51 AM

Sep 17 2018

sylvestre.ledru abandoned D30111: [clang-format] Add a test to check at once all the Mozilla coding style.
Sep 17 2018, 12:06 AM

Sep 16 2018

sylvestre.ledru added a comment to D52151: Also manages clang-X as tool for scan-build.

@kristina sorry, I missed your comment. I just followed what we are doing with gcc. Do you want me to update it?

Sep 16 2018, 12:56 PM
sylvestre.ledru updated the diff for D52153: scan-build: Add support of the option --exclude like in scan-build-py.

Fix the indentation

Sep 16 2018, 12:51 PM
sylvestre.ledru created D52153: scan-build: Add support of the option --exclude like in scan-build-py.
Sep 16 2018, 12:50 PM
sylvestre.ledru updated the summary of D52153: scan-build: Add support of the option --exclude like in scan-build-py.
Sep 16 2018, 12:50 PM
sylvestre.ledru created D52151: Also manages clang-X as tool for scan-build.
Sep 16 2018, 11:47 AM

Sep 14 2018

sylvestre.ledru added a comment to D31338: Move ParsedAttrInfos into a registry and point to one in AttributeList.

@john.brawn Are you still working on that? We would need this change at Mozilla to improve the way we use attributes.
Thanks

Sep 14 2018, 11:49 PM

Sep 11 2018

sylvestre.ledru added a comment to D51108: [PowerPC] Fix wrong ABI for i1 stack arguments on PPC32.

You should ping the reviewer (if she/he is available on IRC).
Once you have the sign off, you ping the reviewer (or someone else like me) to land your patch in the vcs.

Sep 11 2018, 2:53 AM

Sep 8 2018

sylvestre.ledru added a comment to D43630: [Driver] Fix search paths on x32.

@jrtc27 are you going to take care of the tests? Thanks

Sep 8 2018, 7:33 AM

Sep 6 2018

sylvestre.ledru abandoned D51285: Fix a build issue on Debian Jessie.
Sep 6 2018, 1:26 AM

Aug 27 2018

sylvestre.ledru retitled D51285: Fix a build issue on Debian Jessie from Fix a build issue on Debian Jessie. Was failing with openmp/libomptarget/src/omptarget.cpp:37:21: error: 'va_start' was not declared in this scope to Fix a build issue on Debian Jessie.
Aug 27 2018, 12:39 AM
sylvestre.ledru created D51285: Fix a build issue on Debian Jessie.
Aug 27 2018, 12:32 AM

Aug 26 2018

sylvestre.ledru added a comment to D34329: [clang-diff] Initial implementation..

@arphaman @johannes Is that normal that clang-diff isn't installed by cmake? (like clang-format?)

Aug 26 2018, 7:09 AM

Aug 23 2018

sylvestre.ledru added a comment to D44910: [docs] Add Markdown support to Sphinx.

Unfortunately, recommonmark isn't available on some older ubuntu (no backport available) & debian (a backport is available).
This will make the life of some of us harder!

Aug 23 2018, 1:45 PM

Jun 27 2018

sylvestre.ledru added a comment to D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.

If you are interested, I executed this checker on Firefox code. Results can be found here
http://sylvestre.ledru.info/reports/fx-scan-build/

Jun 27 2018, 12:04 PM

Jun 8 2018

sylvestre.ledru added a comment to D12921: clang-format: Support 'template<>' (no space)..

This was merged here: https://reviews.llvm.org/D23317
we can close this review

Jun 8 2018, 7:46 AM

Apr 9 2018

sylvestre.ledru edited reviewers for D45454: Add llvm_gcov_flush to be called outside a shared library, added: marco-c; removed: sylvestre.ledru.
Apr 9 2018, 11:55 PM

Mar 29 2018

sylvestre.ledru updated the diff for D41808: Rename clang link from clang-X.Y to clang-X.

Add to the release notes

Mar 29 2018, 3:03 AM
sylvestre.ledru updated the diff for D41869: Rename llvm library from libLLVM-X.Y to libLLVM-X.

Add to the release notes

Mar 29 2018, 2:44 AM

Mar 21 2018

sylvestre.ledru added a reviewer for D41869: Rename llvm library from libLLVM-X.Y to libLLVM-X: hans.
Mar 21 2018, 3:42 AM
sylvestre.ledru added a reviewer for D41808: Rename clang link from clang-X.Y to clang-X: hans.
Mar 21 2018, 3:42 AM

Mar 20 2018

sylvestre.ledru updated the diff for D41808: Rename clang link from clang-X.Y to clang-X.
Mar 20 2018, 4:14 AM
sylvestre.ledru added a reviewer for D41869: Rename llvm library from libLLVM-X.Y to libLLVM-X: dim.
Mar 20 2018, 1:52 AM

Mar 17 2018

sylvestre.ledru abandoned D30863: [clang-format] make docs/tools/{dump_format_style.py,dump_ast_matchers.py} flake8 compliant.

Looks like it doesn't interest anyone to use coding style on it.

Mar 17 2018, 6:41 AM
sylvestre.ledru updated the diff for D41869: Rename llvm library from libLLVM-X.Y to libLLVM-X.

With llvm-config

Mar 17 2018, 6:34 AM

Mar 8 2018

sylvestre.ledru added a comment to D42624: [clang-tidy] Add a utility Matcher to match the next statement within a statement sequence.

@aaron.ballman Hello, Do you think it is ready to land? Thanks

Mar 8 2018, 12:54 AM · Restricted Project

Feb 26 2018

sylvestre.ledru added a reviewer for D43630: [Driver] Fix search paths on x32: rnk.
Feb 26 2018, 8:51 AM

Feb 12 2018

sylvestre.ledru accepted D42593: GCC compatibility: Ignore -fstack-clash-protection.

Looks good, thanks.

Feb 12 2018, 4:35 AM

Jan 18 2018

sylvestre.ledru updated the diff for D41869: Rename llvm library from libLLVM-X.Y to libLLVM-X.

More things to update (the library name too)

Jan 18 2018, 11:47 AM

Jan 12 2018

sylvestre.ledru updated the diff for D41869: Rename llvm library from libLLVM-X.Y to libLLVM-X.
Jan 12 2018, 10:16 AM

Jan 9 2018

sylvestre.ledru created D41869: Rename llvm library from libLLVM-X.Y to libLLVM-X.
Jan 9 2018, 9:05 AM

Jan 7 2018

sylvestre.ledru added a reviewer for D41808: Rename clang link from clang-X.Y to clang-X: dlj.
Jan 7 2018, 12:22 PM
sylvestre.ledru created D41808: Rename clang link from clang-X.Y to clang-X.
Jan 7 2018, 8:15 AM

Jan 6 2018

sylvestre.ledru added a comment to D41538: [analyzer] Fix some checker's output plist not containing the checker name #2.

It missed the 6.0 branching. Will you try to get it on this branch?
Thanks

Jan 6 2018, 3:51 AM

Dec 21 2017

sylvestre.ledru added a comment to D40610: Flush gcda files before unlocking them.

I added two more reviewers as we would like to get that in trunk before 6.0 branches.

Dec 21 2017, 1:34 AM
sylvestre.ledru added reviewers for D40610: Flush gcda files before unlocking them: rnk, void.
Dec 21 2017, 1:34 AM
sylvestre.ledru added a comment to D38124: Hide some symbols to avoid a crash on shutdown when using code coverage.

I added two more reviewers as we would like to get that in trunk before 6.0 branches.

Dec 21 2017, 1:33 AM · Restricted Project
sylvestre.ledru added reviewers for D38124: Hide some symbols to avoid a crash on shutdown when using code coverage: rnk, void.
Dec 21 2017, 1:32 AM · Restricted Project

Dec 5 2017

sylvestre.ledru added a comment to D40827: [clang-format] Releasenotes for rL319024 : Add option to group multiple #include blocks when sorting includes.

Done!
btw, you should probably request commit permissions ;)

Dec 5 2017, 1:26 AM
sylvestre.ledru updated the summary of D40827: [clang-format] Releasenotes for rL319024 : Add option to group multiple #include blocks when sorting includes.
Dec 5 2017, 1:21 AM
sylvestre.ledru accepted D40827: [clang-format] Releasenotes for rL319024 : Add option to group multiple #include blocks when sorting includes.
Dec 5 2017, 12:15 AM

Dec 4 2017

sylvestre.ledru added a comment to rL319024: [clang-format] Add option to group multiple #include blocks when sorting….

This could be added to the release notes, don't you think?

Dec 4 2017, 1:12 PM

Nov 28 2017

sylvestre.ledru updated the diff for D40453: Add the nvidia-cuda-toolkit Debian package path to search path.
Nov 28 2017, 1:44 PM
sylvestre.ledru updated the diff for D40453: Add the nvidia-cuda-toolkit Debian package path to search path.
Nov 28 2017, 1:30 PM

Nov 27 2017

sylvestre.ledru added a comment to D40453: Add the nvidia-cuda-toolkit Debian package path to search path.

Debian packages don't update the PATH and we are aiming at providing packages working out of the box.

Nov 27 2017, 9:48 AM

Nov 25 2017

sylvestre.ledru created D40453: Add the nvidia-cuda-toolkit Debian package path to search path.
Nov 25 2017, 1:25 AM

Nov 23 2017

sylvestre.ledru added a comment to D38124: Hide some symbols to avoid a crash on shutdown when using code coverage.

@davidxl David, is that ok with you? Can you sign off this change? Thanks

Nov 23 2017, 10:50 PM · Restricted Project

Nov 8 2017

sylvestre.ledru added a comment to D38124: Hide some symbols to avoid a crash on shutdown when using code coverage.

please thank Marco :)

Nov 8 2017, 6:12 AM · Restricted Project
sylvestre.ledru updated the diff for D38124: Hide some symbols to avoid a crash on shutdown when using code coverage.
Nov 8 2017, 5:55 AM · Restricted Project

Nov 7 2017

sylvestre.ledru closed D39717: Always use prctl(PR_SET_PTRACER).
Nov 7 2017, 11:25 PM
sylvestre.ledru updated the summary of D39717: Always use prctl(PR_SET_PTRACER).
Nov 7 2017, 11:25 PM
sylvestre.ledru added a reviewer for D39717: Always use prctl(PR_SET_PTRACER): eugenis.
Nov 7 2017, 1:11 AM

Oct 24 2017

sylvestre.ledru added a comment to D39246: Fix LLVM_LINK_LLVM_DYLIB build (pr35053).

This fixed the issue, thanks!

Oct 24 2017, 11:56 AM

Oct 20 2017

sylvestre.ledru edited reviewers for D28462: clang-format: Add new style option AlignConsecutiveMacros, added: klimek; removed: sylvestre.ledru.
Oct 20 2017, 12:43 AM · Restricted Project

Oct 15 2017

sylvestre.ledru added a comment to D38678: [Sema] Warn about unused variables if we can constant evaluate the initializer..

For the record, this found 5 actual useless variables in Firefox code.

Oct 15 2017, 11:11 AM

Oct 11 2017

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

I also see lots of code that uses {}? Thus, it seems like the code base is not very strict, and going either way enforced with clang-format will be nicer?
https://dxr.mozilla.org/mozilla-central/search?q=%22+%7B%7D&redirect=true

Indeed, this is why we are working on enforcing that.
This patch is one of the last change to match exactly our coding style. This is why Andi wrote it.

Oct 11 2017, 11:56 PM

Sep 28 2017

sylvestre.ledru closed D38186: Add the new -Wnull-pointer-arithmetic warnings to the release notes.
Sep 28 2017, 1:02 AM · Restricted Project
sylvestre.ledru abandoned D38346: Add the new -Wnull-pointer-arithmetic warnings to the release notes.
Sep 28 2017, 1:02 AM
sylvestre.ledru accepted D38186: Add the new -Wnull-pointer-arithmetic warnings to the release notes.

Thanks, it makes sense!
Landed in r314387

Sep 28 2017, 1:02 AM · Restricted Project
sylvestre.ledru created D38346: Add the new -Wnull-pointer-arithmetic warnings to the release notes.
Sep 28 2017, 12:59 AM

Sep 26 2017

sylvestre.ledru closed D38224: Don't move llvm.localescape outside the entry block in the GCOV profiling pass.
Sep 26 2017, 3:30 PM
sylvestre.ledru updated the summary of D38224: Don't move llvm.localescape outside the entry block in the GCOV profiling pass.
Sep 26 2017, 3:30 PM

Sep 25 2017

sylvestre.ledru closed D38228: Fix clangd when built with LLVM_LINK_LLVM_DYLIB=ON.
Sep 25 2017, 7:10 AM
sylvestre.ledru added a comment to D38228: Fix clangd when built with LLVM_LINK_LLVM_DYLIB=ON.

For the record, this is the same approach as in https://reviews.llvm.org/rL240104

Sep 25 2017, 5:03 AM
sylvestre.ledru created D38228: Fix clangd when built with LLVM_LINK_LLVM_DYLIB=ON.
Sep 25 2017, 5:03 AM