[fix][refactor] Deprecate OCLWriteNode and PTXWriteNode and fix store…

[gigiblender]

Description

This patch fixes an issue with storing bytes into arrays.
Additionally, it removes the OCLWriteNode and PTXWriteNode classes since they're no longer needed.

Problem description

Running the uk.ac.manchester.tornado.unittests.arrays.TestArrays#testInitByteArray unit test before the patch would generate the following stores:

  ul_128  =  ul_0 + 151L;
  *((__global int *) ul_128)  =  21;

The issue is that the data type is byte and we store it at a 4 byte address instead of 1 byte.

After the patch, the store becomes:

  ul_128  =  ul_0 + 151L;
  *((__global char *) ul_128)  =  21;

Backend/s tested

• OpenCL
• PTX

OS tested

• Linux
• OSx
• Windows

Did you check on FPGAs?

If possible, check your changes on FPGAs.

• Yes
• No

How to test the new patch?

Run the unit tests. I've added the test below for this:

tornado-test.py -J"-Dtornado.opencl.priority=10" --verbose --printKernel uk.ac.manchester.tornado.unittests.arrays.TestArrays#testInitByteArray

[jjfumero]

LGTM. Thank you Florin.