This revision was automatically updated to reflect the committed changes.
Closed by commit rL366683: Updated the signature for some stack related
intrinsics (CLANG) (authored by cdevadas, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to
arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.
LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64563/new/
https://reviews.llvm.org/D64563
___
cfe-commits mailing list
cdevadas updated this revision to Diff 210648.
cdevadas added a comment.
Herald added a subscriber: jvesely.
updated test/CodeGenOpenCL/builtins-generic-amdgcn.cl to vaildate the address
space.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64563/new/
https://reviews.llvm.org/D64563
arsenm added a comment.
Can you add AMDGPU tests for these showing the correct address space is used
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64563/new/
https://reviews.llvm.org/D64563
___
cfe-commits mailing list
cdevadas updated this revision to Diff 209467.
cdevadas added a comment.
Added alloca address space.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64563/new/
https://reviews.llvm.org/D64563
Files:
lib/CodeGen/CGBuiltin.cpp
lib/CodeGen/CGException.cpp
cdevadas created this revision.
cdevadas added a reviewer: arsenm.
Herald added subscribers: cfe-commits, wdng.
Herald added a project: clang.
This patch includes the intrinsics
int_addressofreturnaddress,
int_frameaddress & int_sponentry.
This patch should go along with the
changes in