[PATCH] D34358: LLD: Move the rest of the Mips into Arch/Mips.cpp

Konstantin Zhuravlyov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 19 11:50:58 PDT 2017


kzhuravl added a comment.

In https://reviews.llvm.org/D34358#784407, @ruiu wrote:

> I think we shouldn't do this. It might make sense to move this file under `Arch`, but merging it with `Arch/Mips.cpp` decreases readability as the file now contains totally unrelated functions.


I had a similar feeling but was not sure. Do you have a name suggestion if it is moved under Arch?


https://reviews.llvm.org/D34358





More information about the llvm-commits mailing list