diff options
| author | Yang Ni <yangni@google.com> | 2016-05-04 09:44:48 -0700 |
|---|---|---|
| committer | Yang Ni <yangni@google.com> | 2016-05-06 00:11:40 +0000 |
| commit | 87643a32d4eaefe349d3a81513fffe0c4e3e8a80 (patch) | |
| tree | 37072124341e768ee7d73afb1f0a0f28fae7b13d /cpu_ref/rsCpuScript.cpp | |
| parent | 633beab5de5ffe4c5068cf1c7a1c0bd09ee9f195 (diff) | |
Avoid breaking on dead launch options
Bug: 28463187
Launch options array[2-4] are not initialized by the support library
in the current SDK.
They may have arbitrary values that invalidate the checks that this CL
is removing, and cause a kernel launch to bail out unnecessarily.
This CL removes those checks, since the array.*Start and array.*End
fields in the launch options are not used anywhere in the current
runtime.
Change-Id: Ib8a2e551619f770b402f569acbde635ee047675a
Diffstat (limited to 'cpu_ref/rsCpuScript.cpp')
| -rw-r--r-- | cpu_ref/rsCpuScript.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/cpu_ref/rsCpuScript.cpp b/cpu_ref/rsCpuScript.cpp index a88af2fe..0400fab7 100644 --- a/cpu_ref/rsCpuScript.cpp +++ b/cpu_ref/rsCpuScript.cpp @@ -548,10 +548,8 @@ bool RsdCpuScriptImpl::setUpMtlsDimensions(MTLaunchStructCommon *mtls, SET_UP_DIMENSION(x, x); SET_UP_DIMENSION(y, y); SET_UP_DIMENSION(z, z); - SET_UP_DIMENSION(array[0], array); - SET_UP_DIMENSION(array[1], array2); - SET_UP_DIMENSION(array[2], array3); - SET_UP_DIMENSION(array[3], array4); + // Checks and setup of fields other than x, y, z are ignored, since those + // fields are not used in the runtime and are not visible in the Java API. #undef SET_UP_DIMENSION return true; |