Page MenuHomePhabricator

riccibruno (Bruno Ricci)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 10 2018, 8:27 AM (113 w, 5 d)

Recent Activity

Tue, Aug 11

riccibruno updated the diff for D85613: [clang] Look through bindings when checking whether a default argument references a local entity..

Remove the now-unused const VarDecl * parameter to DiagnoseIfOdrUse.

Tue, Aug 11, 9:57 AM · Restricted Project
riccibruno committed rGf4dccf115cc1: [clang] Add a matcher for template template parameters. (authored by riccibruno).
[clang] Add a matcher for template template parameters.
Tue, Aug 11, 8:02 AM
riccibruno closed D85536: [clang] Add a matcher for template template parameters..
Tue, Aug 11, 8:01 AM · Restricted Project
riccibruno added inline comments to D85613: [clang] Look through bindings when checking whether a default argument references a local entity..
Tue, Aug 11, 7:41 AM · Restricted Project
riccibruno added inline comments to D85613: [clang] Look through bindings when checking whether a default argument references a local entity..
Tue, Aug 11, 7:39 AM · Restricted Project
riccibruno updated the diff for D85613: [clang] Look through bindings when checking whether a default argument references a local entity..
Tue, Aug 11, 7:38 AM · Restricted Project

Mon, Aug 10

riccibruno added inline comments to D85613: [clang] Look through bindings when checking whether a default argument references a local entity..
Mon, Aug 10, 7:52 AM · Restricted Project
riccibruno updated the diff for D85613: [clang] Look through bindings when checking whether a default argument references a local entity..

Refer to the binding in the diagnostic.

Mon, Aug 10, 7:46 AM · Restricted Project

Sun, Aug 9

riccibruno updated the summary of D85613: [clang] Look through bindings when checking whether a default argument references a local entity..
Sun, Aug 9, 2:11 PM · Restricted Project
riccibruno added a comment to D85144: [clang] Improve Dumping of APValues.

I agree with you that it's fine to use printPretty for leaves (and additionally it would be annoying to duplicate the LValue case); that's what I was planning to do anyway.

Sun, Aug 9, 11:36 AM · Restricted Project
riccibruno requested review of D85613: [clang] Look through bindings when checking whether a default argument references a local entity..
Sun, Aug 9, 9:25 AM · Restricted Project
riccibruno added inline comments to D85612: [Sema] Use proper integral cast for an enumerate with a fixed bool type.
Sun, Aug 9, 9:25 AM · Restricted Project

Sat, Aug 8

riccibruno added inline comments to D80514: [clang-tidy] modernize-use-trailing-return-type support for C++20 concepts and decltype.
Sat, Aug 8, 7:28 AM · Restricted Project, Restricted Project
riccibruno added inline comments to D80514: [clang-tidy] modernize-use-trailing-return-type support for C++20 concepts and decltype.
Sat, Aug 8, 6:19 AM · Restricted Project, Restricted Project

Fri, Aug 7

riccibruno added a comment to D85545: [Diagnostics] Diagnose missing comma in string array initialization.

Is there a way to suppress this diagnostic if someone wants to legitimately initialize an element of the array with a long string by relying on string literal concatenation?

Fri, Aug 7, 1:12 PM · Restricted Project
riccibruno added a comment to D84599: [Index/USRGeneration] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName.

... or if a string such as (unnamed struct at /path/to/input.cc:1:3) is suitable in an USR then printName can be used.

Fri, Aug 7, 12:45 PM · Restricted Project
riccibruno retitled D84599: [Index/USRGeneration] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName from [clang-index] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName to [Index/USRGeneration] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName.
Fri, Aug 7, 12:37 PM · Restricted Project
riccibruno requested review of D85536: [clang] Add a matcher for template template parameters..
Fri, Aug 7, 9:52 AM · Restricted Project
riccibruno added inline comments to D85144: [clang] Improve Dumping of APValues.
Fri, Aug 7, 7:34 AM · Restricted Project

Wed, Aug 5

