[PATCH] D84923: [ARM] Fix so immediates and pc relative checks

[Simon Wallis via Phabricator via llvm-commits]

simonwallis2 marked 2 inline comments as done.
simonwallis2 added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMConstantIslandPass.cpp:287
+      case ARM::LEApcrel:
+        return 4;
+      case ARM::t2LEApcrel:
----------------
dmgreen wrote:
> Why 4 and not 8? I think it's meant to be "2 instructions widths"
Value of 4 was determined empirically.


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

https://reviews.llvm.org/D84923