JDevlieghere marked an inline comment as done.
JDevlieghere added inline comments.


================
Comment at: lldb/include/lldb/Utility/DataExtractor.h:1000-1004
+      uint8_t *dst_data = reinterpret_cast<uint8_t *>(&val);
+      const uint8_t *src_data = reinterpret_cast<const uint8_t *>(src);
+      for (size_t i = 0; i < src_size; ++i)
+        dst_data[src_size - 1 - i] = src_data[i];
+      return val;
----------------
clayborg wrote:
> Might be nice to place this into a method function that swaps data of any 
> size. Then other functions can re-use it if needed.
I left it inline here so we can copy the data in place instead copy & swapping. 


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

https://reviews.llvm.org/D83256



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to