[PATCH] D103144: [X86][Costmodel] Load/store v2i16 VF=2 interleaving costs

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 30 01:54:42 PDT 2021


RKSimon added a comment.

The LGTM was for this patch.

The algorithm makes sense, although in the script I'm slowly writing for cost-tables vs llvm-mca comparisons, I'm always using ceil() for fractional values, this was mainly to keep with the 'worst case' approach of the cost tables.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103144



More information about the llvm-commits mailing list