[PATCH] D67343: [DebugInfo] Change object::RelocationResolver to return Expected<uint64_t>

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 9 02:41:50 PDT 2019


ruiu added a comment.

Could you also add an actual overflow check to some target to show that this is really required? It will also make it possible to write a test for the new code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67343





More information about the llvm-commits mailing list