[all-commits] [llvm/llvm-project] 7e95a5: Remove scalable vector assert from InnerLoopVector...

david-arm via All-commits all-commits at lists.llvm.org
Wed May 19 05:34:15 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7e95a563c89dc158217e2032b716e5be77677846
      https://github.com/llvm/llvm-project/commit/7e95a563c89dc158217e2032b716e5be77677846
  Author: David Sherwood <david.sherwood at arm.com>
  Date:   2021-05-19 (Wed, 19 May 2021)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

  Log Message:
  -----------
  Remove scalable vector assert from InnerLoopVectorizer::setDebugLocFromInst

In InnerLoopVectorizer::setDebugLocFromInst we were previously
asserting that the VF is not scalable. This is because we want to
use the number of elements to create a duplication factor for the
debug profiling data. However, for scalable vectors we only know the
minimum number of elements. I've simply removed the assert for now
and added a FIXME saying that we assume vscale is always 1. When
vscale is not 1 it just means that the profiling data isn't as
accurate, but shouldn't cause any functional problems.




More information about the All-commits mailing list