search

ROCm / ROCm-Device-Libs

ROCm Device Libraries
97 stars 60 forks source link

wgscratch datalayout is different from llvm-link #92

Closed aaronmondal closed 1 year ago

aaronmondal commented 1 year ago

I'm getting the following warning with llvm-project@a268127 :

warning: Linking two modules of different data layouts: '.../ockl/src/wgscratch.ll' is

'e-p:64:64-p1:64:64-p2:32:32-p3:32:32-p4:64:64-p5:32:32-p6:32:32-i64:64-v16:16-v24:32-v32:32-v48:64-v96:128-v192:256-v256:256-v512:512-v1024:1024-v2048:2048-n32:64-S32-A5-G1-ni:7'

whereas 'llvm-link' is

'e-p:64:64-p1:64:64-p2:32:32-p3:32:32-p4:64:64-p5:32:32-p6:32:32-p7:160:256:256:32-p8:128:128-i64:64-v16:16-v24:32-v32:32-v48:64-v96:128-v192:256-v256:256-v512:512-v1024:1024-v2048:2048-n32:64-S32-A5-G1-ni:7:8'

It appears that the difference is

ni:7

vs

p7:160:256:256:32-p8:128:128-ni:7:8
b-sumner commented 1 year ago

Thanks for reporting this. We'll fix it on our side. It should show up soon.

aaronmondal commented 1 year ago

Fixed by https://github.com/RadeonOpenCompute/ROCm-Device-Libs/commit/93b49570ed84a74790fef98497c68ff3656c3743.