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

[Bonfante Nicolas via Phabricator via llvm-commits]

niosega updated this revision to Diff 103990.
niosega added a comment.
Herald added a reviewer: bollu.

The changes made in this update are the following :

- Default initialization of isOnHeap to false and implementation of a setter
- Add a test case to JSONImporter to check the size of an imported array
- Use the IntPtrTy of the DataLayout
- Remove useless StringRef in JSONImporter
- Remove useless FreedArray variable in ISLNodeBuilder
- Remove useless -mem2reg and 2>1 in test case
- Call to malloc and free at the correct position  (polly.start and polly.exiting)

For the malloc and free positions, we have made the following modifications :

- Take into account the patch that modify the signature of executeScopConditionnally (now returning both start and end block)
- Set the InsertPoint before AST building to the end of the block so that the split between newly created branch and old branch is made such that the malloc call is in polly.start




https://reviews.llvm.org/D33688

Files:
  include/polly/CodeGen/IslNodeBuilder.h
  include/polly/ScopInfo.h
  lib/Analysis/ScopInfo.cpp
  lib/CodeGen/CodeGeneration.cpp
  lib/CodeGen/IslNodeBuilder.cpp
  lib/Exchange/JSONExporter.cpp
  test/Isl/CodeGen/MemAccess/create_arrays_heap.ll
  test/Isl/CodeGen/MemAccess/create_arrays_heap___%for.cond1.preheader---%for.end18.jscop
  test/Isl/CodeGen/MemAccess/create_arrays_heap___%for.cond1.preheader---%for.end18.jscop.transformed
  test/JSONExporter/ImportArrays/ImportArrays-Negative-size.ll
  test/JSONExporter/ImportArrays/create_arrays_heap___%for.cond1.preheader---%for.end18.jscop.transformed

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33688.103990.patch
Type: text/x-patch
Size: 24293 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170626/905a8b1c/attachment.bin>