[llvm] 1d32143 - [llvm-xray] Add AArch64 to llvm-xray extract

Aditya K via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 8 11:57:22 PST 2019


oh, I must have missed it. I'll add them tonight when I'm back on my home computer where i have the commit access.
Sorry about that.

-Aditya




From: Roman Lebedev <lebedev.ri at gmail.com>

Sent: Friday, November 8, 2019 8:40 AM

To: Aditya Kumar <hiraditya at msn.com>; Aditya Kumar <llvmlistbot at llvm.org>

Cc: llvm-commits at lists.llvm.org <llvm-commits at lists.llvm.org>

Subject: Re: [llvm] 1d32143 - [llvm-xray] Add AArch64 to llvm-xray extract

 


Where did the test coverage go?



On Fri, Nov 8, 2019 at 5:38 PM Aditya Kumar via llvm-commits

<llvm-commits at lists.llvm.org> wrote:

>

>

> Author: Aditya Kumar

> Date: 2019-11-08T06:37:47-08:00

> New Revision: 1d321434a202b12b3afe6f305262b930bbf05665

>

> URL: 
https://github.com/llvm/llvm-project/commit/1d321434a202b12b3afe6f305262b930bbf05665

> DIFF: 
https://github.com/llvm/llvm-project/commit/1d321434a202b12b3afe6f305262b930bbf05665.diff

>

> LOG: [llvm-xray] Add AArch64 to llvm-xray extract

>

> This required adding support for resolving R_AARCH64_ABS64 relocations to

> get accurate addresses for function names to resolve.

>

> Authored by: ianlevesque (Ian Levesque)

> Reviewers: dberris, phosek, smeenai, tetsuo-cpp

> Differential Revision: https://reviews.llvm.org/D69967

>

> Added:

>

>

> Modified:

>     llvm/lib/XRay/InstrumentationMap.cpp

>

> Removed:

>

>

>

> ################################################################################

> diff  --git a/llvm/lib/XRay/InstrumentationMap.cpp b/llvm/lib/XRay/InstrumentationMap.cpp

> index 7453613c7038..1e9b69a5f9dc 100644

> --- a/llvm/lib/XRay/InstrumentationMap.cpp

> +++ b/llvm/lib/XRay/InstrumentationMap.cpp

> @@ -20,6 +20,7 @@

>  #include "llvm/Object/Binary.h"

>  #include "llvm/Object/ELFObjectFile.h"

>  #include "llvm/Object/ObjectFile.h"

> +#include "llvm/Object/RelocationResolver.h"

>  #include "llvm/Support/DataExtractor.h"

>  #include "llvm/Support/Error.h"

>  #include "llvm/Support/FileSystem.h"

> @@ -59,7 +60,8 @@ loadObj(StringRef Filename, object::OwningBinary<object::ObjectFile> &ObjFile,

>    // Find the section named "xray_instr_map".

>    if ((!ObjFile.getBinary()->isELF() && !ObjFile.getBinary()->isMachO()) ||

>        !(ObjFile.getBinary()->getArch() == Triple::x86_64 ||

> -        ObjFile.getBinary()->getArch() == Triple::ppc64le))

> +        ObjFile.getBinary()->getArch() == Triple::ppc64le ||

> +        ObjFile.getBinary()->getArch() == Triple::aarch64))

>      return make_error<StringError>(

>          "File format not supported (only does ELF and Mach-O little endian 64-bit).",

>          std::make_error_code(std::errc::not_supported));

> @@ -99,12 +101,22 @@ loadObj(StringRef Filename, object::OwningBinary<object::ObjectFile> &ObjFile,

>          return static_cast<uint32_t>(0);

>      }(ObjFile.getBinary());

>

> +    bool (*SupportsRelocation)(uint64_t);

> +    object::RelocationResolver Resolver;

> +    std::tie(SupportsRelocation, Resolver) =

> +        object::getRelocationResolver(*ObjFile.getBinary());

> +

>      for (const object::SectionRef &Section : Sections) {

>        for (const object::RelocationRef &Reloc : Section.relocations()) {

> -        if (Reloc.getType() != RelativeRelocation)

> -          continue;

> -        if (auto AddendOrErr = object::ELFRelocationRef(Reloc).getAddend())

> -          Relocs.insert({Reloc.getOffset(), *AddendOrErr});

> +        if (SupportsRelocation && SupportsRelocation(Reloc.getType())) {

> +          auto AddendOrErr = object::ELFRelocationRef(Reloc).getAddend();

> +          auto A = AddendOrErr ? *AddendOrErr : 0;

> +          uint64_t resolved = Resolver(Reloc, Reloc.getSymbol()->getValue(), A);

> +          Relocs.insert({Reloc.getOffset(), resolved});

> +        } else if (Reloc.getType() == RelativeRelocation) {

> +          if (auto AddendOrErr = object::ELFRelocationRef(Reloc).getAddend())

> +            Relocs.insert({Reloc.getOffset(), *AddendOrErr});

> +        }

>        }

>      }

>    }

>

>

>

> _______________________________________________

> llvm-commits mailing list

> llvm-commits at lists.llvm.org

> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list