Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Constant E-Lists are not working on a PowerPC use case #8

Open
SamuraiCrow opened this issue Jan 30, 2020 · 11 comments
Open

Constant E-Lists are not working on a PowerPC use case #8

SamuraiCrow opened this issue Jan 30, 2020 · 11 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@SamuraiCrow
Copy link
Contributor

When trying to run the list hash test in framework, it doesn't produce a pointer to an EList constant.

@SamuraiCrow SamuraiCrow added bug Something isn't working good first issue Good for newcomers labels Jan 30, 2020
@SamuraiCrow
Copy link
Contributor Author

Under new version tested under MorphOS, the list still doesn't show up but the WriteF statement may be the culprit. When breaking up the WriteF to accept one parameter at a time, all but the last parameter now shows up.

@SamuraiCrow
Copy link
Contributor Author

Getter is returning 0 or NIL to both the listHash and the pointerHash. The problem is either with the compiler or hashBase.

@SamuraiCrow
Copy link
Contributor Author

I rewrote hashBase module to eliminate a pure virtual constructor. (How did I ever expect that to work?) It still returns a bunch of NILs, however my attempt at writing a minimal test case still consistently works. Is there no way to recreate this bug reliably or eliminate it definitively?

@SamuraiCrow SamuraiCrow self-assigned this Nov 14, 2021
@SamuraiCrow SamuraiCrow removed their assignment Dec 20, 2021
@SamuraiCrow SamuraiCrow added the help wanted Extra attention is needed label Dec 20, 2021
@SamuraiCrow SamuraiCrow changed the title Constant E-Lists are not working on AmigaOS4 version Constant E-Lists are not working on a PowerPC use case Dec 20, 2021
@SamuraiCrow
Copy link
Contributor Author

This may involve stack corruption. @mness1978 had found similar stack corruption in the CreativE builds in Assembly. This may need more serious consideration.

@SamuraiCrow SamuraiCrow added High priority and removed good first issue Good for newcomers labels Apr 29, 2023
@dmcoles
Copy link
Contributor

dmcoles commented Apr 30, 2023

Can you explain more accurately where the issue lies and what the symptoms are? I tried compiling the listhashtest using evo and then ecx targeting os3 and os4 and got similar results with all 3

@SamuraiCrow
Copy link
Contributor Author

When I declare an E-List inline, I'd sometimes get a NIL value. The test code is from the EEC framework in the hash data structures. https://github.com/EEC-Developers/framework/blob/master/Hash/ListHashTest.e is one I've never gotten to work right.

@dmcoles
Copy link
Contributor

dmcoles commented Apr 30, 2023

I don't think I can recreate this

@SamuraiCrow
Copy link
Contributor Author

It may be an artifact of another, already-solved bug. I'll see if I can re-create it myself. If not, I'll just close it. Thanks for looking, @dmcoles .

@dmcoles
Copy link
Contributor

dmcoles commented May 1, 2023

I did find an issue and some inconsistency around how ecx works vs original E/E-VO in relation to lists though. I've sent you a pm on Facebook as it may be easier to discuss. I think it would be good to have consistency between these compilers

@SamuraiCrow
Copy link
Contributor Author

Thanks for looking again! I'm not often on Facebook anymore but I'll look.

@SamuraiCrow
Copy link
Contributor Author

Line 7656 in eecmain.e is one place to look because that's where elists are parsed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants