[PATCH] D56178: [Nios2] Update overrides to new virtual method signatures

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 2 10:12:44 PST 2019


craig.topper added inline comments.


================
Comment at: lib/Target/Nios2/MCTargetDesc/Nios2AsmBackend.h:43
                   const MCValue &Target, MutableArrayRef<char> Data,
-                  uint64_t Value, bool IsResolved) const override;
+                  uint64_t Value, bool IsResolved, const MCSubtargetInfo *STI) const override;
 
----------------
Is this line longer than 80 characters? If so please wrap it.


================
Comment at: lib/Target/Nios2/MCTargetDesc/Nios2AsmBackend.h:58
+  /// encoded.
+  bool mayNeedRelaxation(const MCInst &Inst, const MCSubtargetInfo &STI) const override { return false; }
 
----------------
Please wrap this line to 80 characters or less.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56178





More information about the llvm-commits mailing list