search

GPUOpen-LibrariesAndSDKs / RadeonProRender-Baikal

MIT License
335 stars 78 forks source link

Warning fixes #98

Closed fo-fo closed 6 years ago

fo-fo commented 6 years ago

Fixed a couple of things that I spotted from the OpenCL compiler warnings.

Kvalme commented 6 years ago

This change will rise another problem. "AdvanceIterationCount" kernel receives hitcount as ray number, so it's invalid. To fix this you need to pass "size" into this kernel in "path_tracing_estimator.cpp" instead of "m_render_data->hitcount". And modify kernel to receive this parameter by value.

yozhijk commented 6 years ago

Hi @fo-fo ,

Thanks for a good catch. Could you apply the remark @Kvalme pointed out? I will merge you PR afterwards.