rjmccall added a comment.
Okay. That does seem more sensible than trying to change everything around the
other way.
https://reviews.llvm.org/D27627
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
yaxunl abandoned this revision.
yaxunl added a comment.
We consider to switch to use 0 as generic address space and abandon this since
0 as generic address space is more agreeable to both LLVM and Clang's
convention.
https://reviews.llvm.org/D27627
__
kzhuravl added a comment.
Once you rebase, can you also update AMDGPUTargetInfo::getDWARFAddressSpace to
use AddrSpaceKind?
https://reviews.llvm.org/D27627
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailma
yaxunl updated this revision to Diff 90243.
yaxunl edited the summary of this revision.
https://reviews.llvm.org/D27627
Files:
include/clang/AST/ASTContext.h
include/clang/Basic/TargetInfo.h
lib/AST/ASTContext.cpp
lib/Basic/TargetInfo.cpp
lib/Basic/Targets.cpp
lib/CodeGen/CGBuiltin.cp
yaxunl updated this revision to Diff 89944.
yaxunl added a comment.
Fixed invalid bitcasts due to vtable.
Added run line for amdgcn target to more Cuda lit tests.
https://reviews.llvm.org/D27627
Files:
include/clang/AST/ASTContext.h
include/clang/Basic/TargetInfo.h
lib/AST/ASTContext.cpp
yaxunl updated this revision to Diff 89899.
yaxunl retitled this revision from "[WIP] Specify default address space for C++
on AMDGPU Target" to "[WIP] Supporting C++ based kernel languages on AMDGPU
Target".
yaxunl edited the summary of this revision.
yaxunl added a comment.
Fix invalid bitcast
