[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:29:26 PDT 2017


vpykhtin added a comment.

You can do the reset with move - this would not require copy.

If you make advance moving its own iterator then it can skip debugs itself.

Making advance checking end pointer wouldn't increase the number of end checks, as you check it on every iteration of advance loop. You can return bool from advance and break the loop on it. Anyway such checks are cheap.


Repository:
  rL LLVM

https://reviews.llvm.org/D33105





More information about the llvm-commits mailing list