Page MenuHomePhabricator

scchan (Siu Chi Chan)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 9 2016, 9:01 AM (178 w, 6 d)

Recent Activity

Aug 13 2018

scchan accepted D50626: AMDGPU: Implement llvm.amdgcn.icmp/fcmp for i16/f16.

LGTM

Aug 13 2018, 7:44 AM
scchan accepted D50624: AMDGPU: Stop producing icmp/fcmp intrinsics with invalid types.

LGTM

Aug 13 2018, 7:32 AM

Jun 22 2018

scchan added a comment to D48493: [HIP] Support flush denorms bitcode.

LGTM

Jun 22 2018, 10:52 AM · Restricted Project, Restricted Project

May 16 2018

scchan accepted D46769: [AMDGPU] Change llvm.debugtrap to be a debug breakpoint that can resume execution..
May 16 2018, 9:16 AM

Mar 21 2017

scchan added a comment to D31210: [AMDGPU] Switch address space mapping by triple environment.

I'm concerned about the default address space to be 64 bit. It would move alloca into generic address space effectively making private address to be 64 bit.
This may have very undesirable performance implications, like address arithmetic can become expensive 64 bit and only be truncated at load or store.
I realize you will use addrspacecast on an alloca's value, though I'm not sure that is sufficient to mitigate performance hit.
I believe such change shall not be made without a good performance comparison with the feature enabled, provided the very likely performance issues.

Did not we want to use this: http://lists.llvm.org/pipermail/llvm-dev/2017-March/111199.html and use non-0 for our allocas?

Our final goal is to let alloca return private pointer. The Clang changes are mostly common whether alloca returns generic pointer or private pointer. Actually to be able to test the above patch we need to get the changes in Clang done first.

Mar 21 2017, 3:43 PM

Mar 1 2017

scchan accepted D29670: [DebugInfo] Emit address space with DW_AT_address_class attribute for pointer and reference types.

LGTM

Mar 1 2017, 7:12 AM

Aug 31 2016

scchan updated subscribers of D21562: [AMDGPU] Wave and register controls.
Aug 31 2016, 7:26 AM

Mar 18 2016

scchan updated subscribers of D17280: AMDGPU: Implement {BUFFER,FLAT}_ATOMIC_CMPSWAP{,_X2}.
Mar 18 2016, 2:21 PM