[PATCH] D91583: [LTO] Prevent devirtualization for symbols exported to dynamic linker
Teresa Johnson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 12 15:52:31 PST 2021
tejohnson marked 3 inline comments as done.
tejohnson added inline comments.
================
Comment at: lld/test/ELF/lto/devirt_vcall_vis_export_dynamic.ll:43
+; RUN: -mllvm -pass-remarks=. \
+; RUN: --export-dynamic 2>&1 | FileCheck %s --implicit-check-not single-impl --allow-empty
; RUN: llvm-dis %t.o.4.opt.bc -o - | FileCheck %s --check-prefix=CHECK-NODEVIRT-IR
----------------
MaskRay wrote:
> `%s` -> `/dev/null`
Fixed here and elsewhere - but not sure why it matters?
================
Comment at: lld/test/ELF/lto/devirt_vcall_vis_public.ll:8
; RUN: ld.lld %t2.o -o %t3 -save-temps --lto-whole-program-visibility \
-; RUN: -mllvm -pass-remarks=. --export-dynamic 2>&1 | FileCheck %s --check-prefix=REMARK
+; RUN: -mllvm -pass-remarks=. \
+; RUN: 2>&1 | FileCheck %s --check-prefix=REMARK
----------------
MaskRay wrote:
> `s/\t/ /`
Done here and elsewhere
================
Comment at: lld/test/ELF/lto/devirt_vcall_vis_public.ll:9
+; RUN: -mllvm -pass-remarks=. \
+; RUN: 2>&1 | FileCheck %s --check-prefix=REMARK
; RUN: llvm-dis %t2.o.4.opt.bc -o - | FileCheck %s --check-prefix=CHECK-IR
----------------
MaskRay wrote:
> `s/\t/ /`
>
> The two lines can be joined.
ditto
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91583/new/
https://reviews.llvm.org/D91583
More information about the llvm-commits
mailing list