[all-commits] [llvm/llvm-project] 963784: [GlobalISel] Fix non-pow-2 legalization of s56 sto...

Amara Emerson via All-commits all-commits at lists.llvm.org
Fri Jul 16 13:30:21 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9637848f51af0e770c6d19b6aa020250bf1d0c0f
      https://github.com/llvm/llvm-project/commit/9637848f51af0e770c6d19b6aa020250bf1d0c0f
  Author: Amara Emerson <amara at apple.com>
  Date:   2021-07-16 (Fri, 16 Jul 2021)

  Changed paths:
    M llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp
    M llvm/test/CodeGen/AArch64/GlobalISel/legalize-non-pow2-load-store.mir

  Log Message:
  -----------
  [GlobalISel] Fix non-pow-2 legalization of s56 stores.

s56 stores are broken down into s32 + s24 stores. During this step
both of those new stores use an anyextended s64 value, resulting in
truncating stores. With s56, the s24 requires another lower step to
make it legal, and we were crashing because we didn't expect non-pow-2
stores to also be truncating as well.

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




More information about the All-commits mailing list