Re: [PATCH] D23455: [Tooling] Parse compilation database command lines properly on Windows

2016-08-14 Thread Zachary Turner via cfe-commits
It occurred to me that eol detection won't work reliably. Especially for tests, someone will write a compilation database by hand and check it in. Maybe they have git set to convert newlines for them, and we don't want a test to behave differently based on your source control settings. The target

Re: [PATCH] D21256: Improved Visual Studio 2015 visualization of SmallVectorImpl

2016-06-13 Thread Zachary Turner via cfe-commits
I agree that it can be annoying to say "hey guys, i would normally do post commit review on this, but i wanted to give the courtesy of a heads up", and then potentially waiting an indeterminate amount of time. I think that actually discourages these kind of changes going up at all, because people

Re: [PATCH] D21643: Default to using the Unicode version of Win32 APIs

2016-06-23 Thread Zachary Turner via cfe-commits
Should we add overloads of the UTF8 conversion functions that accept wstrings? In line with what rnk said, I'm curious about the implications of simply #undef'ing all the non A/W versions of functions so that you have to make the choice explicit On Thu, Jun 23, 2016 at 8:20 AM Reid Kleckner wrote

Re: [PATCH] D21946: Subject: [PATCH] [Driver] fix windows SDK detect

2016-07-01 Thread Zachary Turner via cfe-commits
zturner accepted this revision. zturner added a comment. This revision is now accepted and ready to land. Looks good, do you have commit access to commit this? http://reviews.llvm.org/D21946 ___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

Re: [PATCH] D21946: Subject: [PATCH] [Driver] fix windows SDK detect

2016-07-01 Thread Zachary Turner via cfe-commits
zturner added a comment. In http://reviews.llvm.org/D21946#473070, @comicfans44 wrote: > I've not commited to cfe before, so I think I havn't commit access permission. If you've committed anywhere in LLVM, you should have commit access to cfe. Feel free to give it a try, as I won't be able to

Re: [PATCH] D17908: Add Visual Studio Visualizers for more Clang types

2016-03-07 Thread Zachary Turner via cfe-commits
Better yet, just delete the number On Mon, Mar 7, 2016 at 6:21 AM Aaron Ballman wrote: > aaron.ballman added inline comments. > > > Comment at: clang.natvis:1-3 > @@ -1,4 +1,4 @@ > > Do you know if these visualizers will work in MSVC 2013 as well, or are > you using MSVC 2015

Re: [PATCH] D17908: Add Visual Studio Visualizers for more Clang types

2016-03-07 Thread Zachary Turner via cfe-commits
zturner added a subscriber: zturner. zturner added a comment. Better yet, just delete the number http://reviews.llvm.org/D17908 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D17908: Add Visual Studio Visualizers for more Clang types

2016-03-07 Thread Zachary Turner via cfe-commits
zturner added a comment. Natvis has hardly changed at all since it was introduced, so I'm guessing the lowest common denominator is always going to be our minimum required VS version http://reviews.llvm.org/D17908 ___ cfe-commits mailing list cfe-co

Re: [PATCH] D17908: Add Visual Studio Visualizers for more Clang types

2016-03-07 Thread Zachary Turner via cfe-commits
Natvis has hardly changed at all since it was introduced, so I'm guessing the lowest common denominator is always going to be our minimum required VS version On Mon, Mar 7, 2016 at 7:09 AM Aaron Ballman wrote: > aaron.ballman added a comment. > > In http://reviews.llvm.org/D17908#368969, @zturner

Re: [PATCH] D18498: Auto-install Clang Visual Studio visualizers for VS2015 and up

2016-03-27 Thread Zachary Turner via cfe-commits
For vs2015, the files still need to be in the project right? (In the vcxproj with a tag) On Sun, Mar 27, 2016 at 9:09 AM Mike Spertus wrote: > mspertus updated this revision to Diff 51741. > mspertus added a comment. > > Apply whitespace comments from http://reviews.llvm.org/D18497 mutatis > mut

Re: [PATCH] D18498: Auto-install Clang Visual Studio visualizers for VS2015 and up

2016-03-27 Thread Zachary Turner via cfe-commits
Yea, CMake doesn't actually support natvis files. You'd have to actually patch CMake to make it work. Can you make the vs2015 and vs2013 paths the same? It will still work for 2015 i think, just that 2015 also supports putting them in the project file On Sun, Mar 27, 2016 at 10:04 AM Mike Spertus

Re: [PATCH] D18498: Auto-install Clang Visual Studio visualizers for VS2015 and up

2016-03-27 Thread Zachary Turner via cfe-commits
Code paths. For vs2013 you're installing the natvis files, can we just do that for vs2015 as well? On Sun, Mar 27, 2016 at 11:23 AM Mike Spertus wrote: > mspertus added a comment. > > What do you mean by "making both paths the same"? > > > http://reviews.llvm.org/D18498 > > > > __

