[PATCH] D97238: [Attributor] Conditinoally delete fns

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

`Attributor::deleteAfterManifest` also needs to be updated or removed (together with the use).



================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:1662
+  /// Whether to delete functions
+  const bool DeleteFns;
+
----------------
Nit: `.`


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:1218
+        ToBeDeletedFunctions.insert(F);
 }
 
----------------
Early exit out of this function if we don't delete, all what is happening here is not necessary then.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97238