MaskRay accepted this revision. MaskRay added inline comments.
================ Comment at: llvm/lib/Target/Mips/MipsAsmPrinter.cpp:402 MipsTargetStreamer &TS = getTargetStreamer(); + const MipsTargetMachine &MTM = static_cast<const MipsTargetMachine &>(TM); ---------------- delete blank line ================ Comment at: llvm/lib/Target/Mips/MipsAsmPrinter.cpp:404 + const MipsTargetMachine &MTM = static_cast<const MipsTargetMachine &>(TM); + const MipsABIInfo &ABI = MTM.getABI(); + ---------------- If `ABI` only used once, avoid the variable CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138179/new/ https://reviews.llvm.org/D138179 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits