Re: FastCode RTL Replacement v0.20 Released!
- From: "John O'Harrow" <john@xxxxxxxxxxxxxxxxxxxx>
- Date: Mon, 10 Oct 2005 13:28:42 +0100
"Dennis" <marianndkc@xxxxxxxxxxxxxxx> wrote in message
news:434a58fc@xxxxxxxxxxxxxxxxxxxxxxxxx
> Hi John
>
>> Are you suggesting that where the same function is the winner on more
>> than
>> one target, that we completely dupliicate the function (where absolutely
>> no code should be required).
>
> Yes. There is no way to have two different function interfaces pointing to
> the same implementation.
Of course there is. This is exactly what defining a constant function
pointer and assigning is to the correct implementation does (with no code
whatsoever). This is what Charalabos has sucessfully implemented.
> This is an old discussion and we have 30+ librariers present on our site
> based on this design. It has been this way for more than 2 years.
I don't recall ever discussing this. Just because it has been that way for
2 years doesn't make it right.
Why implement the same function 8 times (as in Uppercase) when just a single
implementation is needed. If we later create a faster implementation for a
particular target, it can be easily added.
By duplicating the same function 8 times, we could be creating potential
problems further down the line. If for example, we later find a bug on a
particular target, It would be very easy to correct the target version, but
overlook the other identical functions.
regards,
John.
.
- Follow-Ups:
- Re: FastCode RTL Replacement v0.20 Released!
- From: Dennis
- Re: FastCode RTL Replacement v0.20 Released!
- References:
- FastCode RTL Replacement v0.20 Released!
- From: Charalabos Michael
- Re: FastCode RTL Replacement v0.20 Released!
- From: Dennis
- Re: FastCode RTL Replacement v0.20 Released!
- From: Charalabos Michael
- Re: FastCode RTL Replacement v0.20 Released!
- From: Dennis
- Re: FastCode RTL Replacement v0.20 Released!
- From: Dennis
- Re: FastCode RTL Replacement v0.20 Released!
- From: John O'Harrow
- Re: FastCode RTL Replacement v0.20 Released!
- From: Dennis
- FastCode RTL Replacement v0.20 Released!
- Prev by Date: Re: FastCode RTL Replacement v0.20 Released!
- Next by Date: Re: FastCode RTL Replacement v0.20 Released!
- Previous by thread: Re: FastCode RTL Replacement v0.20 Released!
- Next by thread: Re: FastCode RTL Replacement v0.20 Released!
- Index(es):
Relevant Pages
|