[Libclc-dev] [PATCH 2/8] Implement distance builtin

Tom Stellard tom at stellard.net
Mon Mar 9 07:30:01 PDT 2015 

On Sat, Mar 07, 2015 at 07:31:21AM -0600, Aaron Watry wrote:
> Thanks for these Tom.
> 
> I plan on reviewing them asap, and feel silly that I had written distance,
> fast distance and fast length, but hadn't managed to mail them in yet.
> 

I was a little worried about something like this.  I started working and got on
a roll so I figured I would just finish out the geom functions.

> For reference, I'm working on erfc, erf, and ldexp... the only one I
> haven't finished up yet is erf (trying to figure out how to deal with lack
> of subnormal support on devices that don't support doubles).
> 

Supporting subnormals for float types is not required by OpenCL 1.1/1.2.
Does this means we can ignore them in the library?

-Tom


> --Aaron
> On Mar 6, 2015 8:06 PM, "Tom Stellard" <thomas.stellard at amd.com> wrote:
> 
> > This implementation was ported from the AMD builtin library
> > and has been tested with piglit, OpenCV, and the ocl conformance tests.
> > ---
> >  generic/include/clc/clc.h                  |  1 +
> >  generic/include/clc/geometric/distance.h   | 22 ++++++++++++++++++++++
> >  generic/include/clc/geometric/distance.inc | 23 +++++++++++++++++++++++
> >  generic/lib/SOURCES                        |  1 +
> >  generic/lib/geometric/distance.cl          | 30
> > ++++++++++++++++++++++++++++++
> >  generic/lib/geometric/distance.inc         | 25 +++++++++++++++++++++++++
> >  6 files changed, 102 insertions(+)
> >  create mode 100644 generic/include/clc/geometric/distance.inc
> >  create mode 100644 generic/lib/geometric/distance.cl
> >  create mode 100644 generic/lib/geometric/distance.inc
> >
> > diff --git a/generic/include/clc/clc.h b/generic/include/clc/clc.h
> > index cee1614..2fb818a 100644
> > --- a/generic/include/clc/clc.h
> > +++ b/generic/include/clc/clc.h
> > @@ -114,6 +114,7 @@
> >
> >  /* 6.11.5 Geometric Functions */
> >  #include <clc/geometric/cross.h>
> > +#include <clc/geometric/distance.h>
> >  #include <clc/geometric/dot.h>
> >  #include <clc/geometric/length.h>
> >  #include <clc/geometric/normalize.h>
> > diff --git a/generic/include/clc/geometric/distance.h
> > b/generic/include/clc/geometric/distance.h
> > index 3e91332..c7aa6a1 100644
> > --- a/generic/include/clc/geometric/distance.h
> > +++ b/generic/include/clc/geometric/distance.h
> > @@ -1,2 +1,24 @@
> > +/*
> > + * Copyright (c) 2014,2015 Advanced Micro Devices, Inc.
> > + *
> > + * Permission is hereby granted, free of charge, to any person obtaining
> > a copy
> > + * of this software and associated documentation files (the "Software"),
> > to deal
> > + * in the Software without restriction, including without limitation the
> > rights
> > + * to use, copy, modify, merge, publish, distribute, sublicense, and/or
> > sell
> > + * copies of the Software, and to permit persons to whom the Software is
> > + * furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice shall be
> > included in
> > + * all copies or substantial portions of the Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> > EXPRESS OR
> > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > MERCHANTABILITY,
> > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT
> > SHALL THE
> > + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> > ARISING FROM,
> > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> > IN
> > + * THE SOFTWARE.
> > + */
> > +
> >  #define __CLC_BODY <clc/geometric/distance.inc>
> >  #include <clc/geometric/floatn.inc>
> > diff --git a/generic/include/clc/geometric/distance.inc
> > b/generic/include/clc/geometric/distance.inc
> > new file mode 100644
> > index 0000000..6babbdc
> > --- /dev/null
> > +++ b/generic/include/clc/geometric/distance.inc
> > @@ -0,0 +1,23 @@
> > +/*
> > + * Copyright (c) 2014,2015 Advanced Micro Devices, Inc.
> > + *
> > + * Permission is hereby granted, free of charge, to any person obtaining
> > a copy
> > + * of this software and associated documentation files (the "Software"),
> > to deal
> > + * in the Software without restriction, including without limitation the
> > rights
> > + * to use, copy, modify, merge, publish, distribute, sublicense, and/or
> > sell
> > + * copies of the Software, and to permit persons to whom the Software is
> > + * furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice shall be
> > included in
> > + * all copies or substantial portions of the Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> > EXPRESS OR
> > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > MERCHANTABILITY,
> > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT
> > SHALL THE
> > + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> > ARISING FROM,
> > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> > IN
> > + * THE SOFTWARE.
> > + */
> > +
> > +_CLC_OVERLOAD _CLC_DECL __CLC_FLOAT distance(__CLC_FLOATN p0,
> > __CLC_FLOATN p1);
> > diff --git a/generic/lib/SOURCES b/generic/lib/SOURCES
> > index ac3d870..9ac2c05 100644
> > --- a/generic/lib/SOURCES
> > +++ b/generic/lib/SOURCES
> > @@ -33,6 +33,7 @@ common/sign.cl
> >  common/smoothstep.cl
> >  common/step.cl
> >  geometric/cross.cl
> > +geometric/distance.cl
> >  geometric/dot.cl
> >  geometric/length.cl
> >  geometric/normalize.cl
> > diff --git a/generic/lib/geometric/distance.cl b/generic/lib/geometric/
> > distance.cl
> > new file mode 100644
> > index 0000000..4a5b8e2
> > --- /dev/null
> > +++ b/generic/lib/geometric/distance.cl
> > @@ -0,0 +1,30 @@
> > +/*
> > + * Copyright (c) 2014,2015 Advanced Micro Devices, Inc.
> > + *
> > + * Permission is hereby granted, free of charge, to any person obtaining
> > a copy
> > + * of this software and associated documentation files (the "Software"),
> > to deal
> > + * in the Software without restriction, including without limitation the
> > rights
> > + * to use, copy, modify, merge, publish, distribute, sublicense, and/or
> > sell
> > + * copies of the Software, and to permit persons to whom the Software is
> > + * furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice shall be
> > included in
> > + * all copies or substantial portions of the Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> > EXPRESS OR
> > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > MERCHANTABILITY,
> > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT
> > SHALL THE
> > + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> > ARISING FROM,
> > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> > IN
> > + * THE SOFTWARE.
> > + */
> > +
> > +#include <clc/clc.h>
> > +
> > +#ifdef cl_khr_fp64
> > +#pragma OPENCL EXTENSION cl_khr_fp64 : enable
> > +#endif
> > +
> > +#define __CLC_BODY <distance.inc>
> > +#include <clc/geometric/floatn.inc>
> > diff --git a/generic/lib/geometric/distance.inc
> > b/generic/lib/geometric/distance.inc
> > new file mode 100644
> > index 0000000..193665e
> > --- /dev/null
> > +++ b/generic/lib/geometric/distance.inc
> > @@ -0,0 +1,25 @@
> > +/*
> > + * Copyright (c) 2014,2015 Advanced Micro Devices, Inc.
> > + *
> > + * Permission is hereby granted, free of charge, to any person obtaining
> > a copy
> > + * of this software and associated documentation files (the "Software"),
> > to deal
> > + * in the Software without restriction, including without limitation the
> > rights
> > + * to use, copy, modify, merge, publish, distribute, sublicense, and/or
> > sell
> > + * copies of the Software, and to permit persons to whom the Software is
> > + * furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice shall be
> > included in
> > + * all copies or substantial portions of the Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> > EXPRESS OR
> > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > MERCHANTABILITY,
> > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT
> > SHALL THE
> > + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> > ARISING FROM,
> > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> > IN
> > + * THE SOFTWARE.
> > + */
> > +
> > +_CLC_OVERLOAD _CLC_DEF __CLC_FLOAT distance(__CLC_FLOATN p0, __CLC_FLOATN
> > p1) {
> > +  return length(p0 - p1);
> > +}
> > --
> > 2.0.4
> >
> >
> > _______________________________________________
> > Libclc-dev mailing list
> > Libclc-dev at pcc.me.uk
> > http://www.pcc.me.uk/cgi-bin/mailman/listinfo/libclc-dev
> >

> _______________________________________________
> Libclc-dev mailing list
> Libclc-dev at pcc.me.uk
> http://www.pcc.me.uk/cgi-bin/mailman/listinfo/libclc-dev

More information about the Libclc-dev mailing list