[all-commits] [llvm/llvm-project] ecfa01: [lldb] on s390x fix override issue

Konrad Kleine via All-commits all-commits at lists.llvm.org
Fri Jul 10 12:12:21 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ecfa01e956a49ed349683f1cfcfbbec423114b68
      https://github.com/llvm/llvm-project/commit/ecfa01e956a49ed349683f1cfcfbbec423114b68
  Author: Konrad Kleine <kkleine at redhat.com>
  Date:   2020-07-10 (Fri, 10 Jul 2020)

  Changed paths:
    M lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_s390x.h

  Log Message:
  -----------
  [lldb] on s390x fix override issue

Summary:
This fixes an override issue by marking a function as const so that the
signature maps to the signature of the function in the base class.

This is the original error:

In file included from /root/llvm/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_s390x.cpp:11:
/root/llvm/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_s390x.h:79:10: error: 'size_t lldb_private::process_linux::NativeRegisterContextLinux_s390x::GetGPRSize()' marked 'override', but does not override
   79 |   size_t GetGPRSize() override { return sizeof(m_regs); }
      |          ^~~~~~~~~~

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D83580




More information about the All-commits mailing list