riccibruno committed rG4dcbb9cef71a: [clang] Add -fno-delayed-template-parsing to the added unit tests in… (authored by riccibruno).
[clang] Add -fno-delayed-template-parsing to the added unit tests in…
Wed, Aug 5, 6:14 AM
riccibruno committed rGf7a039de7af7: [clang][NFC] DeclPrinter: use NamedDecl::getDeclName instead of NamedDecl… (authored by riccibruno).
[clang][NFC] DeclPrinter: use NamedDecl::getDeclName instead of NamedDecl…
Wed, Aug 5, 5:56 AM
riccibruno committed rG94b43118e220: [clang][NFCI] Get rid of ConstantMatrixTypeBitfields to avoid increasing the… (authored by riccibruno).
[clang][NFCI] Get rid of ConstantMatrixTypeBitfields to avoid increasing the…
Wed, Aug 5, 5:56 AM
riccibruno committed rG19701458d469: [clang][nearly-NFC] Remove some superfluous uses of NamedDecl::getNameAsString (authored by riccibruno).
[clang][nearly-NFC] Remove some superfluous uses of NamedDecl::getNameAsString
Wed, Aug 5, 5:56 AM
riccibruno committed rG6f2fa9d312fc: [clang][NFC] Document NamedDecl::printName (authored by riccibruno).
[clang][NFC] Document NamedDecl::printName
Wed, Aug 5, 5:56 AM
riccibruno committed rGbc29634b93ac: [clang][NFC] Remove an old workaround for MSVC 2013 (authored by riccibruno).
[clang][NFC] Remove an old workaround for MSVC 2013
Wed, Aug 5, 5:56 AM
riccibruno committed rG98b4b4570542: [clang][NFC] Add a test showcasing an unnamed template parameter in a diagnostic (authored by riccibruno).
[clang][NFC] Add a test showcasing an unnamed template parameter in a diagnostic
Wed, Aug 5, 5:55 AM
riccibruno committed rG00b89f66f988: [clang][NFC] Remove spurious +x flag on DeclTemplate.cpp and DeclTemplate.h (authored by riccibruno).
[clang][NFC] Remove spurious +x flag on DeclTemplate.cpp and DeclTemplate.h
Wed, Aug 5, 5:55 AM

Mon, Aug 3

riccibruno added inline comments to D85144: [clang] Improve Dumping of APValues.
Mon, Aug 3, 11:59 AM · Restricted Project
riccibruno added a comment to D85144: [clang] Improve Dumping of APValues.

Thanks for finishing this.

Mon, Aug 3, 11:53 AM · Restricted Project
riccibruno added inline comments to D83261: [OPENMP]Redesign of OMPExecutableDirective representation..
Mon, Aug 3, 9:52 AM · Restricted Project
riccibruno added a comment to D84599: [Index/USRGeneration] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName.

(Disclaimer: I am not at all familiar with this code)

Mon, Aug 3, 6:35 AM · Restricted Project

Sat, Aug 1

