[test-suite] Bitcode simd tests: align memory to 128.

[asbirlea]

Summary

Patch preparing the addition of more tests.
Some require 128 alignment, so update driver to allocate aligned memory.

[asbirlea]

I tested the change on Linux and OSX.
Suggestions on if there are other architecture specific changes I missed are welcome.

[mehdi_amini]

Bitcode/simd_ops/simd_ops.cpp

19 ↗

(On Diff #65153)

Will this work on Windows?

[asbirlea]

Bitcode/simd_ops/simd_ops.cpp

19 ↗

(On Diff #65153)

I looked it up and I think I need to use "_aligned_malloc()".
Looking for ways to test it, I don't have one handy.

[echristo]

Bitcode/simd_ops/simd_ops.cpp

19 ↗

(On Diff #65153)

For portability there's nothing specific that will work across everything until C++17. _aligned_malloc is the right way for windows.

Perhaps abstract it out into an aligned allocation function. Also, if you wouldn't mind commenting this routine as well as to why we're allocating aligned memory :)

[asbirlea]

Update for Windows. Merge options for OSX/Linux.

[asbirlea]

Add comment.

[echristo]

One inline comment :)

Bitcode/simd_ops/simd_ops.cpp

6 ↗

(On Diff #65560)

Comment what's going on here and why we're allocating memory if you wouldn't mind? It might be nicer for this to be a function rather than a macro? Just call it alloc_aligned or something and have the functionality for all 3 platforms in there?

[asbirlea]

Address comment.

[asbirlea]

Bitcode/simd_ops/simd_ops.cpp

6 ↗

(On Diff #65805)

Moved the comment from below here and created a function instead of the macro.
There are just two cases, since I can unify OSX and Linux with the posix method.

[echristo]

good enough. Thanks!

LGTM.

-eric

test-suite/trunk/Bitcode/simd_ops/simd_ops.cpp

 #include "filter_test_op.h"
 
+#ifdef _MSC_VER
+#include <errno.h>
+#include <malloc.h>
+#endif
+
+// Allocate aligned memory, per recent requirement by the
+// Halide tests updated upstream.
+int allocate_aligned(void **mem, size_t alignment, size_t size) {
+#ifdef _MSC_VER
+  *p = _aligned_malloc(size, alignment);
+  return (*p) ? 0 : errno;
+#else
+  return posix_memalign(mem, alignment, size);
+#endif
+}
+
 template<typename T>
 T rand_value() {
     return (T)(rand() * 0.125) - 100;
 }
 
 // Even on android, we want errors to stdout
 extern "C" void halide_print(void *, const char *msg) {
     printf("%s\n", msg);
 }
 
 template<typename T>
 buffer_t make_buffer(int w, int h) {
-    T *mem = new T[w*h];
+    T *mem;
+    int err = allocate_aligned((void **)&mem, 128, w * h * sizeof(T));
+
     buffer_t buf = {0};
     buf.host = (uint8_t *)mem;
     buf.extent[0] = w;
     buf.extent[1] = h;
     buf.elem_size = sizeof(T);
     buf.stride[0] = 1;
     buf.stride[1] = w;