[PATCH] D20912: [Polly][GSoC 2016]Update ScopBuilder's memory management

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 05:47:39 PDT 2016


jdoerfert added inline comments.

================
Comment at: lib/Analysis/ScopInfo.cpp:4901
@@ +4900,3 @@
+  ScopBuilder *SB;
+  SB = new ScopBuilder(R, AC, AA, DL, DT, LI, SD, SE);
+  if (SB) {
----------------
Why allocate the ScopBuilder on the heap anyway?
Doesn't this work:

```
 ScopBuilder SB(R, AC, AA, DL, DT, LI, SD, SE);
```



http://reviews.llvm.org/D20912





More information about the llvm-commits mailing list