[PATCH] D30758: Update clang-cl driver for MSVC 2017

[David Majnemer via Phabricator via llvm-commits]

majnemer added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/MSVCSetupApi.h:3
+// Copyright (C) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT license. See LICENSE.txt in the project root for
+// license information.
----------------
dberlin wrote:
> zturner wrote:
> > dberlin wrote:
> > > There is no project root, you should paste the LICENSE.txt into the header directly.
> > > Also, generally, i would segregate this into a third_party directory, as keeping code like this directly in the rest of the project makes it a lot messier
> > Is a third_party directory directly in this folder ok, or should it be some kind of top level third_party directory?
> Normal practice for our open source projects (ie not llvm) would be top-level, because otherwise they tend to be hard to notice.
> Again, unsure what llvm wants the policy to be here.
> 
LLVM, in typical practice, does not segregate small amounts of third party code.

For example:
https://github.com/llvm-mirror/llvm/blob/a93bc6c25340329e49407f5fc231ca870f5c61e3/include/llvm/Support/MD5.h
https://github.com/llvm-mirror/llvm/blob/a93bc6c25340329e49407f5fc231ca870f5c61e3/include/llvm/Support/SHA1.h#L9
https://github.com/llvm-mirror/llvm/blob/a93bc6c25340329e49407f5fc231ca870f5c61e3/include/llvm/Support/xxhash.h


https://reviews.llvm.org/D30758