Bug 75631 - intel-gen4asm can't assemble intel-gen4disasm's output
Summary: intel-gen4asm can't assemble intel-gen4disasm's output
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: XOrg git
Hardware: Other All
: medium normal
Assignee: haihao
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-01 01:41 UTC by Matt Turner
Modified: 2017-07-24 22:55 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
patch to switch swizzzle and regtype (1.35 KB, patch)
2014-03-25 07:30 UTC, haihao
no flags Details | Splinter Review
distinguish channel .z from condition .z (1.83 KB, patch)
2014-03-25 07:32 UTC, haihao
no flags Details | Splinter Review

Description Matt Turner 2014-03-01 01:41:11 UTC
echo "mov (8) g1<1>.x g2<4,4,1>.y { align16 };" | ./intel-gen4asm - | ./intel-gen4disasm - | ./intel-gen4asm -
error: write mask set in align1 instruction

Setting a break point on set_instruction_options and add_option shows that the options haven't even been parsed when set_instruction_options() is called.

Can't even assemble this, which is straight out of the PRM

dp4 (8) r5.0<1>.z:F r2.0<4;4,1>:F r4.0<4;4,1>:F { align16 };

1: syntax error at ".z"
Comment 1 Daniel Vetter 2014-03-03 07:46:44 UTC
Addina Haihao and Damien as our residential assembler people.

Matt, do you have a specific use-case (in mesa or somewhere) which would make fixing this useful?
Comment 2 Matt Turner 2014-03-03 07:50:45 UTC
I was attempting to assemble some code so substitute into Mesa in place of our generated code to see if it was worth optimizing.
Comment 3 haihao 2014-03-25 07:30:21 UTC
Created attachment 96342 [details] [review]
patch to switch swizzzle and regtype

to fix  echo "mov (8) g1<1>.x g2<4,4,1>.y { align16 };" | ./intel-gen4asm - | ./intel-gen4disasm - | ./intel-gen4asm -
Comment 4 haihao 2014-03-25 07:32:18 UTC
Created attachment 96343 [details] [review]
distinguish channel .z from condition .z

scratch path to fix dp4 (8) r5.0<1>.z:F r2.0<4;4,1>:F r4.0<4;4,1>:F { align16 };
Comment 5 Matt Turner 2014-05-19 21:19:45 UTC
Both patches look good to me. Thanks Haihao!

Tested-by: Matt Turner <mattst88@gmail.com>
Comment 6 Damien Lespiau 2014-05-19 21:51:52 UTC
Pushed, thanks for the report, patches and testing.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.