[PATCH] D33105: [AMDGPU] Turn register pressure estimation into forward tracker

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 10:19:28 PDT 2017


vpykhtin added a comment.

I really don't understand it all. Why not just have reset(MachineInstr *, LiveRegSet ) and do a reset on the next BB with liveset from previous BB? It looks like your code does it. Another question: your reset does skip debugs, but the caller doesn't know about it and can supply unskipped iterator to the next advance.


Repository:
  rL LLVM

https://reviews.llvm.org/D33105





More information about the llvm-commits mailing list