[PATCH] D129003: [WPD] Filter out intrinsics inserted from whole-program-vtables

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 1 10:01:32 PDT 2022


aeubanks added a comment.

this doesn't seem very principled, it's seemingly adding checks in random places hoping that passes won't look too much at assumes/type tests
is there anything wrong with making the instrumented build use -fwhole-program-vtables?



================
Comment at: llvm/test/Transforms/InstCombine/wpvt.ll:1
+; RUN: opt < %s -passes=instcombine -S | FileCheck %s
+target triple = "x86_64-grtev4-linux-gnu"
----------------
this test is very noisy and doesn't really show what's going on, could you reduce it a bit, and also maybe run it through `-passes=metarenamer` (and maybe convert to opaque pointers to remove some unnecessary pointer bitcasts via `opt -S -opaque-pointers foo.ll`)


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

https://reviews.llvm.org/D129003



More information about the llvm-commits mailing list