Remove param pi nodes

[andrii0lomakin]

That is the fix for the first problem described in the issue: #521 .

Namely primitive parameters can not be passed in passed in anonymous lambda expressions.

Backend/s tested

Mark the backends affected by this PR.

• OpenCL
• PTX
• SPIRV

OS tested

Mark the OS where this PR is tested.

• Linux
• OSx
• Windows

Did you check on FPGAs?

If it is applicable, check your changes on FPGAs.

• Yes
• No

How to test the new patch?

// EDIT: Juan

tornado-test -V uk.ac.manchester.tornado.unittests.functional.TestLambdas#testParameterUnboxing

---

[andrii0lomakin]

Once this PR is approved, I will provide another one that fixes issues with code like this one:

static void addVectorToVectorKernel(FloatArray first, int firstOffset, FloatArray second, int secondOffset,
                                        FloatArray result, int resultOffset, KernelContext kernelContext) {

       var i = kernelContext.globalIdx;
        result.set(resultOffset + i, first.get(firstOffset + i) + second.get(secondOffset + i));
    }

[jjfumero]

LGTM, just some minor comments.

[jjfumero]

Thanks @andrii0lomakin for your contribution. I also checked this with all backends and it works.

Question regarding your current and future contribution/s: Are you contributing as part of Jetbrains, or an individual/external contributor? The reason I am asking is that, we have another CLA for Corporate Contributions. If so, I redirect this to @kotselidis that can provide more details.

[andrii0lomakin]

@jjfumero Us for the moment I am providing individual contributions. Once situation changes I will inform you at once :-)

[jjfumero]

Ok, thanks for the clarification.

[mikepapadim]

Thanks for that, overall it looks good to me.

I need some time to examine the graph after some optimization phases.

[stratika]

LGTM, just one minor comment in tornado-test for a double entry.

[stratika]

Suggested change

TestEntry("uk.ac.manchester.tornado.unittests.functional.TestLambdas"),

[stratika]

These unit-tests are already registered in line 54.

[jjfumero]

We have two classes names TestLambdas. Let's keep them for this PR and we will unify them later.

[stratika]

correct, if you check line 54 it already has the functional.TestLambdas

[jjfumero]

Ok, you are right.