Details
Diff Detail
Event Timeline
test/CodeGen/ARM/load-combine-big-endian.ll | ||
---|---|---|
428 | 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). |
test/CodeGen/ARM/load-combine-big-endian.ll | ||
---|---|---|
428 | Ah, good catch! REV16 is available on ARMv6+, Thumb1, 2 and ARM (and our table gen descriptions have that already), but the check lines below are only for "arm" which is "ARMv4". This test is ok (has the CHECK-ARMv6) but the LE below has "CHECK64" which is why I got confused. :) | |
test/CodeGen/ARM/load-combine.ll | ||
395 | So we just need to fix the typo here: CHECK64 -> CHECK-ARMv6 |
Drop the braces (style)