tinycc-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Tinycc-devel] [PATCH 5/5] Fix gv for long longs


From: grischka
Subject: Re: [Tinycc-devel] [PATCH 5/5] Fix gv for long longs
Date: Thu, 11 Sep 2008 05:06:43 +0200

Thanks Daniel.

Now, while I am able to apply your last 5 patches cleanly, I seem to
have some problems with those from:
http://lists.gnu.org/archive/html/tinycc-devel/2003-10/msg00044.html

There are three diffs:
- arm.diff : some hunks seem to exist already in CVS
- bugfixes.diff : applies mostly but not completly
- improvements.diff : applies completly

So, I know it's a pain, but can you check these once more?

Also, as far as those patches address several possibly unrelated issues:
Could you MAYBE consider to split those into single commits with an
applicable (one line) comment each? (I've had to learn that too, lately ;)

However that being said I'm not really in the mood to extract series of
patches that I don't even understand really, from dozens of emails.

So, first of all: Do you think you could ask Kirill Smelkov for commit
rights on http://repo.or.cz/w/tinycc.git and then apply your patches from
2003 (or what is left from them) on top (or below or whatever) of the
"patches-from-Daniel" branch, there?

Alternatively maybe you want Savannah CVS access, so you could work directly
to good/bad old CVS?

Alternatively other suggestions ...

--- grischka



----- Original Message -----
From: "Daniel Glöckner" <address@hidden>
To: <address@hidden>
Sent: Friday, September 05, 2008 9:08 PM
Subject: [Tinycc-devel] [PATCH 5/5] Fix gv for long longs


> long long a();
> long long b() {
>   return a();
> }
>
> At the end of b there will be some useless register shuffling.
> This is because return wants to have the result of a in REG_IRET.
> gv checks if this is the case for BOTH registers of the long long.
> After this test it uses REG_LRET for the second register if the
> first is supposed to be REG_IRET. In other cases it uses RC_INT.
>
> The patch compares the second register against the class it will
> have in the end instead of the register class the first register
> will have.
>
>
>
> At this point I would like to remind those who pick up the patches
> that there are two other mails by me with uncommitted fixes:
> http://lists.gnu.org/archive/html/tinycc-devel/2003-10/msg00044.html
> http://lists.gnu.org/archive/html/tinycc-devel/2008-08/msg00007.html
>
>   Daniel
>


--------------------------------------------------------------------------------


> _______________________________________________
> Tinycc-devel mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/tinycc-devel
>





reply via email to

[Prev in Thread] Current Thread [Next in Thread]