lgtm

On Wed, Mar 1, 2017 at 4:26 PM Jason Majors via Phabricator via
lldb-commits <lldb-commits@lists.llvm.org> wrote:

> jmajors created this revision.
>
> It does not change members or call non-const members.
> HostInfo::GetArchitecture() returns a const object ref (maybe others?),
> which can't access the non-const function.
>
>
> https://reviews.llvm.org/D30515
>
> Files:
>   include/lldb/Core/ArchSpec.h
>   source/Core/ArchSpec.cpp
>
>
> Index: source/Core/ArchSpec.cpp
> ===================================================================
> --- source/Core/ArchSpec.cpp
> +++ source/Core/ArchSpec.cpp
> @@ -657,7 +657,7 @@
>    SetFlags(flag);
>  }
>
> -std::string ArchSpec::GetClangTargetCPU() {
> +std::string ArchSpec::GetClangTargetCPU() const {
>    std::string cpu;
>    const llvm::Triple::ArchType machine = GetMachine();
>
> Index: include/lldb/Core/ArchSpec.h
> ===================================================================
> --- include/lldb/Core/ArchSpec.h
> +++ include/lldb/Core/ArchSpec.h
> @@ -307,7 +307,7 @@
>    /// @return A string representing target CPU for the current
>    ///         architecture.
>    //------------------------------------------------------------------
> -  std::string GetClangTargetCPU();
> +  std::string GetClangTargetCPU() const;
>
>    //------------------------------------------------------------------
>    /// Return a string representing target application ABI.
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to