Re: [PATCH] D29303: In VirtualCallChecker, handle indirect calls

Thank you! On Friday, February 24, 2017, Hans Wennborg wrote: > Yes, this looks very straight-forward. Merged in r296154. > > On Fri, Feb 24, 2017 at 4:29 AM, Sam McCall via cfe-commits > > wrote: > > Thanks Anna, I'm new to the

r296282 - [index] Add 'Parameter' symbol kind and 'Local' symbol property to distinguish function-local symbols

Author: akirtzidis Date: Sat Feb 25 23:37:56 2017 New Revision: 296282 URL: http://llvm.org/viewvc/llvm-project?rev=296282=rev Log: [index] Add 'Parameter' symbol kind and 'Local' symbol property to distinguish function-local symbols Parameters have a 'child' relation to their function/method.

[PATCH] D21279: Fix some issues in clang-format's AlignConsecutive modes

bmharper added a comment. Hi @djasper, This is the first patch I've contributed here, so I'm not familiar with the whole process. I assume this code is ready to land? When exactly does it get merged into master, and is there something else that I still need to do to make that happen? Thanks,

r296276 - Update cxx_dr_status page.

Author: rsmith Date: Sat Feb 25 17:54:18 2017 New Revision: 296276 URL: http://llvm.org/viewvc/llvm-project?rev=296276=rev Log: Update cxx_dr_status page. Modified: cfe/trunk/www/cxx_dr_status.html Modified: cfe/trunk/www/cxx_dr_status.html URL:

r296275 - C++ DR1611, 1658, 2180: implement "potentially constructed subobject" rules for special member functions.

Author: rsmith Date: Sat Feb 25 17:53:05 2017 New Revision: 296275 URL: http://llvm.org/viewvc/llvm-project?rev=296275=rev Log: C++ DR1611, 1658, 2180: implement "potentially constructed subobject" rules for special member functions. Essentially, as a base class constructor does not construct

Re: r296193 - [Test] Make Lit tests C++11 compatible #10

Den 25 feb. 2017 7:23 em skrev "Charles Li via cfe-commits" < cfe-commits@lists.llvm.org>: Author: lcharles Date: Fri Feb 24 17:23:53 2017 New Revision: 296193 URL: http://llvm.org/viewvc/llvm-project?rev=296193=rev Log: [Test] Make Lit tests C++11 compatible #10 Differential Revision:

r296267 - [test] Disable test/Index/pch-from-libclang.c for non-darwin systems.

Author: akirtzidis Date: Sat Feb 25 13:17:11 2017 New Revision: 296267 URL: http://llvm.org/viewvc/llvm-project?rev=296267=rev Log: [test] Disable test/Index/pch-from-libclang.c for non-darwin systems. Modified: cfe/trunk/test/Index/pch-from-libclang.c Modified:

r296265 - [test] Use cc1 -triple for test/Index/pch-from-libclang.c.

Author: akirtzidis Date: Sat Feb 25 12:55:32 2017 New Revision: 296265 URL: http://llvm.org/viewvc/llvm-project?rev=296265=rev Log: [test] Use cc1 -triple for test/Index/pch-from-libclang.c. Note quite sure why driver -target has no effect. Modified: cfe/trunk/test/Index/pch-from-libclang.c

r296263 - [test] Add target to test/Index/pch-from-libclang.c.

Author: akirtzidis Date: Sat Feb 25 12:35:53 2017 New Revision: 296263 URL: http://llvm.org/viewvc/llvm-project?rev=296263=rev Log: [test] Add target to test/Index/pch-from-libclang.c. Attempt to fix the failing bots. Modified: cfe/trunk/test/Index/pch-from-libclang.c Modified:

r296261 - [modules] For -module-file-info, print out the resource dir path, since it is included in the module hash.

