[Lldb-commits] [PATCH] D71575: [LLDB] Add ObjectFileWasm plugin for WebAssembly debugging

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

BTW, I had to fix this patch (cd9e5c32302cd3b34b796683eedb072c6a1cfdc1 <https://reviews.llvm.org/rGcd9e5c32302cd3b34b796683eedb072c6a1cfdc1>) to build on macOS.  uint64_t and size_t are differently spelled (though I think otherwise equivalent.)  One is "long long unsigned int", the other "long unsigned int".  I have no idea why that's true, but std::min refuses to compare a size_t and a unit64_t.  Anyway, I fixed this by casting one of the two sides of the comparison.  But this was causing problems because we have an api (ReadImageData) that takes a uint64_t for the offset and a size_t for the size.  That seems a little weird to me, why are these different types?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71575