[PATCH] D45138: [MC] Change std::sort to llvm::sort in response to r327219

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 9 13:34:01 PDT 2018


ruiu added a comment.

I'm not familiar with r327219, but if a decision to use randomized sort has been made, why don't you replace all occurrences of std::sort with llvm::sort in a single patch? I don't think that an author of each file don't have to understand this kind of change and approve individually.


Repository:
  rL LLVM

https://reviews.llvm.org/D45138





More information about the llvm-commits mailing list