[PATCH] D35043: [ADT] Enable reverse iteration for DenseMap

Mandeep Singh Grang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 30 19:18:27 PDT 2017


mgrang added inline comments.


================
Comment at: include/llvm/Support/ReverseIteration.h:10
+template<class T = void*>
+static bool shouldReverseIterate() {
 #if LLVM_ENABLE_REVERSE_ITERATION
----------------
@dblaikie Is this implementation similar to what you had in mind when you said shouldReverseIterate should be a compile time constant w/o the need for an out-of-line definition?


Repository:
  rL LLVM

https://reviews.llvm.org/D35043





More information about the llvm-commits mailing list