[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