Rewrite to better test the improper behavior

Signed-off-by: James Brodman <[email protected]>

Author: jbrodman

sycl/include/CL/sycl/detail/aligned_allocator.hpp

@@ -53,7 +53,7 @@ template <typename T> class aligned_allocator {
   pointer allocate(size_t Size) {
     size_t NumBytes = Size * sizeof(value_type);
     NumBytes = ((NumBytes - 1) | (MAlignment - 1)) + 1;
-    if (0 == NumBytes)
+    if (Size > NumBytes)
       throw std::bad_alloc();
 
     pointer Result = reinterpret_cast<pointer>(

sycl/test/usm/badmalloc.cpp

@@ -41,18 +41,5 @@ int main(int argc, char *argv[]) {
   p = malloc(-1, q, usm::alloc::unknown);
   assert(p == nullptr);
 
-  // Bad combos for aligned
-  p = aligned_alloc(1, 0, q, usm::alloc::host);
-  assert(p == nullptr);
-
-  p = aligned_alloc(1, 0, q, usm::alloc::device);
-  assert(p == nullptr);
-
-  p = aligned_alloc(1, 0, q, usm::alloc::shared);
-  assert(p == nullptr);
-
-  p = aligned_alloc(1, 0, q, usm::alloc::unknown);
-  assert(p == nullptr);
-
   return 0;
 }