[PATCH] D109858: [GlobalISel][AMDGPU] Add dead code elimination clean up after legalization.

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 17 01:36:18 PDT 2021


foad added a comment.

What is the compile time impact of this vs D109154 <https://reviews.llvm.org/D109154>? I'm assuming they are both equally effective at cleaning up the dead code.

If we're going to go with this approach, should we clean up legalizeMachineFunction by ripping out all the bottom-up RPO order stuff, and the on-the-fly (presumably expensive) isTriviallyDead calls?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109858



More information about the llvm-commits mailing list