[PATCH] D63847: [MC] Add MCInstrAnalysis::evaluateMemoryOperandAddress

[Brian Cain via Phabricator via llvm-commits]

bcain added inline comments.


================
Comment at: llvm/include/llvm/MC/MCInstrAnalysis.h:157
+  /// true on success, and the address in Target.
+  virtual bool
+  evaluateMemoryOperandAddress(const MCInst &Inst, uint64_t Addr, uint64_t Size,
----------------
It may make more sense for this to return an llvm::Optional<uint64_t>.


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

https://reviews.llvm.org/D63847