[PATCH] D85283: [ARM][CostModel] Implement getCFInstrCost

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 5 06:11:05 PDT 2020


dmgreen added a comment.

Registers are free unless you go over a limit. And I wouldn't expect it to be these individual cost functions that attempted to guess whether it was over that limit.

The thing about multiplying by the vector factor sounds like it might well be an issue. I would guess no-one would have run into that in the past if it was.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85283



More information about the llvm-commits mailing list