[PATCH] D72709: [codegen,amdgpu] Enhance MIR DIE and re-arrange it for AMDGPU.

Michael Liao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 13:19:27 PST 2020


hliao added a comment.

In D72709#1820182 <https://reviews.llvm.org/D72709#1820182>, @rampitec wrote:

> You have skipped the dead MO, but was pass reordering really necessary? It seems we have higher register pressure with this change.


The issue is that DIE on MIR assumes SSA form MIR. It's incorrect to run it after DeSSA. It results in invalid IR. I thought we should fix this first and look into the fix for the higher register pressure issue here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72709





More information about the llvm-commits mailing list