Page MenuHomePhabricator

feg208 (Fred Grim)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 25 2021, 7:55 AM (7 w, 5 d)

Recent Activity

Tue, Jun 15

feg208 abandoned D104242: Alters a lit test to simplify and avoid a buildbot error.
Tue, Jun 15, 5:04 PM · Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Another +1 for it failing intermittently on the bot I watch over, llvm-clang-x86_64-sie-ubuntu-fast. Anything that can be done to stabilize the test or to remove it would be appreciated!

Tue, Jun 15, 12:38 PM · Restricted Project, Restricted Project
feg208 updated subscribers of D104295: removes redundant test.
Tue, Jun 15, 12:34 PM · Restricted Project
feg208 added a comment to D104295: removes redundant test.

If I could get someone to land this commit on my behalf I'd be obliged. I am working on the getting commit rights since that seems to be a pretty significant impediment to developing in llvm land but as of now I can't land this myself.

Tue, Jun 15, 8:40 AM · Restricted Project
feg208 added a comment to D104242: Alters a lit test to simplify and avoid a buildbot error.

I put up a separate commit that just removes the test. I think this would resolve the builder issues but I don't think its worth even trying. We should abandon this (I think)

Tue, Jun 15, 6:44 AM · Restricted Project
feg208 updated subscribers of D104242: Alters a lit test to simplify and avoid a buildbot error.
Tue, Jun 15, 6:43 AM · Restricted Project
feg208 requested review of D104295: removes redundant test.
Tue, Jun 15, 6:39 AM · Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

I think, at this point, the sensible thing to do is to pull the test. As @MyDeveloperDay points out it isn't really adding any value above and beyond the unit tests. If it were non-determinism in the change itself the unit tests would catch it. They test the same formatting and test for non-determinism explicitly. Yet they've never failed. And there are quite a few of them.

Tue, Jun 15, 6:34 AM · Restricted Project, Restricted Project

Mon, Jun 14

feg208 added a comment to D104242: Alters a lit test to simplify and avoid a buildbot error.

I don’t have commit rights currently. Could I get someone to land this on my behalf?

Mon, Jun 14, 10:52 PM · Restricted Project
feg208 added inline comments to D101868: [clang-format] Adds a formatter for aligning arrays of structs.
Mon, Jun 14, 8:34 PM · Restricted Project, Restricted Project
feg208 added a reviewer for D104242: Alters a lit test to simplify and avoid a buildbot error: vitalybuka.
Mon, Jun 14, 8:33 PM · Restricted Project
feg208 retitled D104242: Alters a lit test to simplify and avoid a buildbot error from Removes an unused variable and alters a lit test to simplify and avoid a buildbot error to Alters a lit test to simplify and avoid a buildbot error.
Mon, Jun 14, 1:30 PM · Restricted Project
feg208 updated the diff for D104242: Alters a lit test to simplify and avoid a buildbot error.

Looks like the dead code is removed in main

Mon, Jun 14, 1:27 PM · Restricted Project
feg208 added a comment to D104242: Alters a lit test to simplify and avoid a buildbot error.

Could you split this into two diffs? Or are those two changes in any way related?

Mon, Jun 14, 1:15 PM · Restricted Project
feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Forgot the clang-tidy checks

Mon, Jun 14, 11:40 AM · Restricted Project
feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Clang-tidy checks cleaned up and test fixes

Mon, Jun 14, 10:46 AM · Restricted Project
feg208 requested review of D104242: Alters a lit test to simplify and avoid a buildbot error.
Mon, Jun 14, 10:18 AM · Restricted Project
feg208 added a comment to rG673c5ba58497: [clang-format] Adds a formatter for aligning arrays of structs.

Ok. I am going to see if I can replicate the issue locally. Otherwise the easiest path (imho) is to pull the lit test that is failing. It isn't telling us anything we don't already know from the unit tests in any case. But maybe its something odd like the spacing or the single '-' in the option flags on FileChecl

Mon, Jun 14, 6:37 AM

Sun, Jun 13

feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Added command line print

Sun, Jun 13, 3:03 PM · Restricted Project
feg208 added a comment to D101340: Allows for dsymutil crashes to generate reproduceable information.

This is not yet complete. I missed step 4a. "4a. If we crash, we generate the reproducer and print the dsymutil invocation to replay the reproducer." It currently doesn't print the command line

Sun, Jun 13, 2:57 PM · Restricted Project
feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Fixes the broken tests

Sun, Jun 13, 2:34 PM · Restricted Project

Sat, Jun 12

feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Rolled up review comments

Sat, Jun 12, 9:28 AM · Restricted Project
feg208 added a comment to D101340: Allows for dsymutil crashes to generate reproduceable information.

I think this covers the comments

Sat, Jun 12, 9:28 AM · Restricted Project

Thu, Jun 10

feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

If I can get someone to submit this on my behalf I think we can call it a day.

Please post the name and email for the commit. And if no one else commits, I will do it on the weekend.

Thu, Jun 10, 12:58 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

If I can get someone to submit this on my behalf I think we can call it a day.

Thu, Jun 10, 7:00 AM · Restricted Project, Restricted Project

Wed, Jun 9

feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

All the review comments are addressed

Wed, Jun 9, 1:09 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Rolls up the remaining review comments

Wed, Jun 9, 1:05 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

LGTM. That's a great piece work @feg208. Thank you!

Wed, Jun 9, 12:52 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

addressed the remaining review comment

Wed, Jun 9, 12:29 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Regenerated the mangled ClangFormatStyle.rst file

Wed, Jun 9, 12:28 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Ick. Missed a semi colon in the test

Wed, Jun 9, 9:25 AM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Fixes the busted test

Wed, Jun 9, 9:17 AM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Got some changes. The failing test is an open issue

Wed, Jun 9, 8:44 AM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Gets some of the formatting issues. Still chasing a test issue.

Wed, Jun 9, 8:44 AM · Restricted Project, Restricted Project

Tue, Jun 8

feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Fixes from clang-tidy checks

Tue, Jun 8, 8:32 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

This alters the array alignment to allow for left alignment

Tue, Jun 8, 5:21 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Better still how about having

ArraryMemberAligmentEnum AlignArrayOfStructures

of "None, Left and Right"

I find boolean options don't stay boolean for long!

Tue, Jun 8, 6:35 AM · Restricted Project, Restricted Project

Mon, Jun 7

feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Ok. Given @HazardyKnusperkeks comment I'll make the default be left alignment and update the tests accordingly

Mon, Jun 7, 12:58 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Gets some review comments

Mon, Jun 7, 12:54 PM · Restricted Project
feg208 added a comment to D101340: Allows for dsymutil crashes to generate reproduceable information.

I rolled up all the comments save one where I just have an outstanding question and then I'll roll that up as well

Mon, Jun 7, 12:54 PM · Restricted Project

Thu, Jun 3

feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Had some invalid code in the lit test

Thu, Jun 3, 4:35 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Rebased against main

Thu, Jun 3, 4:31 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Got both of these

Thu, Jun 3, 9:39 AM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Grabs up some review comments and adds lit test

Thu, Jun 3, 9:38 AM · Restricted Project, Restricted Project

Wed, Jun 2

feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Adds a simple lit test for some sanity checks

Wed, Jun 2, 5:41 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

one remaining

Wed, Jun 2, 2:07 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Missed a request

Wed, Jun 2, 2:06 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Captured review comments

Wed, Jun 2, 2:04 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

I rolled up the suggested changes.

Wed, Jun 2, 2:04 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

oops left some debugging messages in

Wed, Jun 2, 10:35 AM · Restricted Project, Restricted Project
feg208 added a comment to D101340: Allows for dsymutil crashes to generate reproduceable information.

I am wondering if I could get someone to look at this? I think I have addressed the review comments but I'd be happy to fix anything I missed

Wed, Jun 2, 9:33 AM · Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

I am of the view that this is complete. Or at a minimum all the tests that have
been requested pass and I would be surprised if there were gross errors or other
major things I have missed

Wed, Jun 2, 9:22 AM · Restricted Project, Restricted Project

Thu, May 27

feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Rolls up review comments

Thu, May 27, 11:51 AM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

I picked up most of these. One of the tests is already covered (I think) maybe I am misunderstanding

Thu, May 27, 11:51 AM · Restricted Project, Restricted Project

Wed, May 26

feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Still need to fix and added the tests I said I would but the comment tests are added

Wed, May 26, 10:52 AM · Restricted Project, Restricted Project

Tue, May 25

feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Addresses review comments

Tue, May 25, 9:37 AM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

The tests still need to be added

Tue, May 25, 9:37 AM · Restricted Project, Restricted Project

Mon, May 24

feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Broke a regression test

Mon, May 24, 7:56 PM · Restricted Project
feg208 added a comment to D101340: Allows for dsymutil crashes to generate reproduceable information.

