Bug 385208

Summary:	xxperm instruction exhausts temporary memory
Product:	[Developer tools] valgrind	Reporter:	Carl Love <cel>
Component:	vex	Assignee:	Julian Seward <jseward>
Status:	CLOSED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	3.14 SVN
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	re-implement xxperm instruction

Description Carl Love 2017-09-29 18:03:31 UTC

The xxperm power 9 instruction generates too many iops.  It causes temporary memory to be exhausted.

Comment 1 Carl Love 2017-09-29 18:05:29 UTC

Created attachment 108094 [details]
re-implement xxperm instruction

The xxperm code was re-implemented using the Iop_Perm8x16 similarly to the vperm instruction.

Comment 2 Julian Seward 2017-10-03 10:05:04 UTC

Looks OK to me.

Comment 3 Carl Love 2017-10-03 20:13:42 UTC

commited patch:  commit a1d03d0d11c0b31a6d9f57baa4d46317fdd5f6ef