[PATCH] D40955: [AMDGPU] Make the new addr space mapping default

Yaxun Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 8 12:27:37 PST 2018


yaxunl added inline comments.


================
Comment at: test/Analysis/CostModel/AMDGPU/addrspacecast.ll:2
 ; RUN: opt -cost-model -analyze -mtriple=amdgcn-unknown-amdhsa -mcpu=kaveri < %s | FileCheck %s
+target datalayout = "A5"
 
----------------
yaxunl wrote:
> arsenm wrote:
> > That this is necessary is a bug. The specified triple should be sufficient
> Removed. Thanks.
Fixed this issue in another patch https://reviews.llvm.org/D41832


https://reviews.llvm.org/D40955





More information about the llvm-commits mailing list