Re: [PATCH] D18498: Auto-install Clang Visual Studio visualizers for VS2015 and up

2016-03-27 Thread Zachary Turner via cfe-commits
If it's not using a natvis tag in the vcxproj (which it isn't since CMake doesn't support it during generation) then the new natvis project file support won't work right? In which case the files would need to be copied into documents\vs2015, otherwise it won't work, unless I'm misunderstanding some

Re: [PATCH] D18498: Auto-install Clang Visual Studio visualizers for VS2015 and up

2016-03-27 Thread Zachary Turner via cfe-commits
Ahh that's surprising. If it works even with the none tag, i guess my concerns are not valid On Sun, Mar 27, 2016 at 12:11 PM Mike Spertus wrote: > mspertus added a comment. > > No. Testing shows that it works fine in the project with the `` tag. > VS2015 empirically looks at the extension. > > >

Re: [PATCH] D18498: Auto-install Clang Visual Studio visualizers for VS2015 and up

2016-03-27 Thread Zachary Turner via cfe-commits
Yea sounds fine On Sun, Mar 27, 2016 at 12:20 PM Mike Spertus wrote: > mspertus added a comment. > > I understand your concerns, but on balance, I don't see a better course of > action than what I've done. If I have some time, I'll submit a CMake change > so we can eventually migrate to having CM

Re: [PATCH] D10370: clang-format: Implement AlwaysBreakAfterDeclarationReturnType.

2015-12-11 Thread Zachary Turner via cfe-commits
zturner added a subscriber: zturner. zturner added a comment. What needs to happen for this to go in? If I understand correctly, it is either: 1. Add a new option `TreatDeclarationsLikeDefinitions` 2. Merge this option into `AlwaysBreakAfterDefinitionReturnType` and make it an enum with 5 valu

Re: [PATCH] D10370: clang-format: Implement AlwaysBreakAfterDeclarationReturnType.

2015-12-14 Thread Zachary Turner via cfe-commits
zturner added a comment. In http://reviews.llvm.org/D10370#309789, @djasper wrote: > I think we can add the option AlwaysBreakAfterReturnType but still be > backwards compatible. If clang-format finds only the old option in a > configuration file, it can choose the appropriate value of the new

Re: [PATCH] D10370: clang-format: Implement AlwaysBreakAfterDeclarationReturnType.

2015-12-14 Thread Zachary Turner via cfe-commits
zturner updated this revision to Diff 42774. zturner added a comment. This patch was very old, so it didn't apply against ToT. So this initial update is just to rebase against ToT. I will make changes in a followup patch. http://reviews.llvm.org/D10370 Files: include/clang/Format/Format.h

Re: [PATCH] D10370: clang-format: Implement AlwaysBreakAfterDeclarationReturnType.

2015-12-15 Thread Zachary Turner via cfe-commits
zturner updated this revision to Diff 42895. zturner added a comment. Attempt to address remaining issues in patch. This is my first time touching anything having to do with clang, so there's a good chance I don't know what i'm doing and this needs more work. Let me know. `AlwaysBreakAfterDefi

Re: [PATCH] D10370: clang-format: Implement AlwaysBreakAfterDeclarationReturnType.

2015-12-17 Thread Zachary Turner via cfe-commits
zturner added a comment. ping http://reviews.llvm.org/D10370 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D10370: clang-format: Implement AlwaysBreakAfterDeclarationReturnType.

2015-12-18 Thread Zachary Turner via cfe-commits
zturner added inline comments. Comment at: lib/Format/TokenAnnotator.h:168 @@ -158,1 +167,3 @@ + bool mustBreakForReturnType(const AnnotatedLine &Line, + FormatToken &Token) const; djasper wrote: > Some comment might help. E.g. at t

r256046 - Support AlwaysBreakAfterReturnType

2015-12-18 Thread Zachary Turner via cfe-commits
Author: zturner Date: Fri Dec 18 16:20:15 2015 New Revision: 256046 URL: http://llvm.org/viewvc/llvm-project?rev=256046&view=rev Log: Support AlwaysBreakAfterReturnType This changes the behavior of AlwaysBreakAfterDeclarationReturnType so that it supports breaking after declarations, definitions,

r256055 - Fix invalid enum comparison.

2015-12-18 Thread Zachary Turner via cfe-commits
Author: zturner Date: Fri Dec 18 16:58:42 2015 New Revision: 256055 URL: http://llvm.org/viewvc/llvm-project?rev=256055&view=rev Log: Fix invalid enum comparison. Modified: cfe/trunk/unittests/Format/FormatTest.cpp Modified: cfe/trunk/unittests/Format/FormatTest.cpp URL: http://llvm.org/vie

<    1   2   3