[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