riccibruno updated the diff for D85033: [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures..

Don't forget to increment the field iterator in the loop of maybePrintFieldForLambdaCapture, and modify the tests to test this.

Sat, Aug 1, 5:33 AM · Restricted Project

Fri, Jul 31

riccibruno updated the summary of D85033: [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures..
Fri, Jul 31, 2:52 PM · Restricted Project
riccibruno updated the diff for D85033: [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures..

Add -fno-delayed-template-parsing to the new unit tests to also pass on Windows.

Fri, Jul 31, 2:33 PM · Restricted Project
riccibruno committed rG38d3e7533279: [clang] Use the location of the void parameters when complaining that only a… (authored by Jac1494).
[clang] Use the location of the void parameters when complaining that only a…
Fri, Jul 31, 12:37 PM
riccibruno closed D84678: [clang] False line number in a function definition with "void" parameter.
Fri, Jul 31, 12:37 PM · Restricted Project
riccibruno updated the diff for D85033: [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures..

Make the unit tests in NamedDeclPrinterTest.cpp more robust.

Fri, Jul 31, 12:13 PM · Restricted Project
riccibruno added a comment to D84678: [clang] False line number in a function definition with "void" parameter.

Hi @aaron.ballman ,
Address your review comments.
Thank you for accepting this. I don't have commit access please commit this.
Thanks.

As discussed with Aaron on IRC I can commit it for you. To do that I need a name and an email for the attribution.
But first move the test to the already existing test/Sema/void_arg.c which already test this diagnostic.

Fri, Jul 31, 10:39 AM · Restricted Project
riccibruno added a comment to D84678: [clang] False line number in a function definition with "void" parameter.

Hi @aaron.ballman ,
Address your review comments.
Thank you for accepting this. I don't have commit access please commit this.
Thanks.

Fri, Jul 31, 10:27 AM · Restricted Project
riccibruno added a reviewer for D85033: [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures.: aaron.ballman.
Fri, Jul 31, 9:44 AM · Restricted Project
riccibruno updated the summary of D85033: [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures..
Fri, Jul 31, 9:25 AM · Restricted Project
riccibruno retitled D85033: [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures. from [clang] Provide a better name for unnamed parameters, lambda classes and lambda captures. to [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures..
Fri, Jul 31, 9:24 AM · Restricted Project
riccibruno requested review of D85033: [clang] Provide a better pretty-printed name for unnamed parameters, lambda classes and lambda captures..
Fri, Jul 31, 9:02 AM · Restricted Project

Thu, Jul 30

riccibruno updated the diff for D81003: [clang] SequenceChecker: Also visit default arguments and default initializers..

Rebased with a few clang-format fixes.

Thu, Jul 30, 11:33 AM · Restricted Project

Wed, Jul 29

riccibruno added inline comments to D84005: Introduce ns_error_domain attribute..
Wed, Jul 29, 6:57 AM · Restricted Project
riccibruno committed rG517fe058d42a: [clang][NFC] clang-format fix after eb10b065f2a870b425dcc2040b9955e0eee464b4 (authored by riccibruno).
[clang][NFC] clang-format fix after eb10b065f2a870b425dcc2040b9955e0eee464b4
Wed, Jul 29, 6:55 AM
riccibruno committed rG1ae63b4179c2: [clang][NFC] Pass the ASTContext to CXXRecordDecl::setCaptures (authored by riccibruno).
[clang][NFC] Pass the ASTContext to CXXRecordDecl::setCaptures
Wed, Jul 29, 6:55 AM
riccibruno added a reviewer for D84599: [Index/USRGeneration] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName: kadircet.
Wed, Jul 29, 6:04 AM · Restricted Project
riccibruno added inline comments to D84005: Introduce ns_error_domain attribute..
Wed, Jul 29, 5:48 AM · Restricted Project

Tue, Jul 28

riccibruno updated the diff for D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.

Add the forgotten context.

Tue, Jul 28, 3:03 PM · Restricted Project
riccibruno updated the diff for D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.

Use a less generic name instead of get{Open,Close}Delimiter. I went with get{Open,Close}DelimiterForUnnamedEntity but I am happy to change it.

Tue, Jul 28, 3:02 PM · Restricted Project
riccibruno added a comment to D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.

Note that I am planning two improvements as a follow-up to this patch:

Tue, Jul 28, 8:20 AM · Restricted Project
riccibruno committed rG984cf99055a2: [clang][NFC] Add some documentation about the use of NamedDecl::getDeclName in… (authored by riccibruno).
[clang][NFC] Add some documentation about the use of NamedDecl::getDeclName in…
Tue, Jul 28, 7:43 AM
riccibruno committed rG01302ec08609: [Support][NFC] Fix a Wdocumentation warning in FileSystem.h (authored by riccibruno).
[Support][NFC] Fix a Wdocumentation warning in FileSystem.h
Tue, Jul 28, 7:36 AM
riccibruno committed rGb81fd5aeecd8: [clang-format][NFC] Fix a Wdocumentation warning in TokenAnnotator.cpp (authored by riccibruno).
[clang-format][NFC] Fix a Wdocumentation warning in TokenAnnotator.cpp
Tue, Jul 28, 2:59 AM
riccibruno added a comment to D84599: [Index/USRGeneration] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName.

The overloads of NamedDecl::printName are in D84658.

Tue, Jul 28, 2:43 AM · Restricted Project
riccibruno committed rGeb10b065f2a8: [clang] Pass the NamedDecl* instead of the DeclarationName into many… (authored by riccibruno).
[clang] Pass the NamedDecl* instead of the DeclarationName into many…
Tue, Jul 28, 2:36 AM
riccibruno committed rGf5acd11d2c0e: [clang-format][NFC] Be more careful about the layout of FormatToken. (authored by riccibruno).
[clang-format][NFC] Be more careful about the layout of FormatToken.
Tue, Jul 28, 2:36 AM
riccibruno closed D84656: [clang] Pass the NamedDecl* instead of the DeclarationName into many diagnostics..
Tue, Jul 28, 2:36 AM · Restricted Project
riccibruno closed D84306: [clang-format][NFC] Be more careful about the layout of FormatToken..
Tue, Jul 28, 2:36 AM · Restricted Project, Restricted Project
riccibruno added inline comments to D84678: [clang] False line number in a function definition with "void" parameter.
Tue, Jul 28, 2:26 AM · Restricted Project
riccibruno added a comment to D84306: [clang-format][NFC] Be more careful about the layout of FormatToken..

Thank you for the patch, this LGTM, I think this kind of change should help reduce memory usage and I feel improves the readability.

Tue, Jul 28, 2:22 AM · Restricted Project, Restricted Project

Mon, Jul 27

riccibruno added inline comments to D84678: [clang] False line number in a function definition with "void" parameter.
Mon, Jul 27, 3:01 PM · Restricted Project
riccibruno added inline comments to D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.
Mon, Jul 27, 1:36 PM · Restricted Project
riccibruno updated the diff for D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.

Address Erich's comments.

Mon, Jul 27, 1:32 PM · Restricted Project
riccibruno added a comment to D84656: [clang] Pass the NamedDecl* instead of the DeclarationName into many diagnostics..

The changes LGTM but it seems like there may be some formatting issues with the patch (or the lint tool is acting up).

Mon, Jul 27, 12:44 PM · Restricted Project
riccibruno updated the diff for D84306: [clang-format][NFC] Be more careful about the layout of FormatToken..

Remove a few missed getPackingKind.

Mon, Jul 27, 12:26 PM · Restricted Project, Restricted Project
riccibruno added inline comments to D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.
Mon, Jul 27, 7:49 AM · Restricted Project
riccibruno added inline comments to D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.
Mon, Jul 27, 7:44 AM · Restricted Project
riccibruno added reviewers for D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities: hokein, akyrtzi.
Mon, Jul 27, 7:37 AM · Restricted Project
riccibruno retitled D81865: [clang] Use string tables for static diagnostic descriptions from use string tables for static diagnostic descriptions to [clang] Use string tables for static diagnostic descriptions.
Mon, Jul 27, 7:27 AM · Restricted Project
riccibruno added a comment to D81865: [clang] Use string tables for static diagnostic descriptions.

Add some reviewers based on git diff --name-only | xargs -n 1 git blame --porcelain | grep "^author " | sort | uniq -c | sort -nr | head -30.

Mon, Jul 27, 7:24 AM · Restricted Project
riccibruno added reviewers for D81865: [clang] Use string tables for static diagnostic descriptions: akyrtzi, bkramer, dblaikie, MaskRay.
Mon, Jul 27, 7:24 AM · Restricted Project
riccibruno updated the summary of D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.
Mon, Jul 27, 7:14 AM · Restricted Project
riccibruno added a comment to D84656: [clang] Pass the NamedDecl* instead of the DeclarationName into many diagnostics..

I looked through the code changes, and they all seem quite mechanical. I believe they are all correct.

After reading through the test changes, I believe that this change is strictly an improvement thanks to printing the template arguments. Therefore, I don't believe I know of any way that htis is controversial.

That said, because I'm approving so quickly after submission, please give at least a few hours before committing to give the other reviewers a chance to take a look.

Mon, Jul 27, 7:08 AM · Restricted Project
riccibruno created D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities.
Mon, Jul 27, 7:07 AM · Restricted Project
riccibruno created D84656: [clang] Pass the NamedDecl* instead of the DeclarationName into many diagnostics..
Mon, Jul 27, 6:56 AM · Restricted Project
riccibruno added inline comments to D84461: [Concepts] Fix ast dump for immediately declared constraint..
Mon, Jul 27, 2:55 AM · Restricted Project
riccibruno added a comment to D84136: [clang] Fix visitation of ConceptSpecializationExpr in constrained-parameter.

Will do. By the way, is there something more tailored than ninja check-clang to run these ast-dump tests? ninja check-clang takes quite a while to run...

Mon, Jul 27, 2:52 AM · Restricted Project

Sun, Jul 26

riccibruno updated the diff for D84599: [Index/USRGeneration] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName.

Update a comment I originally missed.

Sun, Jul 26, 9:56 AM · Restricted Project
riccibruno created D84599: [Index/USRGeneration] Use NamedDecl::getDeclName() instead of NamedDecl::printName in USRGenerator::EmitDeclName.
Sun, Jul 26, 9:49 AM · Restricted Project
riccibruno committed rGb0512eed1e9d: [clang][NFC] Add a test for __attribute__((flag_enum)) with an unnamed… (authored by riccibruno).
[clang][NFC] Add a test for __attribute__((flag_enum)) with an unnamed…
Sun, Jul 26, 9:27 AM
riccibruno committed rGa2f83d5a07da: [clang][NFC] Add tests for the use of NamedDecl::getDeclName in the… (authored by riccibruno).
[clang][NFC] Add tests for the use of NamedDecl::getDeclName in the…
Sun, Jul 26, 9:27 AM
riccibruno committed rGca9bfc20f48c: [clang][NFC] Remove spurious +x flag on SemaConcept.cpp (authored by riccibruno).
[clang][NFC] Remove spurious +x flag on SemaConcept.cpp
Sun, Jul 26, 9:27 AM
riccibruno updated the diff for D84306: [clang-format][NFC] Be more careful about the layout of FormatToken..

Use is and isNot.

Sun, Jul 26, 9:12 AM · Restricted Project, Restricted Project

Fri, Jul 24

riccibruno added inline comments to D84005: Introduce ns_error_domain attribute..
Fri, Jul 24, 5:33 AM · Restricted Project

Thu, Jul 23

riccibruno added inline comments to D84306: [clang-format][NFC] Be more careful about the layout of FormatToken..
Thu, Jul 23, 8:03 AM · Restricted Project, Restricted Project

Wed, Jul 22

riccibruno added inline comments to D84343: [AST] Keep FP options in trailing storage of CallExpr.
Wed, Jul 22, 10:08 AM · Restricted Project
Herald added a project to D84306: [clang-format][NFC] Be more careful about the layout of FormatToken.: Restricted Project.
Wed, Jul 22, 4:04 AM · Restricted Project, Restricted Project

Tue, Jul 21

riccibruno added inline comments to rG473fbc90d1fb: [clang][NFC] Store a pointer to the ASTContext in ASTDumper and TextNodeDumper.
Tue, Jul 21, 3:35 PM
riccibruno added a comment to D81003: [clang] SequenceChecker: Also visit default arguments and default initializers..

Friendly ping.

Tue, Jul 21, 12:44 PM · Restricted Project
riccibruno added a comment to D84048: DR2303: Prefer 'nearer' base classes during template deduction..

Ah, doh, I see a few now. Is that number the compiler version number?

Tue, Jul 21, 12:23 PM · Restricted Project
riccibruno added a comment to D84048: DR2303: Prefer 'nearer' base classes during template deduction..

Also make sure to update the the cxx_dr_status.html document.

It is auto-generated. You need to tag the test case: dr2303: 12.

Edit: And put it at the right place in the file (not at the end).

Do you have an example of this? I've not seen that anywhere before.

Tue, Jul 21, 12:22 PM · Restricted Project
riccibruno added a comment to D84048: DR2303: Prefer 'nearer' base classes during template deduction..

Also make sure to update the the cxx_dr_status.html document.

Tue, Jul 21, 12:19 PM · Restricted Project
riccibruno committed rG7b5bddfd034e: [clang] Partially revert "Disable a few formatting options for test/" (authored by riccibruno).
[clang] Partially revert "Disable a few formatting options for test/"
Tue, Jul 21, 6:54 AM
riccibruno added a comment to D84090: [clang-format] Add BitFieldColonSpacing option.

Nit:clang-format the patch

Here is a Tip as to what I do (In case it helps other):

  1. I generate the diff off the staged items so I've already git added all the files I'm working on.
  1. then I can do git clang-format, this will fix up any files in the diff that need formatting (you'll need to git add them again if they have)
  1. I then check the documentation builds with /usr/bin/sphinx-build -n ./docs ./html (if the review contains rst files)
  1. then I do git diff --cached -U999999 > patch_to_submitt.diff
  1. I check the patch to ensure I'm not changing the mode of the files with grep -A0 -B2 "new mode" patch_to_submitt.diff
  1. and this is the patch_tosubmitt.diff I upload to the review

These steps try to reduce the number of review fails I get based on clang-format issues (all of this is scripted so making a patch is repeatable, quick and easy and all based off whats in my staged area)

Apart from that this patch LGTM (Before and After are good names)

But please clang-format before pushing (including the tests)

Tue, Jul 21, 6:00 AM · Restricted Project, Restricted Project
riccibruno added inline comments to rG473fbc90d1fb: [clang][NFC] Store a pointer to the ASTContext in ASTDumper and TextNodeDumper.
Tue, Jul 21, 5:34 AM

Sun, Jul 19

riccibruno committed rG89ff9bf061b4: [clang] Fix the warning for a non-void consteval function without a return… (authored by riccibruno).
[clang] Fix the warning for a non-void consteval function without a return…
Sun, Jul 19, 9:24 AM
riccibruno committed rG13316a770535: [clang] Disable a few formatting options for test/ (authored by riccibruno).
[clang] Disable a few formatting options for test/
Sun, Jul 19, 9:24 AM
riccibruno closed D83901: [clang] Disable a few formatting options for test/.
Sun, Jul 19, 9:24 AM · Restricted Project