[PATCH] D146810: [CodeGen] Compile-time improvement for Machine Late Instructions Cleanup Pass.

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 25 01:42:45 PDT 2023


vpykhtin added a comment.

> On SystemZ, the two versions show practically identical runtimes, so there is no winner. Is there any difference at all on your machine?

Not really :) Something barely measurable like 2.13 and 2.09 seconds on the huge testcase. I'm curious which cases you use to compare to the current, non-map version?

> I am thinking maybe it would be possible to lookup Reg only once by calling removeRedundantDef() with MI and also the KillMI. That way, clearKillsForDef() wouldn't have to duplicate the same lookup.

I need to dig more into what is going on there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146810



More information about the llvm-commits mailing list