I think I addressed all the comments? If so I can't push the changes into the repo. Could you? Assuming there aren't outstanding concerns of course.

Mon, May 24, 5:59 PM · Restricted Project
feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Addresses some review comments

Mon, May 24, 5:58 PM · Restricted Project
feg208 added reviewers for D101340: Allows for dsymutil crashes to generate reproduceable information: void, ruiu, bkramer.
Mon, May 24, 1:08 PM · Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

clang-tidy and clang-format changes

Mon, May 24, 9:13 AM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

This reworks substantially this commit. I recognize there are lacking/broken
tests but I just would like to ensure that the general direction doesn't
seem likely to end in tears

I'm not deep enough in clang-format and currently have not enough time to check that in depth. But why are you right aligning?

Mon, May 24, 9:03 AM · Restricted Project, Restricted Project

Sun, May 23

feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Fixes/adds a test that yielded a seg fault and quiets some clang-tidy checks

Sun, May 23, 9:06 AM · Restricted Project, Restricted Project

Sat, May 22

feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Adds some tests and fixes a broken test

Sat, May 22, 12:35 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Forgot to alter the documentation to reflect right justified column alignment

Sat, May 22, 11:09 AM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

This reworks substantially this commit. I recognize there are lacking/broken
tests but I just would like to ensure that the general direction doesn't
seem likely to end in tears

Sat, May 22, 11:06 AM · Restricted Project, Restricted Project

Fri, May 21

feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

This looks like a good start..

Fri, May 21, 10:25 AM · Restricted Project, Restricted Project

May 8 2021

feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Something which just came to my mind. Since you wrote your own LineFormatter, you have to add test cases for all kinds of indention and wrapping. It seems that the string wrapping does not happen, how about comments? Are classes, namespaces, functions, etc. correctly indented with various configurations?

Given my expectations I think the alignment has to happen differently, but I'm happy to be proven wrong.

May 8 2021, 10:00 AM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

I still need to handle the 20 character case appropriately

May 8 2021, 8:59 AM · Restricted Project, Restricted Project

May 7 2021

feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Missed a request in the release notes

May 7 2021, 1:15 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Oops missed a request

May 7 2021, 1:15 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

Added more tests and addressed review comments

May 7 2021, 1:12 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

I think I have all the tests and requested review comments rolled up

May 7 2021, 1:12 PM · Restricted Project, Restricted Project

May 6 2021

feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

I have a few outstanding questions about additional testing

May 6 2021, 4:45 PM · Restricted Project, Restricted Project
feg208 added a comment to D101868: [clang-format] Adds a formatter for aligning arrays of structs.

To answer the question of why I think this is different then other alignment options....It seems to me that each alignment option emphasizes a specific thing, be it macros, bitfields, or (maybe closer in spirit) more simple declarations and assignments. I think this case is unique and not currently addressed by any of the existing alignment mechanisms.

May 6 2021, 4:36 PM · Restricted Project, Restricted Project
feg208 updated the diff for D101868: [clang-format] Adds a formatter for aligning arrays of structs.

This addresses most of the review comments. There remain a few tests to add

May 6 2021, 4:36 PM · Restricted Project, Restricted Project

May 4 2021

feg208 requested review of D101868: [clang-format] Adds a formatter for aligning arrays of structs.
May 4 2021, 4:08 PM · Restricted Project, Restricted Project

May 1 2021

feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

clang-format misses

May 1 2021, 10:38 AM · Restricted Project

Apr 29 2021

feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Cleans up clang-tidy/format issues and some minor branch pecadillos

Apr 29 2021, 2:10 PM · Restricted Project

Apr 28 2021

feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Forgot to remove raise(SIGSEGV)

Apr 28 2021, 8:57 PM · Restricted Project
feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

This alters the prior commit by incorporating the Reproducer for the purpose of persisting the files on crash

Apr 28 2021, 8:54 PM · Restricted Project

Apr 27 2021

feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Missing more clang-tidy checks

Apr 27 2021, 8:58 AM · Restricted Project

Apr 26 2021

feg208 updated the diff for D101340: Allows for dsymutil crashes to generate reproduceable information.

Fixed clang-tidy warnings

Apr 26 2021, 9:21 PM · Restricted Project
feg208 requested review of D101340: Allows for dsymutil crashes to generate reproduceable information.
Apr 26 2021, 8:14 PM · Restricted Project