[all-commits] [llvm/llvm-project] c1909d: [DAGCombiner] Fix crash for the merge stores with ...

chenglin.bi via All-commits all-commits at lists.llvm.org
Fri Oct 14 10:16:56 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c1909d73372e669a44a2aefe82de873c2161cc44
      https://github.com/llvm/llvm-project/commit/c1909d73372e669a44a2aefe82de873c2161cc44
  Author: chenglin.bi <chenglin.bi at linaro.org>
  Date:   2022-10-15 (Sat, 15 Oct 2022)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    A llvm/test/CodeGen/AArch64/pr58350.ll

  Log Message:
  -----------
  [DAGCombiner] Fix crash for the merge stores with different value type

The crash case comes from #58350. It have two stores, one store is type f32 and the other is v1f32.
When we try to merge these two stores on v1f32, the memVT is vector type so the old code will use ISD::EXTRACT_SUBVECTOR for type f32 also then compiler crash.
So this patch insert a build_vector for f32 store to generate v1f32 also when memVT is v1f32.

Reviewed By: RKSimon

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




More information about the All-commits mailing list