[PATCH] D12985: [ARM] Take into account address spaces in interleaved access vectorization

Jeroen Ketema via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 03:47:59 PDT 2015


jketema updated this revision to Diff 35602.
jketema added a comment.

This now upgrades the vld[234] and vst[234] intrinsics to take into account an address space, as discussed.

Open issues as far as I'm concerned:

- Should vld1 and vst1 also be upgraded for uniformity? Similarly for the vld[234]lane and vst[234]lane instructions?
- Should all the tests referring to vld[234] and vst[234] be updated to use the new intrinsic names, or are they allowed to depend on the auto-upgrade code?


http://reviews.llvm.org/D12985

Files:
  include/llvm/IR/IntrinsicsARM.td
  lib/IR/AutoUpgrade.cpp
  lib/Target/ARM/ARMISelLowering.cpp
  test/CodeGen/ARM/arm-interleaved-accesses.ll
  test/Transforms/InstCombine/neon-intrinsics.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12985.35602.patch
Type: text/x-patch
Size: 10027 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150924/1d299c24/attachment.bin>


More information about the llvm-commits mailing list