[PATCH] D87889: [SVE] Make EVT::getScalarSizeInBits and others consistent with Type::getScalarSizeInBits

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 21 00:03:10 PDT 2020


david-arm added a comment.

Hi @efriedma, so it turns out it's over 800, not 1000. :) My initial guess was a bit too high, but here are the counts I found:

grep -R getScalarValueSizeInBits llvm/ | wc -l
149

grep -R getScalarSizeInBits llvm/lib/CodeGen/SelectionDAG/ | wc -l
214

grep -R getScalarSizeInBits llvm/lib/Target/ | wc -l
456

Of course, a small percentage of calls to getScalarSizeInBits could be to the existing IR Type class member function, but having looked through the results I think are most are for MVTs and EVTs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87889



More information about the llvm-commits mailing list