[PATCH] D105104: [AMDGPU] PHI node cost should not be counted for the size and latency.

Daniil Fukalov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 29 07:26:15 PDT 2021


dfukalov added inline comments.


================
Comment at: llvm/test/Analysis/CostModel/AMDGPU/control-flow.ll:7
 ; SPEED: estimated cost of 4 for instruction: br label
-; SPEED: estimated cost of 1 for instruction: %phi = phi i32 [
 ; SPEED: estimated cost of 10 for instruction: ret void
 ; SIZE: estimated cost of 5 for instruction: br i1
----------------
Please update the sizes reported after your change instead of just removing the test lines here and below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105104



More information about the llvm-commits mailing list