Opened 11 years ago

Closed 11 years ago

Last modified 9 years ago

#372 closed defect (fixed)

make check error on Linux/PPC (Error: assertion failed: (= -1 (pointer-s8-ref some-chunk)))

Reported by: Mario Domenech Goulart Owned by: felix winkelmann
Priority: major Milestone: 4.9.0
Component: compiler Version: 4.5.0
Keywords: make check Cc:
Estimated difficulty:

Description

make check causes an error on Linux/PPC. I caught that when trying the make-refactoring branch, but could also reproduce it for 4.5.0 (the release tarball).

The logs are attached.

Nevermind the /home/mario/chicken-safer-scheduler prefix in make-check-make-refacoring.log -- it's really the make-refactoring branch (I don't know where my mind was when I typed PREFIX=/home/mario/chicken-safer-scheduler...).

Attachments (3)

make-check-4.5.0.log (43.1 KB) - added by Mario Domenech Goulart 11 years ago.
make check log for 4.5.0
make-check-make-refactoring.log (45.0 KB) - added by Mario Domenech Goulart 11 years ago.
make check log for chicken from the make-refactoring branch
signed-chars.patch (3.0 KB) - added by sjamaan 11 years ago.
Explicitly signed chars for s8 vector ops

Download all attachments as: .zip

Change History (8)

Changed 11 years ago by Mario Domenech Goulart

Attachment: make-check-4.5.0.log added

make check log for 4.5.0

Changed 11 years ago by Mario Domenech Goulart

make check log for chicken from the make-refactoring branch

comment:1 Changed 11 years ago by sjamaan

Just checked and it fails here too (NetBSD/macppc). This is the old "characters are unsigned by default on some platforms" problem we've had before in the FFI.

This means "char *" is equivalent to "unsigned char *", which is easy to fix; just put an explicit "signed" in front of each bare "char". A patch follows.

Changed 11 years ago by sjamaan

Attachment: signed-chars.patch added

Explicitly signed chars for s8 vector ops

comment:2 Changed 11 years ago by felix winkelmann

Owner: set to felix winkelmann
Status: newassigned

comment:3 Changed 11 years ago by felix winkelmann

Resolution: fixed
Status: assignedclosed

patch applied.

comment:4 Changed 10 years ago by felix winkelmann

Milestone: 4.7.04.8.0

Milestone 4.7.0 deleted

comment:5 Changed 9 years ago by felix winkelmann

Milestone: 4.8.04.9.0

Milestone 4.8.0 deleted

Note: See TracTickets for help on using tickets.