[PATCH] D103129: [llvm-reduce] Don't delete arguments of debug intrinsics

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/tools/llvm-reduce/deltas/ReduceArguments.cpp:47
+static bool shouldRemoveArguments(const Function &F) {
+  return !F.arg_empty() && F.getIntrinsicID() == Intrinsic::not_intrinsic;
+}
----------------
aeubanks wrote:
> `F.isIntrinsic()`
While i could be okay with ignoring actual intriniscs,
i'm not sure why we need to ignore all functions starting with `@llvm.`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103129