[PATCH] D92060: [lld] Add --no-lto-whole-program-visibility

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.


================
Comment at: lld/test/ELF/lto/devirt_vcall_vis_public.ll:34
 ; RUN: llvm-dis %t2.o.4.opt.bc -o - | FileCheck %s --check-prefix=CHECK-NODEVIRT-IR
+; Try again with explicit --no-lto-whole-program-visibility.
+; RUN: ld.lld %t2.o -o %t3 -save-temps --no-lto-whole-program-visibility \
----------------
MaskRay wrote:
> Could you also add some comments that the existing --export-dynamic is to retain the symbols?
Will do.


================
Comment at: lld/test/ELF/lto/devirt_vcall_vis_public.ll:39
+; Ensure --no-lto-whole-program-visibility overrides explicit --lto-whole-program-visibility.
+; RUN: ld.lld %t2.o -o %t3 -save-temps --lto-whole-program-visibility --no-lto-whole-program-visibility \
+; RUN: 	 -mllvm -pass-remarks=. --export-dynamic 2>&1 | FileCheck %s --implicit-check-not single-impl --allow-empty
----------------
MaskRay wrote:
> A `--lto-whole-program-visibility --no-lto-whole-program-visibility` RUN list is sufficient, no need for the previous RUN line.
I assume you mean the variant I added with just the explicit --no-lto-whole-program-visibility, and not --lto-whole-program-visibility. Yeah that seems redundant, will remove.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92060