scalar kernel arguments broken in tce driver

Bug #1138472 reported by Kalle Raiskila
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pocl
Fix Released
High
Jesse Towner

Bug Description

"Basic ttasim driver test" gets kernel parameters 3 and 4 (the float a and int b) as zeros.
This broke in rev. 584.

Reverting lib/llvmopencl/Workgroup.cc from 583 seems to fix this issue.

Also, 581 masks this issue, as device parameters are checked pedantically,
work-around is to set max_mem_alloc_size to e.g. 1024 in lib/CL/devices/tce/ttasim/ttasim.h

Related branches

Revision history for this message
Pekka Jääskeläinen (pekka-jaaskelainen) wrote :

Do you have guesses on this? Something related to that 'byval' fix somehow caused this breakage. Probably not easiest to reproduce as it needs the TCE installed, but if you have an idea, let us know.

Changed in pocl:
assignee: nobody → Jesse Towner (jessetowner)
Revision history for this message
Pekka Jääskeläinen (pekka-jaaskelainen) wrote :

(That was to Jesse)

Revision history for this message
Pekka Jääskeläinen (pekka-jaaskelainen) wrote :

Fixed in r632.

Changed in pocl:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.