[PATCH] D35754: [ForwardOpTree] Introduce the -polly-optree pass.

[Siddharth via Phabricator via llvm-commits]

bollu added inline comments.


================
Comment at: polly/trunk/lib/Transform/ForwardOpTree.cpp:108
+  /// @return When DoIt==true, return whether the operand tree can be forwarded.
+  ///         When DoIt==false, return FD_DidForward.
+  ForwardingDecision canForwardTree(ScopStmt *TargetStmt, Value *UseVal,
----------------
@Meinersbur - Is the comment incorrect? 

When `DoIt == true`, does it not return `FD_DidForward / FD_DidNotFowrard`

And when `DoIt == false`, it returns `FD_CanForward` ?

I am basing this off of this piece of code:

```lang=cpp,name=snippet-with-do-it-returns.cpp
      if (DoIt)
        return FD_DidForward;
      return FD_CanForward;
```


Repository:
  rL LLVM

https://reviews.llvm.org/D35754