[PATCH] D34222: Split Target.cpp into small files.

Sean Silva via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 15 14:36:40 PDT 2017


silvas added a comment.

Thanks, I like this idea. We might want to do something similar for the tests.



================
Comment at: lld/ELF/CMakeLists.txt:10
 add_lld_library(lldELF
+  arch/AArch64.cpp
+  arch/AMDGPU.cpp
----------------
Sorry for such a stupid little comment, but it really should be `Arch` to be consistent with most LLVM directory naming conventions.


https://reviews.llvm.org/D34222





More information about the llvm-commits mailing list