[PATCH] D103167: [ARM] Fix Machine Outliner LDRD/STRD handling in Thumb mode

Yvan Roux via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 2 06:47:57 PDT 2021


yroux added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp:6001
   default:
     llvm_unreachable("Unsupported addressing mode!");
   }
----------------
yroux wrote:
> chill wrote:
> > Can't we instead `return false;` here and get rid of enumerating all unsupported addressing modes on line 5933?
> hmm, yes I agree it'll make it easier to read
The issue is that getting the offset value line 5948 can only be done if the addressing mode has one, so we can't get rid of the checking line 5933.  I already planned to rework this part and extract it since that kind of logic is used in multiple places in the backend, but what about keeping it as it for this bugfix ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103167



More information about the llvm-commits mailing list