[PATCH] D16041: Change vfs::FileSystem to be managed with std::shared_ptr

2016-01-10 Thread Owen Anderson via cfe-commits
resistor created this revision. resistor added reviewers: chandlerc, bkramer, klimek. resistor added a subscriber: cfe-commits. resistor set the repository for this revision to rL LLVM. Herald added a subscriber: klimek. Managing it with IntrusiveRefCntPtr caused the virtual destructor not to be

Add BeforeWhileInDoWhile BraceWrapping option

2016-01-10 Thread Eric Baker via cfe-commits
Code like this: do { // Some code } while (1); becomes: do { // Some code } while (1); I couldn't think of a better name for the option than BeforeWhileInDoWhile. I'm open to suggestions! Also, this is my first attempt at submitting a patch for this project.

r257291 - Accidentally removed part of the file header. Restoring it back.

2016-01-10 Thread Ekaterina Romanova via cfe-commits
Author: kromanova Date: Sun Jan 10 10:18:09 2016 New Revision: 257291 URL: http://llvm.org/viewvc/llvm-project?rev=257291&view=rev Log: Accidentally removed part of the file header. Restoring it back. Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp Modified: cfe/trunk/lib/Frontend/Co

[PATCH] D16040: [OpenCL] Refine OpenCLImageAccessAttr to OpenCLAccessAttr

2016-01-10 Thread Xiuli PAN via cfe-commits
pxli168 created this revision. pxli168 added reviewers: Anastasia, pekka.jaaskelainen. pxli168 added a subscriber: cfe-commits. OpenCL access qualifiers are now not only used for image types, refine it to avoid misleading, Add semacheck for OpenCL access qualifier as well as test caees. http://

Re: r257260 - [vfs] Normalize working directory if requested.

2016-01-10 Thread Benjamin Kramer via cfe-commits
Fixed in r257286, sorry for the breakage. On Sun, Jan 10, 2016 at 7:59 AM, Ismail Donmez wrote: > Hi, > >> + NormalizedFS.setCurrentWorkingDirectory("/b/c"); >> + NormalizedFS.setCurrentWorkingDirectory("."); >> + ASSERT_EQ("/b/c", NormalizedFS.getCurrentWorkingDirectory().get()); >> + Normal

Re: [PATCH] D15686: PR25910: clang allows two var definitions with the same mangled name

2016-01-10 Thread Andrey Bokhanko via cfe-commits
andreybokhanko added a comment. In http://reviews.llvm.org/D15686#319725, @tra wrote: > A better description of the problem would help. PR itself is somewhat short > on details. Sorry for being not clear. I expected my main reviewer to be @rjmccall, who knows exactly what the problem is. > I

Re: [PATCH] D15686: PR25910: clang allows two var definitions with the same mangled name

2016-01-10 Thread Andrey Bokhanko via cfe-commits
andreybokhanko updated the summary for this revision. andreybokhanko updated this revision to Diff 44425. andreybokhanko marked 7 inline comments as done. andreybokhanko added a comment. Fixed tra's notes. All the fixes are local, logic of the patch is not changed. http://reviews.llvm.org/D15686

r257287 - Simplify test code with initializer lists.

2016-01-10 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sun Jan 10 04:45:19 2016 New Revision: 257287 URL: http://llvm.org/viewvc/llvm-project?rev=257287&view=rev Log: Simplify test code with initializer lists. No functional change intended. Modified: cfe/trunk/unittests/Basic/VirtualFileSystemTest.cpp Modified: cfe/trunk/unitt

r257286 - Make test work on windows by turning \ in paths back into /.

2016-01-10 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sun Jan 10 04:36:59 2016 New Revision: 257286 URL: http://llvm.org/viewvc/llvm-project?rev=257286&view=rev Log: Make test work on windows by turning \ in paths back into /. Modified: cfe/trunk/unittests/Basic/VirtualFileSystemTest.cpp Modified: cfe/trunk/unittests/Basic/Vir