[PATCH] D63120: Share /machine: handling code with llvm-cvtres too

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 11 05:32:47 PDT 2019


ruiu added a comment.

I can see why you created a new library, but it still look a bit odd that the actual library code is less than 40 lines. Do you think adding this to Object make sense? This is not really Object-related, but I guess adding a small amount of code there wouldn't harm anyone.


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

https://reviews.llvm.org/D63120





More information about the llvm-commits mailing list