Add CTS for scratch register reading. #89
Conversation
| } | ||
| } | ||
|
|
||
| TEST_F( |
There was a problem hiding this comment.
Query why cant we use the same test case which @bmyates mentioned in our sync?
There was a problem hiding this comment.
Then we might need to modify the kernel too. Kernel is not allocating any local memory as such.
There was a problem hiding this comment.
Any difficulty in modifing the kernel? I hope scratch is forced using IGC_OPTs, so maybe with these options kernel modification might not be required
There was a problem hiding this comment.
yes please try to use the same test. The kernel should not need to allocate any local memory. Local memory is for SLM - scratch is different. The compiler will determine if scratch is needed or not based on register usage.
Try to use same kernel. If scratch isnt allocated then we can try to use IGC opt. If we still are not getting any spills I can give some suggestions to modify kernel
aviralni
force-pushed
the
scratch_reg_cts
branch
from
4697059
to
79e34c0
Compare
No description provided.