Implement Negative Tests for clDevice Functions
@chemis01 do you mind if I split off the infrastructure required for the #1318 fix into a separate PR (with attribution of course)? Or you can do it if you'd rather. That would also simplify this one a bit and maybe help get it pushed through faster.
@chemis01 do you mind if I split off the infrastructure required for the #1318 fix into a separate PR (with attribution of course)? Or you can do it if you'd rather. That would also simplify this one a bit and maybe help get it pushed through faster.
@jlewis-austin I am completely fine with you splitting it off into a separate PR.
@chemis01 do you mind if I split off the infrastructure required for the #1318 fix into a separate PR (with attribution of course)? Or you can do it if you'd rather. That would also simplify this one a bit and maybe help get it pushed through faster.
@jlewis-austin I am completely fine with you splitting it off into a separate PR.
We ended up going with a simpler fix in #1374 to expedite it and keep it separate from the refactor work.
@kpet Rebased and CI checks are now passing.
I'm going to take off "focused review" until the review comments have been addressed - thanks!
Closing this PR because it is superseded by #2495. Outstanding comments addressed there.