[PATCH] D29892: ar: add llvm-dlltool support

[Martell Malone via Phabricator via llvm-commits]

martell added inline comments.


================
Comment at: lib/CMakeLists.txt:24
 add_subdirectory(Passes)
+add_subdirectory(DlltoolDriver)
 add_subdirectory(LibDriver)
----------------
pcc wrote:
> martell wrote:
> > pcc wrote:
> > > Just a comment on the location: I'm not sure if we want too many of these tool drivers to live in their own top-level directories under `lib`. If we're planning to add this one and maybe others (cvtres?) we should think about creating a new top-level directory and moving the drivers underneath that directory. My preferred bikeshed colour is: `lib/ToolDrivers/llvm-lib`,  `lib/ToolDrivers/llvm-dlltool`, etc.
> > > 
> > > If this sounds good I'll go ahead and move `LibDriver`.
> > Sure that sounds good to me.
> r302995
thanks, I will follow with an update


Repository:
  rL LLVM

https://reviews.llvm.org/D29892