[PATCH] D138937: [AMDGPU] Update InstrCost calculation

Yashwant Singh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 20 19:48:14 PST 2022


yassingh added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/perfhint.ll:2
 ; RUN: llc -march=amdgcn < %s | FileCheck -check-prefix=GCN %s
+; RUN: llc -march=amdgcn -opaque-pointers=1 < %s | FileCheck -check-prefix=GCN %s
 
----------------
arsenm wrote:
> This is the wrong way to switch this. You need to textually replace all the pointers in the test. You can run https://gist.github.com/nikic/98357b71fd67756b0f064c9517b62a34, although that should be in a separate change from any functional patch
Updted the test to use opaque pointers [[ https://reviews.llvm.org/D140452 | D140452 ]]


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138937/new/

https://reviews.llvm.org/D138937



More information about the llvm-commits mailing list