[PATCH] D33688: [Polly] Heap allocation for new arrays

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 7 03:56:03 PDT 2017


Meinersbur added a comment.

Thanks for explaining it (again).

You can mark your patch with "WIP" (Work in Progress) if you do not intend to have it committed as-is.

For being commit-ready, I think the test case should be smaller (as already mentioned by Andres), and the malloc'ed memory must also be released.


https://reviews.llvm.org/D33688





More information about the llvm-commits mailing list