[PATCH] D20840: [codeview] Translate basic DITypes to CV type records

Amjad Aboud via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 1 13:58:47 PDT 2016


aaboud added a comment.

In http://reviews.llvm.org/D20840#445895, @rnk wrote:

> Amjad, I went ahead and landed this. I know it conflicts with what you're working on, but I didn't see any updates at http://reviews.llvm.org/D20435 and this is the most important thing for me to work on. Hopefully we agree on the direction we're going here, I'll try to work very incrementally and CC you on every patch. I was going to look at POD structs (ignoring C++ methods) and function types next.


I intended to update http://reviews.llvm.org/D20435 (was waiting for your other patch of improving codeview assembly readability).
Anyway, I stopped working on http://reviews.llvm.org/D20435 once I noticed this patch.
Sorry, I did not have enough time to review it yet, but I will do soon and let you know if there is any change need to be done.


Repository:
  rL LLVM

http://reviews.llvm.org/D20840





More information about the llvm-commits mailing list