search for: clover

Hello, I am the developer of Clover, and so much activity about OpenCL these days is really exciting. Here is my point of view, mainly on Clover and how the projects could use each other. Clover is made in a way that allow a certain level of modularity. Although POCL would be very difficult to merge into Clover (or Clover into POCL)...

I am trying to analyse some data and was given R code to do this with but there seem to be errors in the code. My level of knowledge is improving but still limited. The details are; Data on clover lines; Lines.txt attached. Comma seperations Code: options(digits=3) clover <- read.table("Lines.txt",header=T,sep=",") vnames <- names(clover);nv <- length(vnames) flevels <- levels(clover$Line) par(oma=c(0,6,4,6),mfrow=c(2,1),mar=c(4,4,1,0)) clover$Y <- cl...

After pipe_grid_info.indirect was introduced, clover was not modified to set it causing it to pass uninitialized memory for it to launch_grid. This commit fixes this by zero-ing the entire pipe_grid_info struct when declaring it, to avoid similar problems popping-up in the future. Signed-off-by: Hans de Goede <hdegoede at redhat.com> --- src...

Hi, On 14-03-16 14:01, Samuel Pitoiset wrote: > > > On 03/14/2016 01:50 PM, Hans de Goede wrote: >> After pipe_grid_info.indirect was introduced, clover was not modified >> to set it causing it to pass uninitialized memory for it to launch_grid. >> >> This commit fixes this by zero-ing the entire pipe_grid_info struct when >> declaring it, to avoid similar problems popping-up in the future. >> >> Signed-off-by: H...

After pipe_grid_info.indirect was introduced, clover was not modified to set it causing it to pass uninitialized memory for it to launch_grid. This commit fixes this by zero-ing the entire pipe_grid_info struct when declaring it, to avoid similar problems popping-up in the future. Cc: "11.2" <mesa-stable at lists.freedesktop.org> Si...

After pipe_grid_info.indirect was introduced, clover was not modified to set it causing it to pass uninitialized memory for it to launch_grid. This commit fixes this by zero-ing the entire pipe_grid_info struct when declaring it, to avoid similar problems popping-up in the future. Cc: "11.2" <mesa-stable at lists.freedesktop.org> Si...

...h is all right for SIMD architectures but might not be > so for multi-issue processors. I don't know what you mean with "speculation" here, but other than that you are right: for best performance, we explicitly target machines with SIMD instruction sets. >> In contrast to Clover and pocl, we aimed at maximum performance before >> full support of the API (which simply requires more manpower than one >> PhD student). > > That is wrong, at least for pocl. We do not (by far) support the whole > API, the main new point on pocl is the LLVM passes to statical...

...uel Pitoiset wrote: > > > On 03/14/2016 02:26 PM, Hans de Goede wrote: >> Hi, >> >> On 14-03-16 14:01, Samuel Pitoiset wrote: >>> >>> >>> On 03/14/2016 01:50 PM, Hans de Goede wrote: >>>> After pipe_grid_info.indirect was introduced, clover was not modified >>>> to set it causing it to pass uninitialized memory for it to >>>> launch_grid. >>>> >>>> This commit fixes this by zero-ing the entire pipe_grid_info struct >>>> when >>>> declaring it, to avoid similar pro...

On 03/14/2016 01:50 PM, Hans de Goede wrote: > After pipe_grid_info.indirect was introduced, clover was not modified > to set it causing it to pass uninitialized memory for it to launch_grid. > > This commit fixes this by zero-ing the entire pipe_grid_info struct when > declaring it, to avoid similar problems popping-up in the future. > > Signed-off-by: Hans de Goede <hdegoed...

On 03/14/2016 02:26 PM, Hans de Goede wrote: > Hi, > > On 14-03-16 14:01, Samuel Pitoiset wrote: >> >> >> On 03/14/2016 01:50 PM, Hans de Goede wrote: >>> After pipe_grid_info.indirect was introduced, clover was not modified >>> to set it causing it to pass uninitialized memory for it to launch_grid. >>> >>> This commit fixes this by zero-ing the entire pipe_grid_info struct when >>> declaring it, to avoid similar problems popping-up in the future. >>> >&...

