29 August 2017, 20:19 | #161 | |
Banned
Join Date: Jan 2010
Location: Kansas
Posts: 1,284
|
Quote:
Code:
addq.l #(THRESHOLD+1),d4 ; +1 for the former dbf result Last edited by matthey; 29 August 2017 at 21:03. |
|
29 August 2017, 22:15 | #162 | |
Defendit numerus
Join Date: Mar 2017
Location: Crossing the Rubicon
Age: 54
Posts: 4,498
|
Quote:
But to be more consistent I made the decision to make all operations 32bit where possible (anyhow word access at odd locations requires 020+..) and note in braces the minimum required specifier (see attach). I also corrected a bug that you have reintroduced inserting the original A registers initialization. If LZSS packed data bound is used in final check you need to cmp.l a4,a3. See my previous message why i had changed comparison. Best regards, ross |
|
29 August 2017, 23:24 | #163 | |
Banned
Join Date: Jan 2010
Location: Kansas
Posts: 1,284
|
Quote:
Doh! How did I miss that. Thanks. Last edited by matthey; 29 August 2017 at 23:56. |
|
30 August 2017, 00:23 | #164 |
Defendit numerus
Join Date: Mar 2017
Location: Crossing the Rubicon
Age: 54
Posts: 4,498
|
wait!
With the bound check of the compressed data we can have the same problem of original code. The only properly check is the unpacked data bound. But if it was acceptable before.. EDIT: in fact is not the case as original code, the check is at the loop end, so if the stream is not corrupted last byte/word is ever a byte/word_match copy (a good collateral effect -> no out of buffer read/write) Code checked and works ross Last edited by ross; 30 August 2017 at 01:19. Reason: [] |
30 August 2017, 01:21 | #165 |
Banned
Join Date: Jan 2010
Location: Kansas
Posts: 1,284
|
@ross
Good. I already sent an e-mail to Vince today but I linked to this thread in case any late changes were necessary. Update: Vince e-mailed me but he is busy with school starting. Last edited by matthey; 03 September 2017 at 00:56. |
12 February 2018, 11:51 | #166 |
Defendit numerus
Join Date: Mar 2017
Location: Crossing the Rubicon
Age: 54
Posts: 4,498
|
Just noticed that my 54 byte version is not signaled.
http://www.deater.net/weave/vmwprod/asm/ll/ (best is the 56 bytes 8086 version) 68k deserve the throne |
Currently Active Users Viewing This Thread: 1 (0 members and 1 guests) | |
Thread Tools | |
Similar Threads | ||||
Thread | Thread Starter | Forum | Replies | Last Post |
Generated code and CPU Instruction Cache | Mrs Beanbag | Coders. Asm / Hardware | 11 | 23 May 2014 11:05 |
EAB Christmas Song-writing Contest | mr_a500 | project.EAB | 64 | 24 May 2009 02:44 |
AmigaSYS Wallpaper Contest | Calo Nord | News | 10 | 22 April 2005 09:33 |
Landover's Amiga Arcade Conversion Contest | Frog | News | 1 | 28 January 2005 23:41 |
Battlechess Contest (EAB vs A500) | Bloodwych | Nostalgia & memories | 67 | 14 August 2003 14:37 |
|
|