bugKawa - Bugs: bug #5651, primitive-array-set <int>...

 
 

bug #5651: primitive-array-set <int> broken

Submitted by:  Chris Dean <ctdean>
Submitted on:  Wed 01 Oct 2003 02:28:06 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Per Bothner <bothner>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Thu 09 Oct 2003 01:48:43 AM UTC, comment #2:

I checked in fixes for this.

Per Bothner <bothner>
Project AdministratorIn charge of this item.
Wed 01 Oct 2003 06:32:17 AM UTC, comment #1:

Well, primitive-array-set <int> does work it you
use it "directly", so it can be inlined. The
problem is when it idn't inlined, the compiler
runs into a problem compiling <int>.

There are various ways to fix this. What I'm looking
at is a modification to LitTable where it searches for
a matching public static final field, and compiles a
reference for that. That would be nice and general
(and I can get rid of some existing special cases such
as for Boolean). I've sketched it out; I'll try to
have something tested tomorrow.

Per Bothner <bothner>
Project AdministratorIn charge of this item.
Wed 01 Oct 2003 02:28:06 AM UTC, original submission:

When I attempt to compile this code, I get a compiler error:

(define (foo (array :: <int[]>))
(let ((setter (primitive-array-set <int>)))
(setter array 0 99)))

java.lang.Error: gnu.kawa.lispexpr.LangPrimType does not implement
Externalizable
at gnu.expr.LitTable.error(LitTable.java:82)
at gnu.expr.LitTable.writeObject(LitTable.java:217)
at gnu.kawa.reflect.ArraySet.writeExternal(ArraySet.java:40)
at gnu.expr.LitTable.writeObject(LitTable.java:187)
at gnu.expr.LitTable.emit(LitTable.java:45)
at gnu.expr.Compilation.emitLiterals(Compilation.java:473)
at gnu.expr.Compilation.addClass(Compilation.java:1612)
at gnu.expr.Compilation.compile(Compilation.java:816)
at gnu.expr.Compilation.compileToFiles(Compilation.java:863)
at kawa.lang.CompileFile.compile_to_files(CompileFile.java:92)

Chris Dean <ctdean>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #703:  pa.patch added by ctdean (2KiB - application/octet-stream - Patch for testsuite)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 3 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Thu 09 Oct 2003 01:48:43 AM UTCbothnerStatusNone=>(Error - Not Found)
  Assigned toNone=>NA
  Open/ClosedOpen=>(Error - Not Found)

Back to the top


Powered by Savane 3.1-cleanup1