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

[Rui Ueyama via llvm-commits]

On Mon, Apr 9, 2018 at 3:13 PM Mandeep Singh Grang via Phabricator <
reviews at reviews.llvm.org> wrote:

> mgrang added a comment.
>
> In https://reviews.llvm.org/D45138#1062213, @ruiu wrote:
>
> > 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.
>
>
> I had a patch to replace *all* occurrences of std::sort to llvm::sort
> https://reviews.llvm.org/D44363. However, the reviewers felt that the
> patch was getting too big and suggested splitting it into smaller patches
> target-wise, tablegen, MC, etc. Hence these separate patches.
>

If the only reason not to submit them as a single patch is its size, I can
actually approve. As long as it is a mechanical patch, I'm not worried too
much about its size.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180409/70316a04/attachment.html>