[PATCH] D38201: Use a BumpPtrAllocator for Loop objects

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

LGTM!



================
Comment at: include/llvm/Analysis/LoopInfo.h:170
+  /// Return true if this loop is no longer valid.  The only valid use of this
+  /// helper is "assert(L.isInvalid())" or equivalent, since IsValid is set to
+  /// false by the destructor.  In other words, if this accessor returns false,
----------------
The comment says `IsValid`, but the code uses `IsInvalid`...


https://reviews.llvm.org/D38201