...t;> On 03/14/2016 02:26 PM, Hans de Goede wrote: >>> Hi, >>> >>> On 14-03-16 14:01, Samuel Pitoiset wrote: >>>> >>>> >>>> On 03/14/2016 01:50 PM, Hans de Goede wrote: >>>>> After pipe_grid_info.indirect was introduced, clover was not modified >>>>> to set it causing it to pass uninitialized memory for it to >>>>> launch_grid. >>>>> >>>>> This commit fixes this by zero-ing the entire pipe_grid_info struct >>>>> when >>>>> declaring...

Add support for clover / OpenCL kernel input parameters. Signed-off-by: Hans de Goede <hdegoede at redhat.com> --- .../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cp...

On 03/10/2016 04:23 PM, Ilia Mirkin wrote: > On Thu, Mar 10, 2016 at 10:14 AM, Hans de Goede <hdegoede at redhat.com> wrote: >> Add support for clover / OpenCL kernel input parameters. >> >> Signed-off-by: Hans de Goede <hdegoede at redhat.com> >> --- >> .../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 18 +++++++++++++++--- >> 1 file changed, 15 insertions(+), 3 deletions(-) >> >> diff -...

Add support for clover / OpenCL kernel input parameters. Signed-off-by: Hans de Goede <hdegoede at redhat.com> Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu> --- Changes in v2: -s/local/private/ -Add: Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu> --- .../drivers/nouveau/codegen/nv50_ir_from_t...

...oiset > <samuel.pitoiset at gmail.com> wrote: >> >> >> On 03/10/2016 04:23 PM, Ilia Mirkin wrote: >>> >>> On Thu, Mar 10, 2016 at 10:14 AM, Hans de Goede <hdegoede at redhat.com> >>> wrote: >>>> >>>> Add support for clover / OpenCL kernel input parameters. >>>> >>>> Signed-off-by: Hans de Goede <hdegoede at redhat.com> >>>> --- >>>> .../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 18 >>>> +++++++++++++++--- >>>> 1 file changed...

...03/10/2016 05:03 PM, Pierre Moreau wrote: > On 04:27 PM - Mar 10 2016, Samuel Pitoiset wrote: >> >> >> On 03/10/2016 04:23 PM, Ilia Mirkin wrote: >>> On Thu, Mar 10, 2016 at 10:14 AM, Hans de Goede <hdegoede at redhat.com> wrote: >>>> Add support for clover / OpenCL kernel input parameters. >>>> >>>> Signed-off-by: Hans de Goede <hdegoede at redhat.com> >>>> --- >>>> .../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 18 +++++++++++++++--- >>>> 1 file changed, 15 insertions(+),...

On Thu, Mar 10, 2016 at 10:14 AM, Hans de Goede <hdegoede at redhat.com> wrote: > Add support for clover / OpenCL kernel input parameters. > > Signed-off-by: Hans de Goede <hdegoede at redhat.com> > --- > .../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 18 +++++++++++++++--- > 1 file changed, 15 insertions(+), 3 deletions(-) > > diff --git a/src/gallium/drivers/nou...

On Thu, Mar 10, 2016 at 11:03 AM, Pierre Moreau <pierre.morrow at free.fr> wrote: > You might want to increment the address by at least > `info->prop.cp.inputOffset`, and if inputs still end up in shared on Tesla, There's a cp.sharedOffset just for that :) However it doesn't appear to get set anywhere...

.../llvm-target-spirv/, stripped off of everything not related to SPIR-V and updated to work with llvm HEAD. (Some of the commits were lost on the way, so we are in the process of redoing the stripping and updating.) One user of that external tool will be Mesa [1] (for its OpenCL implementation named clover) for 1. enabling clCreateProgramWithIL(KHR)? on AMD cards through clover, by translating SPIR-V to LLVM IR; (patches under review [2]) 2. allowing additional drivers (nouveau, freedreno, etnativ) to be supported by clover, by compiling OpenCL C (via LLVM IR) to SPIR-V (and then translating...

...Mar 10, 2016 at 10:27 AM, Samuel Pitoiset <samuel.pitoiset at gmail.com> wrote: > > > On 03/10/2016 04:23 PM, Ilia Mirkin wrote: >> >> On Thu, Mar 10, 2016 at 10:14 AM, Hans de Goede <hdegoede at redhat.com> >> wrote: >>> >>> Add support for clover / OpenCL kernel input parameters. >>> >>> Signed-off-by: Hans de Goede <hdegoede at redhat.com> >>> --- >>> .../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 18 >>> +++++++++++++++--- >>> 1 file changed, 15 insertions(+), 3 dele...