[PATCH] D49984: Fix uninitialized read in ARM's PrintAsmOperand

Momchil Velikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 30 07:33:49 PDT 2018


chill added inline comments.


================
Comment at: lib/Target/ARM/ARMAsmPrinter.cpp:379
         const TargetRegisterInfo *TRI = MF->getSubtarget().getRegisterInfo();
         unsigned Reg = TRI->getSubReg(MO.getReg(), ExtraCode[0] == 'Q' ?
             ARM::gsub_0 : ARM::gsub_1);
----------------
Is this doing the right thing for big-endian? (re: https://reviews.llvm.org/D49778)
(arguably a slightly separate issue, indeed).


Repository:
  rL LLVM

https://reviews.llvm.org/D49984





More information about the llvm-commits mailing list