Author: akirtzidis Date: Sat Feb 25 12:14:31 2017 New Revision: 296261 URL: http://llvm.org/viewvc/llvm-project?rev=296261=rev Log: [modules] For -module-file-info, print out the resource dir path, since it is included in the module hash. Modified: cfe/trunk/lib/Frontend/FrontendActions.cpp

r296262 - [driver] Pass a resource dir without the '/../' part.

Author: akirtzidis Date: Sat Feb 25 12:14:35 2017 New Revision: 296262 URL: http://llvm.org/viewvc/llvm-project?rev=296262=rev Log: [driver] Pass a resource dir without the '/../' part. This get the resource dir string to match with the one from libclang (which is not adding '/../'), and allows

r296193 - [Test] Make Lit tests C++11 compatible #10

Author: lcharles Date: Fri Feb 24 17:23:53 2017 New Revision: 296193 URL: http://llvm.org/viewvc/llvm-project?rev=296193=rev Log: [Test] Make Lit tests C++11 compatible #10 Differential Revision: https://reviews.llvm.org/D21626 Modified: cfe/trunk/test/Modules/Inputs/merge-using-decls/b.h

Re: [PATCH] Improved plugin/tool support by expanding an existing attribute

As interesting the subject of pluggable attributes may be could we please drop that discussion and focus on the intention of and what the patch actually does? Two revisions of the patch have been supplied and both should reasonable not cause any controversy since they only enriches an already

r296184 - [Test] Make Lit tests C++11 compatible #9

Author: lcharles Date: Fri Feb 24 16:22:05 2017 New Revision: 296184 URL: http://llvm.org/viewvc/llvm-project?rev=296184=rev Log: [Test] Make Lit tests C++11 compatible #9 [Test] Make Lit tests C++11 compatible #9 Differential Revision: https://reviews.llvm.org/D20710 Modified:

r296256 - Also test OpenBSD/powerpc here for the various types expected to be the same across archs.

Author: brad Date: Sat Feb 25 10:35:18 2017 New Revision: 296256 URL: http://llvm.org/viewvc/llvm-project?rev=296256=rev Log: Also test OpenBSD/powerpc here for the various types expected to be the same across archs. Modified: cfe/trunk/test/Preprocessor/init.c Modified:

Re: r296166 - clang-format: Don't leave behind temp files in -i mode on Windows, PR26125

Reverted in r296237 . (MemoryBuffer)Code is referred after Code.reset(). On Sun, Feb 26, 2017 at 12:40 AM Nico Weber via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Thanks, I had fixed this in 171 a bit over an hour before your mail. > > On Feb 24, 2017 5:17 PM, "Renato Golin via

Re: r296166 - clang-format: Don't leave behind temp files in -i mode on Windows, PR26125

Thanks, I had fixed this in 171 a bit over an hour before your mail. On Feb 24, 2017 5:17 PM, "Renato Golin via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > On 24 February 2017 at 20:49, Nico Weber via cfe-commits > wrote: > > Author: nico > > Date: Fri Feb 24

RE: r296171 - Try to unbreak tests after r296166

Can't you just put gnuwin rm on your bot? Our Windows bots are happy with the test, and having to support an rm without wildcard support seems pretty strange. (Https://is.gd/chromeclang -> tools -> gnuwin-6.zip are the executables our bot uses to run llvm tests) On Feb 24, 2017 8:13 PM, "Yung,

[PATCH] D30378: [DebugInfo] [DWARFv5] Collect calling convention info for C++ types during codegen

vleschuk created this revision. When generating debug info collect information on calling convention for types (http://www.dwarfstd.org/ShowIssue.php?issue=141215.1) and pass it to backend. https://reviews.llvm.org/D30378 Files: lib/CodeGen/CGDebugInfo.cpp lib/CodeGen/CGDebugInfo.h

[PATCH] D30375: Function with unparsed body is a definition

sepavloff created this revision. While a function body is being parsed, the function declaration is not considered as a definition because it does not have a body yet. In some cases it leads to incorrect interpretation, the case is presented in https://bugs.llvm.org/show_bug.cgi?id=14785: