Bug 110309 - [icl][bisected] regression on piglit arb_gpu_shader_int 64.execution.fs-ishl-then-* tests
Summary: [icl][bisected] regression on piglit arb_gpu_shader_int 64.execution.fs-ishl-...
Status: ASSIGNED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/i965 (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Ian Romanick
QA Contact: Intel 3D Bugs Mailing List
URL:
Whiteboard:
Keywords: bisected, regression
Depends on:
Blocks:
 
Reported: 2019-04-02 17:35 UTC by Clayton Craft
Modified: 2019-04-02 17:46 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
output from bisected commit with INTEL_DEBUG=fs (26.50 KB, text/plain)
2019-04-02 17:35 UTC, Clayton Craft
Details
output from previous (working) commit with INTEL_DEBUG=fs (35.62 KB, text/plain)
2019-04-02 17:35 UTC, Clayton Craft
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Clayton Craft 2019-04-02 17:35:14 UTC
Created attachment 143848 [details]
output from bisected commit with INTEL_DEBUG=fs

The following tests regressed in CI:

piglit.spec.arb_gpu_shader_int64.execution.fs-ishl-then-ushr-loop
piglit.spec.arb_gpu_shader_int64.execution.fs-ishl-then-ishr piglit.spec.arb_gpu_shader_int64.execution.fs-ishl-then-ishr-loop piglit.spec.arb_gpu_shader_int64.execution.fs-ishl-then-ushr


Output from bisected and previous commits run with INTEL_DEBUG=fs is attached.

Bisected to the following commit:

commit cbad201c2b34297dda11ffa414de2ef781225259
Author: Ian Romanick <ian.d.romanick@intel.com>
Date:   Wed Feb 27 20:12:46 2019 -0800

    nir/algebraic: Add missing 64-bit extract_[iu]8 patterns

    No shader-db changes on any Intel platform.

    v2: Use a loop to generate patterns.  Suggested by Jason.

    v3: Fix a copy-and-paste bug in the extract_[ui] of ishl loop that would
    replace an extract_i8 with and extract_u8.  This broke ~180 tests.  This
    bug was introduced in v2.

    Reviewed-by: Matt Turner <mattst88@gmail.com> [v1]
    Reviewed-by: Dylan Baker <dylan@pnwbakers.com> [v2]
    Acked-by: Jason Ekstrand <jason@jlekstrand.net> [v2]
Comment 1 Clayton Craft 2019-04-02 17:35:53 UTC
Created attachment 143849 [details]
output from previous (working) commit with INTEL_DEBUG=fs
Comment 2 Ian Romanick 2019-04-02 17:46:26 UTC
Heh... when you said it failed, I didn't realize the failure was an assertion. :)  ICL doesn't have 64-bit integers.  We lower 64-bit shifts, but it seems that there is no lowering for 64-bit extract operations.

We either need to disable this optimization for platforms that are going to lower 64-bit integer operations, add a lowering for 64-bit extract operations, or both.


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.