gtbercea marked an inline comment as done.
gtbercea added inline comments.

================
Comment at: libomptarget/deviceRTLs/nvptx/src/supporti.h:191
   PRINT(LD_MEM, "malloc data of size %zu for %s: 0x%llx\n", size, msg, 
P64(ptr));
-  ASSERT(LT_SAFETY, ptr, "failed to allocate %zu bytes for %s\n", size, msg);
   return ptr;
----------------
ABataev wrote:
> Same, do you really need to remove this?
Yes, leaving this in will lead to the whole malloc to be optimized out after 
inlining.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D53141



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to