[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