[PATCH] D51005: [aarch64][mc] Don't lookup symbols when there is no symbol lookup callback

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 21 08:48:10 PDT 2018


This revision was automatically updated to reflect the committed changes.
Closed by commit rL340287: [aarch64][mc] Don't lookup symbols when there is no symbol lookup callback (authored by dsanders, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D51005?vs=161582&id=161738#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D51005

Files:
  llvm/trunk/lib/Target/AArch64/Disassembler/AArch64ExternalSymbolizer.cpp


Index: llvm/trunk/lib/Target/AArch64/Disassembler/AArch64ExternalSymbolizer.cpp
===================================================================
--- llvm/trunk/lib/Target/AArch64/Disassembler/AArch64ExternalSymbolizer.cpp
+++ llvm/trunk/lib/Target/AArch64/Disassembler/AArch64ExternalSymbolizer.cpp
@@ -60,6 +60,8 @@
 bool AArch64ExternalSymbolizer::tryAddingSymbolicOperand(
     MCInst &MI, raw_ostream &CommentStream, int64_t Value, uint64_t Address,
     bool IsBranch, uint64_t Offset, uint64_t InstSize) {
+  if (!SymbolLookUp)
+    return false;
   // FIXME: This method shares a lot of code with
   //        MCExternalSymbolizer::tryAddingSymbolicOperand. It may be possible
   //        refactor the MCExternalSymbolizer interface to allow more of this


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51005.161738.patch
Type: text/x-patch
Size: 766 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180821/87eead06/attachment.bin>


More information about the llvm-commits mailing list