<div dir="ltr">LGTM<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 6, 2015 at 7:01 PM, Tom Stellard <span dir="ltr"><<a href="mailto:thomas.stellard@amd.com" target="_blank">thomas.stellard@amd.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This implementation was ported from the AMD builtin library<br>
and has been tested with piglit, OpenCV, and the ocl conformance tests.<br>
---<br>
 generic/include/clc/clc.h                     |  1 +<br>
 generic/include/clc/geometric/fast_length.h   | 24 +++++++++++<br>
 generic/include/clc/geometric/fast_length.inc | 23 ++++++++++<br>
 generic/lib/SOURCES                           |  1 +<br>
 generic/lib/geometric/<a href="http://fast_length.cl" target="_blank">fast_length.cl</a>          | 60 +++++++++++++++++++++++++++<br>
 5 files changed, 109 insertions(+)<br>
 create mode 100644 generic/include/clc/geometric/fast_length.h<br>
 create mode 100644 generic/include/clc/geometric/fast_length.inc<br>
 create mode 100644 generic/lib/geometric/<a href="http://fast_length.cl" target="_blank">fast_length.cl</a><br>
<br>
diff --git a/generic/include/clc/clc.h b/generic/include/clc/clc.h<br>
index d657cad..760a086 100644<br>
--- a/generic/include/clc/clc.h<br>
+++ b/generic/include/clc/clc.h<br>
@@ -117,6 +117,7 @@<br>
 #include <clc/geometric/cross.h><br>
 #include <clc/geometric/distance.h><br>
 #include <clc/geometric/dot.h><br>
+#include <clc/geometric/fast_length.h><br>
 #include <clc/geometric/length.h><br>
 #include <clc/geometric/normalize.h><br>
<br>
diff --git a/generic/include/clc/geometric/fast_length.h b/generic/include/clc/geometric/fast_length.h<br>
new file mode 100644<br>
index 0000000..2c7b80a<br>
--- /dev/null<br>
+++ b/generic/include/clc/geometric/fast_length.h<br>
@@ -0,0 +1,24 @@<br>
+/*<br>
+ * Copyright (c) 2014,2015 Advanced Micro Devices, Inc.<br>
+ *<br>
+ * Permission is hereby granted, free of charge, to any person obtaining a copy<br>
+ * of this software and associated documentation files (the "Software"), to deal<br>
+ * in the Software without restriction, including without limitation the rights<br>
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell<br>
+ * copies of the Software, and to permit persons to whom the Software is<br>
+ * furnished to do so, subject to the following conditions:<br>
+ *<br>
+ * The above copyright notice and this permission notice shall be included in<br>
+ * all copies or substantial portions of the Software.<br>
+ *<br>
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br>
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,<br>
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE<br>
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER<br>
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,<br>
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN<br>
+ * THE SOFTWARE.<br>
+ */<br>
+<br>
+#define __CLC_BODY <clc/geometric/fast_length.inc><br>
+#include <clc/geometric/floatn.inc><br>
diff --git a/generic/include/clc/geometric/fast_length.inc b/generic/include/clc/geometric/fast_length.inc<br>
new file mode 100644<br>
index 0000000..5a7c275<br>
--- /dev/null<br>
+++ b/generic/include/clc/geometric/fast_length.inc<br>
@@ -0,0 +1,23 @@<br>
+/*<br>
+ * Copyright (c) 2014,2015 Advanced Micro Devices, Inc.<br>
+ *<br>
+ * Permission is hereby granted, free of charge, to any person obtaining a copy<br>
+ * of this software and associated documentation files (the "Software"), to deal<br>
+ * in the Software without restriction, including without limitation the rights<br>
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell<br>
+ * copies of the Software, and to permit persons to whom the Software is<br>
+ * furnished to do so, subject to the following conditions:<br>
+ *<br>
+ * The above copyright notice and this permission notice shall be included in<br>
+ * all copies or substantial portions of the Software.<br>
+ *<br>
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br>
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,<br>
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE<br>
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER<br>
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,<br>
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN<br>
+ * THE SOFTWARE.<br>
+ */<br>
+<br>
+_CLC_OVERLOAD _CLC_DECL __CLC_FLOAT fast_length(__CLC_FLOATN p0);<br>
diff --git a/generic/lib/SOURCES b/generic/lib/SOURCES<br>
index fb56135..02a05e4 100644<br>
--- a/generic/lib/SOURCES<br>
+++ b/generic/lib/SOURCES<br>
@@ -35,6 +35,7 @@ common/<a href="http://step.cl" target="_blank">step.cl</a><br>
 geometric/<a href="http://cross.cl" target="_blank">cross.cl</a><br>
 geometric/<a href="http://distance.cl" target="_blank">distance.cl</a><br>
 geometric/<a href="http://dot.cl" target="_blank">dot.cl</a><br>
