search

GPUOpen-Drivers / llpc

LLVM-Based Pipeline Compiler
MIT License
166 stars 115 forks source link

Replace getInt8PtrTy which has now been removed from upstream LLVM #2864

Closed dstutt closed 10 months ago

dstutt commented 10 months ago

This function was changed to return just getPtrTy some time ago in upstream LLVM, but the function was left in place to assist updating.

The upstream function has now been removed with the instruction that anything using the getInt8PtrTy builder function should switch to getPtrTy.

Since the updates for this happened several months ago, I'm not adding version guards.

amdvlk-admin commented 10 months ago

Test summary for commit b9723c06beccee7b1c7b29ae3d032269338c4a4c

CTS tests (Failed: 0/138443)
  • Built with version 1.3.5.2
    • Ubuntu navi3x, Srdcvk
    • Passed: 35211/69228 (50.9%)
    • Failed: 0/69228 (0.0%)
    • Not Supported: 34017/69228 (49.1%)
    • Warnings: 0/69228 (0.0%)
    Ubuntu navi2x, Srdcvk
    • Passed: 35241/69215 (50.9%)
    • Failed: 0/69215 (0.0%)
    • Not Supported: 33974/69215 (49.1%)
    • Warnings: 0/69215 (0.0%)
amdvlk-admin commented 10 months ago

Test summary for commit 9e133040bc9cabf15a6de15b2ffcc30c8d522110

CTS tests (Failed: 0/138378)
  • Built with version 1.3.5.2
    • Ubuntu navi3x, Srdcvk
    • Passed: 35162/69163 (50.8%)
    • Failed: 0/69163 (0.0%)
    • Not Supported: 34001/69163 (49.2%)
    • Warnings: 0/69163 (0.0%)
    Ubuntu navi2x, Srdcvk
    • Passed: 35241/69215 (50.9%)
    • Failed: 0/69215 (0.0%)
    • Not Supported: 33974/69215 (49.1%)
    • Warnings: 0/69215 (0.0%)