[all-commits] [llvm/llvm-project] 7f1dcc: [InstCombine] Skip scalable vectors in combineLoad...

[Diana via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 7f1dcc8952e6a1a4ba918159ab86bd787d584930
      https://github.com/llvm/llvm-project/commit/7f1dcc8952e6a1a4ba918159ab86bd787d584930
  Author: Diana Picus <diana.picus at linaro.org>
  Date:   2019-11-12 (Tue, 12 Nov 2019)

  Changed paths:
    M llvm/lib/Analysis/Loads.cpp
    M llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
    M llvm/test/Transforms/InstCombine/load.ll

  Log Message:
  -----------
  [InstCombine] Skip scalable vectors in combineLoadToOperationType

Don't try to canonicalize loads to scalable vector types to loads
of integers.

This removes one assertion when trying to use a TypeSize as a parameter
to DataLayout::isLegalInteger. It does not handle the second part of the
function (which looks at bitcasts).

This patch also contains a NFC fix for Load Analysis, where a variable
initialization that would cause the same assertion is moved closer to
its use. This allows us to run the new test for InstCombine without
having to teach LocationSize to play nicely with scalable vectors.

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