[PATCH] D105476: [lld/mac] Rename internalize.ll test to lto-internalize.ll
Jez Ng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 6 08:20:59 PDT 2021
int3 added a comment.
I intentionally omitted the `lto-` prefix from this (and several other tests) because it seemed redundant -- practically all our `.ll` tests are LTO tests. I guess that's not strictly true any more though given things like `lc-linker-option.ll`. There are also some LTO tests that do have the `lto-` prefix, so we're not consistent about it either. I guess it doesn't really matter which scheme we pick as long as we're consistent. So... if we are going to add said prefix, can we do it for all the other LTO tests as well?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105476/new/
https://reviews.llvm.org/D105476
More information about the llvm-commits
mailing list