[PATCH] D70487: [DAGCombiner] Allow zextended load combines.

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 21 06:28:35 PST 2019


dmgreen added inline comments.


================
Comment at: llvm/test/CodeGen/ARM/load-combine-big-endian.ll:5
 ; RUN: llc < %s -mtriple=thumbv6m-none-eabi | FileCheck %s --check-prefix=CHECK-THUMBv6
 ; RUN: llc < %s -mtriple=thumbv7m-none-eabi | FileCheck %s --check-prefix=CHECK-THUMBv7
 
----------------
courbet wrote:
> dmgreen wrote:
> > RKSimon wrote:
> > > @dmgreen Is this a big endian triple?
> > Ah, yeah. Try thumbv6meb-none-eabi and thumbv7meb-none-eabi for big endian.
> D'oh. Sorry about that.
Nice one.

I'm happy, if RKSimon is happy.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70487/new/

https://reviews.llvm.org/D70487





More information about the llvm-commits mailing list