+cc: Law, Biener,
On Wed, 16 Dec 2020 at 20:27, Dimitar Dimitrov <dimi...@dinux.eu> wrote:

> On понеделник, 14 декември 2020 г. 20:53:36 EET Dimitar Dimitrov wrote:
> > On петък, 11 декември 2020 г. 19:00:35 EET abebeos wrote:
> > > After "digesting" a bit more your review, I need to thank you for
> opening
> > > my eyes re "cherrypick" suggestion and... the missing g++ summaries. I
> > > need
> > > to update my setup to provide the g++ test-deltas, too. Note that in my
> > > test-setup, more c++ tests pass than in yours, not exactly sure why.
> It's
> > > in the "unresovled testcases".
> >
> > Regarding the additional "unresolved testcases" in my results. I traced
> it
> > to a bug in my setup. I did not prepare $HOME/.dejagnurc properly for
> AVR.
> > I'll rerun the tests and post the results.
> >
> > I'm using stock avr-libc, while I see you are using the avr-libc3 fork.
> > Hopefully the main differences are in HW support, not core libc.
> >
> > Regards,
> > Dimitar
>
> Here are the tests results with my environment fixes. Outcome is the same
> -
> saaadhu has no regressions.
>

Very nice!

@all

This review:

https://gcc.gnu.org/pipermail/gcc-patches/2020-December/561757.html

shows that there is still much work to do.

But the patch author seems "away" (and not available anyways), the listed
avr maintainer seems "super-away", main "executing analyst" (me) refuses to
"touch even whitespace".

To my understanding, the non-availability of the avr maintainer should mean
that the "upper-up" maintainers decide.

Commiting means essentially:

=> A low-quality avr-cc0 backend will be replaced with a 0-regressions
low(possibly even lower)-quality avr-mode_cc backend.

If anyone insists to polish things even more in context of a $7K bounty...
start thinking about this:

* Where is the line where bounty-work becomes IT-worker-abuse?
* Do you want people start saying (rightfully) that "gcc bounties are
essentially slave-labour"?
* Enjoying fixed employment?

And look at this:

* 200h (estimation) for the patch
* 200+h for the integration-work (dev-systems, testing etc.) - general
reading not counted.
* 100h (estimation) polishing
* 400h (estimation) for quality enhancement

no way, dishwashing pays better (
https://www.payscale.com/research/US/Job=Dishwasher/Hourly_Rate).

I could, as a follow-up task, "enhance the quality of the avr-mode-cc
backend", nothing special with the code.

But me not sure if it's worth assimilating "gcc's md & related internals"
to the point neccessary to fulfill the task at hand.

I've done my work here.

Note that I'll stop monitoring gcc-patches, but I'm still monitoring
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92729#c41


=================================================
> baseline beb9afcaf1466996a301c778596c5df209e7913c
>                 === gcc Summary ===
>
> # of expected passes            105527
> # of unexpected failures        584
> # of unexpected successes       16
> # of expected failures          582
> # of untested testcases         2
> # of unresolved testcases       33
> # of unsupported tests          5031
>
>                 === g++ Summary ===
>
> # of expected passes            146432
> # of unexpected failures        8012
> # of unexpected successes       21
> # of expected failures          624
> # of untested testcases         10
> # of unresolved testcases       3174
> # of unsupported tests          11112
>
>
> =================================================
> pipcet/avr-ccmode
>                 === gcc Summary ===
>
> # of expected passes            105431
> # of unexpected failures        709
> # of unexpected successes       16
> # of expected failures          582
> # of untested testcases         2
> # of unresolved testcases       69
> # of unsupported tests          5032
>
>                 === g++ Summary ===
>
> # of expected passes            146229
> # of unexpected failures        8291
> # of unexpected successes       21
> # of expected failures          624
> # of untested testcases         10
> # of unresolved testcases       3236
> # of unsupported tests          11112
>
> =================================================
> saadhu/avr-cc0
>                 === gcc Summary ===
>
> # of expected passes            105527
> # of unexpected failures        584
> # of unexpected successes       16
> # of expected failures          582
> # of untested testcases         2
> # of unresolved testcases       33
> # of unsupported tests          5031
>
>                 === g++ Summary ===
>
> # of expected passes            146432
> # of unexpected failures        8012
> # of unexpected successes       21
> # of expected failures          624
> # of untested testcases         10
> # of unresolved testcases       3174
> # of unsupported tests          11112
>
> Regards,
> Dimitar
>
>
>

Reply via email to