+geometric/<a href="http://fast_length.cl" target="_blank">fast_length.cl</a><br>
 geometric/<a href="http://length.cl" target="_blank">length.cl</a><br>
 geometric/<a href="http://normalize.cl" target="_blank">normalize.cl</a><br>
 integer/<a href="http://abs.cl" target="_blank">abs.cl</a><br>
diff --git a/generic/lib/geometric/<a href="http://fast_length.cl" target="_blank">fast_length.cl</a> b/generic/lib/geometric/<a href="http://fast_length.cl" target="_blank">fast_length.cl</a><br>
new file mode 100644<br>
index 0000000..425dcf7<br>
--- /dev/null<br>
+++ b/generic/lib/geometric/<a href="http://fast_length.cl" target="_blank">fast_length.cl</a><br>
@@ -0,0 +1,60 @@<br>
+/*<br>
+ * Copyright (c) 2014 Advanced Micro Devices, Inc.<br>
+ *<br>
+ * Permission is hereby granted, free of charge, to any person obtaining a copy<br>
+ * of this software and associated documentation files (the "Software"), to deal<br>
+ * in the Software without restriction, including without limitation the rights<br>
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell<br>
+ * copies of the Software, and to permit persons to whom the Software is<br>
+ * furnished to do so, subject to the following conditions:<br>
+ *<br>
+ * The above copyright notice and this permission notice shall be included in<br>
+ * all copies or substantial portions of the Software.<br>
+ *<br>
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br>
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,<br>
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE<br>
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER<br>
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,<br>
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN<br>
+ * THE SOFTWARE.<br>
+ */<br>
+<br>
+#include <clc/clc.h><br>
+<br>
+_CLC_OVERLOAD _CLC_DEF float fast_length(float p) {<br>
+  return fabs(p);<br>
+}<br>
+<br>
+_CLC_OVERLOAD _CLC_DEF float fast_length(float2 p) {<br>
+  return half_sqrt(dot(p, p));<br>
+}<br>
+<br>
+_CLC_OVERLOAD _CLC_DEF float fast_length(float3 p) {<br>
+  return half_sqrt(dot(p, p));<br>
+}<br>
+<br>
+_CLC_OVERLOAD _CLC_DEF float fast_length(float4 p) {<br>
+  return half_sqrt(dot(p, p));<br>
+}<br>
+<br>
+#ifdef cl_khr_fp64<br>
+#pragma OPENCL EXTENSION cl_khr_fp64 : enable<br>
+<br>
+_CLC_OVERLOAD _CLC_DEF double fast_length(double p) {<br>
+  return fabs(p);<br>
+}<br>
+<br>
+_CLC_OVERLOAD _CLC_DEF double fast_length(double2 p) {<br>
+  return half_sqrt(dot(p, p));<br>
+}<br>
+<br>
+_CLC_OVERLOAD _CLC_DEF double fast_length(double3 p) {<br>
+  return half_sqrt(dot(p, p));<br>
+}<br>
+<br>
+_CLC_OVERLOAD _CLC_DEF double fast_length(double4 p) {<br>
+  return half_sqrt(dot(p, p));<br>
+}<br>
+<br>
+#endif<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.0.4<br>
<br>
<br>
_______________________________________________<br>
Libclc-dev mailing list<br>
<a href="mailto:Libclc-dev@pcc.me.uk">Libclc-dev@pcc.me.uk</a><br>
<a href="http://www.pcc.me.uk/cgi-bin/mailman/listinfo/libclc-dev" target="_blank">http://www.pcc.me.uk/cgi-bin/mailman/listinfo/libclc-dev</a><br>
</font></span></blockquote></div><br></div>