[PATCH] D29397: [DAGCombiner] Support bswap as a part of load combine patterns
Filipe Cabecinhas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 1 09:06:30 PST 2017
filcab added inline comments.
Comment at: test/CodeGen/ARM/load-combine-big-endian.ll:428
+; CHECK: ldr r0, [r0]
+; CHECK-NEXT: mov r1, #65280
+; CHECK-NEXT: mov r2, #16711680
> I'm surprised this doesn't use `REV16`.
`REV16` has ` Requires<[IsARM, HasV6]>,`, but from the run lines, it seems that the default for "arm" is probably before v6 (I tried looking for the default, but couldn't find it).
Could that be it?
More information about the llvm-commits