[PATCH] D84462: Rename options for basic block sections to be consistent

[Fangrui Song via Phabricator via llvm-commits]

MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

`lld/test/ELF/lto/basic-block-sections.ll` should be placed in another patch. Adding a new test is unrelated to renaming the whole bunch of test files.



================
Comment at: lld/test/ELF/lto/basic-block-sections.ll:5
+; RUN: llvm-readobj -s %t.lto.o | FileCheck --check-prefix=SECNAMES %s
+; RUN: ld.lld %t.o -o %t --lto-basic-block-sections=all --lto-unique-basic-block-section-names --lto-O0 --save-temps
+; RUN: llvm-readobj -s %t.lto.o | FileCheck --check-prefix=SECNAMES-FULL %s
----------------
`--lto-O2` is the default. I wonder if `--lto-O0` is significant. If not, please remove it.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84462/new/

https://reviews.llvm.org/D84462