[PATCH] D139000: [AMDGPU] Remove function with incompatible features

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp:1077
+
+  if (RemoveIncompatibleFunctions)
+    addPass(
----------------
Joe_Nash wrote:
> For the sake of compile time, can we avoid running this pass on O2 and O3? We know dead code elimination will remove the functions anyway.
We don't know that. DCE will only remove a function if the compiler can easily prove that it is never called.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139000