[PATCH] D14961: LLVM CodeView library

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 5 23:24:46 PST 2016


echristo added a subscriber: echristo.
echristo added a comment.

Hi Dave,

This looks pretty good, I appreciate all the work. Couple of questions/comments:

a) No tests? :(
b) Do all of the headers need to be in llvm/include rather than in llvm/lib? I have no real complaints or objection, just curious if they're going to be used outside or if it was just a "while I was there" sort of change.

Thanks!

-eric


Repository:
  rL LLVM

http://reviews.llvm.org/D14961





More information about the llvm-commits mailing list