[PATCH] D21866: [ADT] Add a new data structure for managing a priority worklist where re-insertion of entries into the worklist moves them to the end.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 19:38:39 PDT 2016


chandlerc added a comment.

Thanks all, submitting. Let me know if there are further suggestions around this.


================
Comment at: include/llvm/ADT/PriorityWorklist.h:178
@@ +177,3 @@
+  ///
+  /// This predicate wraps a predicate suitable for use with std::remove_if to
+  /// call M.erase(x) on each element which is slated for removal. This just
----------------
sanjoy wrote:
> Given that there are tests, I'm fine with this.
There was actually a bug here... I've improved the test and fixed the bug. =]


http://reviews.llvm.org/D21866





More information about the llvm-commits mailing list