[PATCH] D135042: [opt] Don't initialize legacy instrumentation passes

Fri Oct 7 14:24:08 PDT 2022

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

LGTM -- I assume you plan to follow up on this by actually dropping the remaining legacy instrumentation passes? Most of them have already been removed.

================
Comment at: llvm/test/Instrumentation/DataFlowSanitizer/args-unreachable-bb.ll:1
-; RUN: opt < %s -dfsan -verify -S | FileCheck %s
+; RUN: opt < %s -passes=dfsan,verify -S | FileCheck %s
 target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128-n8:16:32:64-S128"
----------------
Can drop the `verify` while here.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135042