[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