[PATCH] D14659: [AArch64LoadStoreOpt] Add support for pairing volatile loads/stores.

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 13 09:06:09 PST 2015


mcrosier created this revision.
mcrosier added subscribers: flyingforyou, t.p.northover, llvm-commits.
Herald added subscribers: rengolin, aemerson.

This is just an example for D14489.

This is untested.

This also needs test cases to be updated (i.e., test/CodeGen/AArch64/arm64-volatile.ll and test/CodeGen/AArch64/arm64-platform-reg.ll).

There needs to be tests for stps similar to those included with ldps.

I'm concerned this patch will pair (and I believe reorder) things such as:

    ldr x8, [x0, #8]
    ldr x9, [x0]

into

    ldp x9, x8, [x0]

This might be okay.. I don't know.  Tim/James/Other?


http://reviews.llvm.org/D14659

Files:
  lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp
  test/CodeGen/AArch64/arm64-ldp.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14659.40155.patch
Type: text/x-patch
Size: 5322 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151113/f193b56e/attachment.bin>


More information about the llvm-commits mailing list