Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Fix latent translation issues of GLSL built-in pack functions#1300

Merged
amdrexu merged 1 commit intoGPUOpen-Drivers:devfrom
amdrexu:bugfix
Jun 29, 2021
Merged

Fix latent translation issues of GLSL built-in pack functions#1300
amdrexu merged 1 commit intoGPUOpen-Drivers:devfrom
amdrexu:bugfix

Conversation

@amdrexu
Copy link
Copy Markdown
Contributor

@amdrexu amdrexu commented Jun 28, 2021

The pack functions packUnorm4x8/packSnorm4x8/packUnorm2x16/packSnorm2x16
always need a rounding calculation. This is missing in the translation.

Change-Id: Ib8e53debf56b94fde54225592bf85db498cc3391

@amdvlk-admin
Copy link
Copy Markdown

Test summary for commit d4e8dd9

Driver commits used in build
  • CWPACK: amd-master 7387247eb9889ddcabbc1053b9c2052e253b088e
  • METROHASH: amd-master 3c566dd9cda44ca7fd97659e0b53ac953f9037d2
  • PAL: dev 66f4ef75eed4d098f7652739b9f7dd3cd8bda6e6
  • SPVGEN: dev 79f8e32dd7242d7d34fa0074f761c9d794f11811
  • XGL: dev 5c3c2dd430222cdf1b46c9d385e9c6c5e89604e1
  • LLVM-PROJECT: amd-gfx-gpuopen-dev 0cb96bc82ca13e0fef3684a4ac8f6eff50bce257
CTS tests (Failed: 0/228128)
  • Built with version 1.2.5.2
  • Rhel 8.2, Gfx10
    • Passed: 31146/57032 (54.6%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25886/57032 (45.4%)
    Ubuntu 18.04, Gfx9
    • Passed: 31145/57032 (54.6%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25887/57032 (45.4%)
    Ubuntu 20.04, Gfx8
    • Passed: 31285/57032 (54.9%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25747/57032 (45.1%)
    Ubuntu 20.04, Gfx103
    • Passed: 34853/57032 (61.1%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 22179/57032 (38.9%)

linqun
linqun previously approved these changes Jun 28, 2021
@amdvlk-admin
Copy link
Copy Markdown

Test summary for commit b9ea7d3

Driver commits used in build
  • CWPACK: amd-master 7387247eb9889ddcabbc1053b9c2052e253b088e
  • METROHASH: amd-master 3c566dd9cda44ca7fd97659e0b53ac953f9037d2
  • PAL: dev 66f4ef75eed4d098f7652739b9f7dd3cd8bda6e6
  • SPVGEN: dev 79f8e32dd7242d7d34fa0074f761c9d794f11811
  • XGL: dev 5c3c2dd430222cdf1b46c9d385e9c6c5e89604e1
  • LLVM-PROJECT: amd-gfx-gpuopen-dev 0cb96bc82ca13e0fef3684a4ac8f6eff50bce257
CTS tests (Failed: 0/228128)
  • Built with version 1.2.5.2
  • Rhel 8.2, Gfx10
    • Passed: 31146/57032 (54.6%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25886/57032 (45.4%)
    Ubuntu 18.04, Gfx9
    • Passed: 31145/57032 (54.6%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25887/57032 (45.4%)
    Ubuntu 20.04, Gfx8
    • Passed: 31285/57032 (54.9%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25747/57032 (45.1%)
    Ubuntu 20.04, Gfx103
    • Passed: 34853/57032 (61.1%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 22179/57032 (38.9%)

The pack functions packUnorm4x8/packSnorm4x8/packUnorm2x16/packSnorm2x16
always need a rounding calculation. This is missing in the translation.

Change-Id: Ib8e53debf56b94fde54225592bf85db498cc3391
@amdvlk-admin
Copy link
Copy Markdown

Test summary for commit 8ce121a

Driver commits used in build
  • CWPACK: amd-master 7387247eb9889ddcabbc1053b9c2052e253b088e
  • METROHASH: amd-master 3c566dd9cda44ca7fd97659e0b53ac953f9037d2
  • PAL: dev 66f4ef75eed4d098f7652739b9f7dd3cd8bda6e6
  • SPVGEN: dev 79f8e32dd7242d7d34fa0074f761c9d794f11811
  • XGL: dev 5c3c2dd430222cdf1b46c9d385e9c6c5e89604e1
  • LLVM-PROJECT: amd-gfx-gpuopen-dev 0cb96bc82ca13e0fef3684a4ac8f6eff50bce257
CTS tests (Failed: 0/228128)
  • Built with version 1.2.5.2
  • Rhel 8.2, Gfx10
    • Passed: 31146/57032 (54.6%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25886/57032 (45.4%)
    Ubuntu 18.04, Gfx9
    • Passed: 31145/57032 (54.6%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25887/57032 (45.4%)
    Ubuntu 20.04, Gfx8
    • Passed: 31285/57032 (54.9%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25747/57032 (45.1%)
    Ubuntu 20.04, Gfx103
    • Passed: 34853/57032 (61.1%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 22179/57032 (38.9%)

@amdrexu amdrexu merged commit 1e5a67b into GPUOpen-Drivers:dev Jun 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants