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

[Bonfante Nicolas via Phabricator via llvm-commits]

niosega added inline comments.


================
Comment at: lib/Exchange/JSONExporter.cpp:722
+        auto CurrentSAI =
+            S.getArrayInfoByName(Arrays[ArrayIdx]["name"].asCString());
+        CurrentSAI->setIsOnHeap(true);
----------------
I need to have a pointer to the newly created ScopArrayInfo to call the setter.
But the method createScopArrayInfo returns only a const SAI *. The solution 
I found is to query the Scop to obtain the SAI by name.

An alternative would be to pass a parameter to createScopArrayInfo then to 
getOrCreateScopArrayInfo that represents the value that isOnHeap must take.


https://reviews.llvm.org/D33688