[PATCH] D63572: [Polly][NFC][ScopBuilder] Move addRecordedAssumption to ScopBuilder

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a comment.

[suggestion] `RecordedAssumptions` itself is only used during SCoP constructions (since it's empty once the SCoP is full constructed).  It could be removed entirely into ScopBuilder, maybe in a future patch?



================
Comment at: polly/lib/Analysis/ScopBuilder.cpp:391
+
+    //    Assumption &AS = scop->popRecordedAssumption();
+    if (!AS.BB) {
----------------
Please remove commented-out code entirely.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63572/new/

https://reviews.llvm.org/D63572