[PATCH] D14094: Add multikey_qsort to STLExtras.h

Jim Grosbach via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 26 14:43:42 PDT 2015


grosbach added a comment.

Not 100% sure it's the best place, either; it's just what came to mind since the function we're replacing the invocation of is also there.


================
Comment at: include/llvm/ADT/STLExtras.h:376
@@ +375,3 @@
+    ++Pos;
+    goto tailcall;
+  }
----------------
Does the compiler not auto-recognize the tail call pattern here? If not, it seems like it should.


http://reviews.llvm.org/D14094





More information about the llvm-commits mailing list