[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