[all-commits] [llvm/llvm-project] a7697c: [ARM] Do not assume alignment in vld1xN and vst1xN...

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a7697c86559e9d57c9c0e2b5f2daaa5cec4e5119
      https://github.com/llvm/llvm-project/commit/a7697c86559e9d57c9c0e2b5f2daaa5cec4e5119
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2024-09-05 (Thu, 05 Sep 2024)

  Changed paths:
    M llvm/lib/Target/ARM/ARMISelLowering.cpp
    M llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
    M llvm/test/CodeGen/ARM/arm-vld1.ll
    M llvm/test/CodeGen/ARM/arm-vst1.ll
    M llvm/test/CodeGen/ARM/bf16-intrinsics-ld-st.ll
    M llvm/test/Transforms/InstCombine/ARM/neon-intrinsics.ll

  Log Message:
  -----------
  [ARM] Do not assume alignment in vld1xN and vst1xN intrinsics (#106984)

These intrinsics currently assume natural alignment. Instead, respect
the alignment attribute on the intrinsic. Teach InstCombine to improve
that alignment.

If desired I could also adjust the clang frontend to add alignment
annotations equivalent to the previous behavior, but I don't see any
indication that such an assumption is correct in the ARM intrinsics
docs.

Fixes https://github.com/llvm/llvm-project/issues/59081.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications