]> granicus.if.org Git - llvm/commit
[ARM] Mark LEApcrel as not having side effects
authorJohn Brawn <john.brawn@arm.com>
Mon, 15 May 2017 11:50:21 +0000 (11:50 +0000)
committerJohn Brawn <john.brawn@arm.com>
Mon, 15 May 2017 11:50:21 +0000 (11:50 +0000)
commit91719efd8f00fe7108e323c5fd6da4902f1a1b7b
tree1fcf1dfa0d8b57b76190af0ce6a5f45a760964aa
parentfbe48085e657e42388366d9c865e1414e186c68d
[ARM] Mark LEApcrel as not having side effects

Doing this lets us hoist it out of loops, and I've also marked it as
rematerializable the same as the thumb1 and thumb2 counterparts.

It looks like it being marked as such was just a mistake, as the commit that
made that change only mentions LEApcrelJT and in thumb1 and thumb2 only the
LEApcrelJT instructions were marked as having side-effects, so it looks like
the intent was to only mark LEApcrelJT as having side-effects but LEApcrel was
accidentally marked as such also.

Differential Revision: https://reviews.llvm.org/D32857

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@303053 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/ARM/ARMInstrInfo.td
test/CodeGen/ARM/adr-hoist.ll [new file with mode: 0644]