[all-commits] [llvm/llvm-project] 2ea549: [InstCombine][SVE] Fix InstCombiner::visitAllocaIn...

huihzhang via All-commits all-commits at lists.llvm.org
Wed Mar 18 20:57:26 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 2ea5495759f61d7bcf9c7e0c35451aaf1b5db6d2
      https://github.com/llvm/llvm-project/commit/2ea5495759f61d7bcf9c7e0c35451aaf1b5db6d2
  Author: Huihui Zhang <huihuiz at quicinc.com>
  Date:   2020-03-18 (Wed, 18 Mar 2020)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
    A llvm/test/Transforms/InstCombine/vscale_alloca.ll

  Log Message:
  -----------
  [InstCombine][SVE] Fix InstCombiner::visitAllocaInst for scalable vector.

Summary:
DataLayout::getTypeAllocSize() return TypeSize. For cases where scalable
property doesn't matter (check for zero-sized alloca), we should explicitly
call getKnownMinSize() to avoid implicit type conversion to uint64_t, which is
invalid for scalable vector type.

Reviewers: sdesmalen, efriedma, spatel, apazos

Reviewed By: efriedma

Subscribers: tschuett, hiraditya, rkruppe, psnobl, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D76386




More information about the All-commits mailing list