r290166 - Add test for DR692.

Author: rsmith Date: Tue Dec 20 01:24:53 2016 New Revision: 290166 URL: http://llvm.org/viewvc/llvm-project?rev=290166&view=rev Log: Add test for DR692. Modified: cfe/trunk/test/CXX/drs/dr6xx.cpp Modified: cfe/trunk/test/CXX/drs/dr6xx.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/t

[PATCH] D27971: Make FormatStyle.GetStyleOfFile test work on MSVC

amaiorano created this revision. amaiorano added reviewers: klimek, djasper, hans, cfe-commits. This is more a workaround than a real fix. The real problem is that FS.addFile uses clang::vfs::FileSystem::makeAbsolute to convert the input path to an absolute path, while getStyle uses llvm::sys::f

[PATCH] D25206: [Parser] Correct typo after lambda capture initializer is parsed

ahatanak added a comment. Thanks all for the review. Comment at: cfe/trunk/test/SemaCXX/lambda-expressions.cpp:572 +void foo1() { + auto s0 = S1{[name=]() {}}; // expected-error 2 {{expected expression}} + auto s1 = S1{[name=name]() {}}; // expected-error {{use of undeclared

Re: [PATCH] D27683: Prepare PrettyStackTrace for LLDB adoption

On Mon, Dec 19, 2016 at 9:39 AM Sean Callanan wrote: > That would require making LLDB crash and collecting the relevant crash log > data out of the user's own logs. This isn't impossible – but additionally > the generation of that log is asynchronous and you don't quite know when > it'll land. >

Re: [PATCH] D27180: Testbed and skeleton of a new expression parser

I don't know much about who owns this sort of code, nor Vassil's work/responsibility in this area - but if he's OK with it/feels able to sign off on it, that'd be sufficient/fine by me (& others closer to it can pipe up if he seems like not the right person to approve). Thanks! - Dave On Mon, Dec

[PATCH] D27430: [libc++] Annotate template methods with visibility

smeenai requested a review of this revision. smeenai added a comment. I think this actually stands well on its own. I'll do the extern template annotations in a different diff. Also, ping :) https://reviews.llvm.org/D27430 ___ cfe-commits mailing

[PATCH] D25208: [libc++] Make _LIBCPP_TYPE_VIS export members

smeenai requested a review of this revision. smeenai added a comment. Actually, I think this can stand on its own. I'll do the extern template fixes in another diff. Also, ping :) https://reviews.llvm.org/D25208 ___ cfe-commits mailing list cfe-co

[PATCH] D26949: [libc++abi] Clean up visibility

smeenai added a comment. Ping. https://reviews.llvm.org/D26949 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27153: [libc++] Make __num_get_float hidden

smeenai added a comment. Ping. https://reviews.llvm.org/D27153 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D21113: Add support for case-insensitive header lookup

pcc added inline comments. Comment at: lib/Basic/VirtualFileSystem.cpp:483 +CaseInsensitiveFileSystem::openFileForRead(const Twine &Path) { + SmallVector NewPath; + if (std::error_code EC = findCaseInsensitivePath(Path.str(), NewPath)) I wonder whether it would

[PATCH] D21113: Add support for case-insensitive header lookup

francisco.lopes added a comment. I'm waiting for this to land as well, it's crucial for coding windows code with libclang assistance on a linux machine. https://reviews.llvm.org/D21113 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://

r290159 - Fix the spelling of 'bitfield' in diagnostics to be consistently 'bit-field'.

Author: chandlerc Date: Mon Dec 19 20:43:58 2016 New Revision: 290159 URL: http://llvm.org/viewvc/llvm-project?rev=290159&view=rev Log: Fix the spelling of 'bitfield' in diagnostics to be consistently 'bit-field'. The latter agrees with most existing diagnostics and the C and C++ standards. Diff

[PATCH] D26530: Fix the spelling of 'bitfield' in diagnostics to be consistently 'bit-field'.

This revision was automatically updated to reflect the committed changes. Closed by commit rL290159: Fix the spelling of 'bitfield' in diagnostics to be consistently 'bit-field'. (authored by chandlerc). Changed prior to commit: https://reviews.llvm.org/D26530?vs=81008&id=82055#toc Repository:

[PATCH] D27872: [APFloat] Switch from (PPCDoubleDoubleImpl, IEEEdouble) layout to (IEEEdouble, IEEEdouble)

mehdi_amini added inline comments. Comment at: llvm/include/llvm/ADT/APFloat.h:800 - void makeLargest(bool Neg) { getIEEE().makeLargest(Neg); } + void makeLargest(bool Neg) { +if (usesLayout(getSemantics())) { jtony wrote: > I know it is allowed to return

[PATCH] D25206: [Parser] Correct typo after lambda capture initializer is parsed

This revision was automatically updated to reflect the committed changes. Closed by commit rL290156: [Parser] Correct typo after lambda capture initializer is parsed. (authored by ahatanak). Changed prior to commit: https://reviews.llvm.org/D25206?vs=75819&id=82052#toc Repository: rL LLVM h

r290156 - [Parser] Correct typo after lambda capture initializer is parsed.

Author: ahatanak Date: Mon Dec 19 20:11:29 2016 New Revision: 290156 URL: http://llvm.org/viewvc/llvm-project?rev=290156&view=rev Log: [Parser] Correct typo after lambda capture initializer is parsed. This patch fixes an assertion that is triggered when RecordLayoutBuilder tries to compute the si

r290155 - Update for LLVM global variable debug info API change.

Author: adrian Date: Mon Dec 19 20:10:02 2016 New Revision: 290155 URL: http://llvm.org/viewvc/llvm-project?rev=290155&view=rev Log: Update for LLVM global variable debug info API change. This reapplies r289921. Modified: cfe/trunk/lib/CodeGen/CGDebugInfo.cpp cfe/trunk/lib/CodeGen/CGDebu

Re: r289996 - IRGen: Fix assertion failure when creating debug info for an integer constant wider than 64 bits.

> On Dec 16, 2016, at 2:10 PM, Peter Collingbourne via cfe-commits > wrote: > > Author: pcc > Date: Fri Dec 16 16:10:52 2016 > New Revision: 289996 > > URL: http://llvm.org/viewvc/llvm-project?rev=289996&view=rev > Log: > IRGen: Fix assertion failure when creating debug info for an integer con

buildbot numbers for the week of 12/11/2016 - 12/17/2016

Hello everyone, Below are some buildbot numbers for the last week of 12/11/2016 - 12/17/2016. Please see the same data in attached csv files: The longest time each builder was red during the last week; "Status change ratio" by active builder (percent of builds that changed the builder status fro

Buildbot numbers for the week of 12/04/2016 - 12/10/2016

Hello everyone, Below are some buildbot numbers for the week of 12/04/2016 - 12/10/2016. Please see the same data in attached csv files: The longest time each builder was red during the last week; "Status change ratio" by active builder (percent of builds that changed the builder status from gre

Re: Regression caused by r276159 - [modules] Don't emit initializers for VarDecls within a module eagerly whenever

> On Dec 14, 2016, at 1:20 PM, Manman wrote: > > >> On Jul 20, 2016, at 12:10 PM, Richard Smith via cfe-commits >> wrote: >> >> Author: rsmith >> Date: Wed Jul 20 14:10:16 2016 >> New Revision: 276159 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=276159&view=rev >> Log: >> [modules] Do

[PATCH] D26410: [CodeGen] Don't emit the same global block multiple times.

george.burgess.iv abandoned this revision. george.burgess.iv added a comment. Committed in https://reviews.llvm.org/rL290149. Thanks again! https://reviews.llvm.org/D26410 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org

[PATCH] D14274: Add alloc_size attribute to clang

This revision was automatically updated to reflect the committed changes. Closed by commit rL290149: Add the alloc_size attribute to clang. (authored by gbiv). Changed prior to commit: https://reviews.llvm.org/D14274?vs=77222&id=82046#toc Repository: rL LLVM https://reviews.llvm.org/D14274

[PATCH] D14274: Add alloc_size attribute to clang

george.burgess.iv marked an inline comment as done. george.burgess.iv added a comment. Thanks, everyone! :) Repository: rL LLVM https://reviews.llvm.org/D14274 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/

r290149 - Add the alloc_size attribute to clang.

Author: gbiv Date: Mon Dec 19 19:05:42 2016 New Revision: 290149 URL: http://llvm.org/viewvc/llvm-project?rev=290149&view=rev Log: Add the alloc_size attribute to clang. This patch does three things: - Gives us the alloc_size attribute in clang, which lets us infer the number of bytes handed b

Re: r290146 - Fix completely bogus types for some builtins:

On 12/19/2016 3:59 PM, Richard Smith via cfe-commits wrote: Author: rsmith Date: Mon Dec 19 17:59:34 2016 New Revision: 290146 URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/vfprintf-valid-redecl.c?rev=290146&r1=290145&r2=290146&view=diff ==

[Diffusion] rL290025: [libclang] Remove the 'extern "C"' blocks from the implementation files.

chapuni added subscribers: cfe-commits, chapuni. chapuni added a comment. See r290113. I saw linker error on mingw dll build. Cannot export clang_findIncludesInFileWithBlock: symbol not defined Cannot export clang_findReferencesInFileWithBlock: symbol not defined Cannot export clang_visitC

Re: r289754 - [c++1z] Permit constant evaluation of a call through a function pointer whose

On 19 December 2016 at 13:22, Mike Aizatsky wrote: > Richard, > > Clang crashes for me on this code while doing "check-all". This change > seems to introduce the assert. Can you take a look? > > BTW I'm not sure why bots are green. Do we build libcxx with bootstrap > compiler? > Apparently not i

r290146 - Fix completely bogus types for some builtins:

Author: rsmith Date: Mon Dec 19 17:59:34 2016 New Revision: 290146 URL: http://llvm.org/viewvc/llvm-project?rev=290146&view=rev Log: Fix completely bogus types for some builtins: * In C++, never create a FunctionNoProtoType for a builtin (fixes C++1z crasher from r289754). * Fix type of __s

[PATCH] D26546: [PPC] Add vec_insert4b/vec_extract4b to altivec.h

sfertile updated this revision to Diff 82031. sfertile marked 6 inline comments as done. sfertile added a comment. Updated to swap the arguments when generating the intrinsic. Updated a number of the comments, and added some tests with the index out of range. Repository: rL LLVM https://revi

r290145 - Make another test insensitive to the default C++ dialect.

Author: probinson Date: Mon Dec 19 17:32:10 2016 New Revision: 290145 URL: http://llvm.org/viewvc/llvm-project?rev=290145&view=rev Log: Make another test insensitive to the default C++ dialect. Differential Revision: http://reviews.llvm.org/D27955 Modified: cfe/trunk/test/CodeGenCXX/arm-swif

[PATCH] D27955: Make CodeGenCXX/arm-swiftcall.cpp tolerate C++11

This revision was automatically updated to reflect the committed changes. Closed by commit rL290145: Make another test insensitive to the default C++ dialect. (authored by probinson). Changed prior to commit: https://reviews.llvm.org/D27955?vs=82021&id=82028#toc Repository: rL LLVM https://

[PATCH] D27955: Make CodeGenCXX/arm-swiftcall.cpp tolerate C++11

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. LGTM, thanks! https://reviews.llvm.org/D27955 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/l

[PATCH] D27478: Make ASTContext::getDeclAlign return the correct alignment for FunctionDecls

ahatanak added a comment. ping https://reviews.llvm.org/D27478 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D24969: [Sema] Use the instantiated name of destructors in FindInstantiatedDecl and RebuildMemberExpr

ahatanak added a comment. ping https://reviews.llvm.org/D24969 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r290143 - [analyzer] Add checker modeling gtest APIs.

Author: dcoughlin Date: Mon Dec 19 16:50:31 2016 New Revision: 290143 URL: http://llvm.org/viewvc/llvm-project?rev=290143&view=rev Log: [analyzer] Add checker modeling gtest APIs. gtest is a widely-used unit-testing API. It provides macros for unit test assertions: ASSERT_TRUE(p != nullptr);

[PATCH] D27769: Update MSVC compat docs about debug info.

thakis closed this revision. thakis added a comment. r289712 https://reviews.llvm.org/D27769 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27955: Make CodeGenCXX/arm-swiftcall.cpp tolerate C++11

probinson updated this revision to Diff 82021. probinson added a comment. Force C++03 on this test, to make it insensitive to future changes in the default dialect. https://reviews.llvm.org/D27955 Files: test/CodeGenCXX/arm-swiftcall.cpp Index: test/CodeGenCXX/arm-swiftcall.cpp ===

r290141 - Don't try to emit nullability fix-its within/around macros.

Author: jrose Date: Mon Dec 19 16:35:24 2016 New Revision: 290141 URL: http://llvm.org/viewvc/llvm-project?rev=290141&view=rev Log: Don't try to emit nullability fix-its within/around macros. The newly-added notes from r290132 are too noisy even when the fix-it is valid. For the existing warning

[PATCH] D27956: Make CodeGenCXX/stack-reuse-miscompile.cpp tolerate C++11

probinson created this revision. probinson added a reviewer: lenykholodov. probinson added a subscriber: cfe-commits. In this test, the allocas for the temps come out in a different order depending on whether the dialect is C++03 or C++11. To avoid depending on the default dialect, I forced it

[PATCH] D27955: Make CodeGenCXX/arm-swiftcall.cpp tolerate C++11

rjmccall added a comment. Yes, I think being more specific about the dialect would be better. https://reviews.llvm.org/D27955 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r290140 - [analyzer] Add sink after construction of temporary with no-return destructor.

Author: dcoughlin Date: Mon Dec 19 16:23:22 2016 New Revision: 290140 URL: http://llvm.org/viewvc/llvm-project?rev=290140&view=rev Log: [analyzer] Add sink after construction of temporary with no-return destructor. The analyzer's CFG currently doesn't have nodes for calls to temporary destructors

[PATCH] D27955: Make CodeGenCXX/arm-swiftcall.cpp tolerate C++11

probinson created this revision. probinson added a reviewer: rjmccall. probinson added a subscriber: cfe-commits. Herald added subscribers: rengolin, aemerson. The test conjures up and returns a temp which has a struct type, and the struct has some empty/padding bytes in the middle. In C++03 the

[PATCH] D27180: Testbed and skeleton of a new expression parser

spyffe updated this revision to Diff 81991. spyffe marked 3 inline comments as done. spyffe added a comment. Applied Aleksei's comments, and integrated all the CMakeFiles fixes required to make the bots happy. Repository: rL LLVM https://reviews.llvm.org/D27180 Files: test/CMakeLists.txt

Re: r289754 - [c++1z] Permit constant evaluation of a call through a function pointer whose

Richard, Clang crashes for me on this code while doing "check-all". This change seems to introduce the assert. Can you take a look? BTW I'm not sure why bots are green. Do we build libcxx with bootstrap compiler? FAIL: libc++ :: std/experimental/string.view/string.view.find/find_last_of_pointer

r290135 - [Format] Remove dead code.

Author: d0k Date: Mon Dec 19 15:10:50 2016 New Revision: 290135 URL: http://llvm.org/viewvc/llvm-project?rev=290135&view=rev Log: [Format] Remove dead code. No functionality change. Modified: cfe/trunk/lib/Format/Encoding.h cfe/trunk/lib/Format/TokenAnalyzer.h cfe/trunk/lib/Format/Wh

[PATCH] D24812: Lit C++11 Compatibility Patch #11

tigerleapgorge updated the summary for this revision. tigerleapgorge updated this revision to Diff 81987. tigerleapgorge added a comment. Update Lit patch #11 to match latest Clang behavior 2 minor changes in this update 1. Back out test/CodeGenCXX/mangle-unnamed.cpp because it has already been

r290134 - [ASTReader] Sort RawComments before merging

Author: bruno Date: Mon Dec 19 15:06:06 2016 New Revision: 290134 URL: http://llvm.org/viewvc/llvm-project?rev=290134&view=rev Log: [ASTReader] Sort RawComments before merging `RawComments` are sorted by comparing underlying `SourceLocation`'s. This is done by comparing `FileID` and `Offset`; whe

Re: [libcxx] r289963 - [CMake] Put headers relative to clang

Sorry I didn't reply to this email over the weekend. I did push a fix in r290052. My weekend was just a bit nuts, so I didn't take the time to reply here. The bot went green after that fix. If there are other issues please let me know. -Chris > On Dec 16, 2016, at 5:58 PM, Evgenii Stepanov >

[PATCH] D27837: Add fix-it notes to the nullability consistency warning

jordan_rose accepted this revision. jordan_rose added a reviewer: jordan_rose. jordan_rose added a comment. This revision is now accepted and ready to land. Doug was okay with the idea and I trust Alex and Akira would have caught any major problems. Committed in r290132. Repository: rL LLVM

r290132 - Add fix-it notes to the nullability consistency warning.

Author: jrose Date: Mon Dec 19 14:58:20 2016 New Revision: 290132 URL: http://llvm.org/viewvc/llvm-project?rev=290132&view=rev Log: Add fix-it notes to the nullability consistency warning. This is especially important for arrays, since no one knows the proper syntax for putting qualifiers in arra

[PATCH] D27936: C++11 test cleanup: nonthrowing destructors

probinson created this revision. probinson added a reviewer: rsmith. probinson added a subscriber: cfe-commits. If a dtor has no interesting members, then it ends up being nothrow, which affects the generated IR. Modify some tests to tolerate this difference between C++03 and C++11. In C++11, a

[PATCH] D27165: Add format_dynamic_key_arg attribute to improve "-Wformat" warnings for functions that load the formatting string dynamically based on a key value

aaron.ballman added inline comments. Comment at: include/clang/Basic/Attr.td:862 +def FormatDynamicKeyArg : InheritableAttr { + let Spellings = [GCC<"format_dynamic_key_arg">]; + let Args = [IntArgument<"FormatIdx">]; arphaman wrote: > aaron.ballman wrote: > >

[PATCH] D27621: [clang-tidy] check to find declarations declaring more than one name

aaron.ballman added inline comments. Comment at: clang-tidy/readability/OneNamePerDeclarationCheck.cpp:22 + +const internal::VariadicDynCastAllOfMatcher tagDecl; + We may want to consider adding this to ASTMatchers.h at some point (can be done in a future patch)

[PATCH] D26750: [clang-tidy] Add modernize-use-default-member-init check

aaron.ballman added inline comments. Comment at: clang-tidy/modernize/UseDefaultMemberInitCheck.cpp:77-78 +static bool sameValue(const Expr *E1, const Expr *E2) { + E1 = ignoreUnaryPlus(getInitializer(E1->IgnoreImpCasts())); + E2 = ignoreUnaryPlus(getInitializer(E2->IgnoreImpCa

[PATCH] D26750: [clang-tidy] Add modernize-use-default-member-init check

malcolm.parsons added inline comments. Comment at: clang-tidy/modernize/UseDefaultMemberInitCheck.cpp:77-78 +static bool sameValue(const Expr *E1, const Expr *E2) { + E1 = ignoreUnaryPlus(getInitializer(E1->IgnoreImpCasts())); + E2 = ignoreUnaryPlus(getInitializer(E2->IgnoreImp

[PATCH] D26750: [clang-tidy] Add modernize-use-default-member-init check

aaron.ballman added inline comments. Comment at: clang-tidy/modernize/UseDefaultMemberInitCheck.cpp:77-78 +static bool sameValue(const Expr *E1, const Expr *E2) { + E1 = ignoreUnaryPlus(getInitializer(E1->IgnoreImpCasts())); + E2 = ignoreUnaryPlus(getInitializer(E2->IgnoreImpCa

[PATCH] D26750: [clang-tidy] Add modernize-use-default-member-init check

malcolm.parsons marked 9 inline comments as done. malcolm.parsons added inline comments. Comment at: clang-tidy/modernize/UseDefaultMemberInitCheck.cpp:77-78 +static bool sameValue(const Expr *E1, const Expr *E2) { + E1 = ignoreUnaryPlus(getInitializer(E1->IgnoreImpCasts())); +

[PATCH] D27832: Add -plugin-opt=sample-profile for thinLTO build.

bruno added inline comments. Comment at: lib/Driver/Tools.cpp:2211 + if (Arg *A = Args.getLastArg(options::OPT_fprofile_sample_use_EQ)) { +StringRef fname = A->getValue(); +if (!llvm::sys::fs::exists(fname)) "fname" -> "FName", "FileName", etc, or any na

[PATCH] D27872: [APFloat] Switch from (PPCDoubleDoubleImpl, IEEEdouble) layout to (IEEEdouble, IEEEdouble)

timshen updated this revision to Diff 81981. timshen added a comment. Remove more 'else' after return. https://reviews.llvm.org/D27872 Files: clang/test/CodeGen/ppc64-complex-parms.c llvm/include/llvm/ADT/APFloat.h llvm/lib/Support/APFloat.cpp llvm/test/CodeGen/PowerPC/fp128-bitcast-aft

[PATCH] D23325: [WIP] Binding of references to packed fields

aaron.ballman added inline comments. Comment at: include/clang/AST/Decl.h:2388 + /// its alignment is smaller than the alignment of its field type. + bool isPackedField(const ASTContext& context) const; + Should be `&Context` instead of `& context`. =

[PATCH] D27872: [APFloat] Switch from (PPCDoubleDoubleImpl, IEEEdouble) layout to (IEEEdouble, IEEEdouble)

timshen updated this revision to Diff 81980. timshen added a comment. Remove 'else' after return. Remove 'return' on void type. https://reviews.llvm.org/D27872 Files: clang/test/CodeGen/ppc64-complex-parms.c llvm/include/llvm/ADT/APFloat.h llvm/lib/Support/APFloat.cpp llvm/test/CodeGen/

[PATCH] D27410: Always issue vtables when generating coverage instrumentation

vsk added inline comments. Comment at: llvm/tools/clang/test/CodeGenCXX/vtable-coverage-gen.cpp:3 +// RUN: FileCheck %s < %t +// CHECK: @_ZTV8Category = linkonce_odr unnamed_addr constant {{.*}}, comdat, + vsk wrote: > ahatanak wrote: > > I'm not sure I understo

Re: [PATCH] D27180: Testbed and skeleton of a new expression parser

Reverted by 290130. I have new comments from Aleksei in https://reviews.llvm.org/D27180 . I'll apply those and update the patch. Sean > On Dec 19, 2016, at 9:48 AM, Sean Callanan wrote: > > David, > > thanks for keeping an eye on this and sorry for the bre

r290130 - Reverting r290004, r290006, r290010 pending review.

Author: spyffe Date: Mon Dec 19 13:15:43 2016 New Revision: 290130 URL: http://llvm.org/viewvc/llvm-project?rev=290130&view=rev Log: Reverting r290004, r290006, r290010 pending review. Removed: cfe/trunk/test/Import/ cfe/trunk/tools/clang-import-test/ Modified: cfe/trunk/test/CMakeLis

[PATCH] D27794: Make some diagnostic tests C++11 clean

probinson added a reviewer: rnk. probinson updated this revision to Diff 81977. probinson added a comment. Remove the OpenMP tests from this review (committed in r290128). +rnk who added test/Parser/backtrack-off-by-one.cpp originally. https://reviews.llvm.org/D27794 Files: test/FixIt/fixit.

[PATCH] D27410: Always issue vtables when generating coverage instrumentation

vsk added inline comments. Comment at: llvm/tools/clang/test/CodeGenCXX/vtable-coverage-gen.cpp:3 +// RUN: FileCheck %s < %t +// CHECK: @_ZTV8Category = linkonce_odr unnamed_addr constant {{.*}}, comdat, + ahatanak wrote: > I'm not sure I understood the purpose

[PATCH] D27180: Testbed and skeleton of a new expression parser

a.sidorin added a comment. Sorry for being late :( Comment at: cfe/trunk/tools/clang-import-test/clang-import-test.cpp:99 +llvm::errs() << LineString << '\n'; +std::string Space(LocColumn, ' '); +llvm::errs() << Space.c_str() << '\n'; I still think

[PATCH] D26544: [PPC] support for arithmetic builtins in the FE

amehsan closed this revision. amehsan added a comment. https://reviews.llvm.org/rL287872 https://reviews.llvm.org/D26544 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r290128 - Make a few OpenMP tests "C++11 clean."

Author: probinson Date: Mon Dec 19 12:43:26 2016 New Revision: 290128 URL: http://llvm.org/viewvc/llvm-project?rev=290128&view=rev Log: Make a few OpenMP tests "C++11 clean." This time trying to commit just the relevant 3 tests! Reviewed by abataev (in D27794) Modified: cfe/trunk/test/OpenM

[PATCH] D27569: [OpenCL] Enabling the usage of CLK_NULL_QUEUE as compare operand.

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! https://reviews.llvm.org/D27569 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman

r290125 - Revert another accidental bit

Author: probinson Date: Mon Dec 19 12:20:19 2016 New Revision: 290125 URL: http://llvm.org/viewvc/llvm-project?rev=290125&view=rev Log: Revert another accidental bit Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp URL: http://l

[PATCH] D26964: Handle more declarations in DeclContextPrinter to fix -print-decl-contexts crashes

mehdi_amini accepted this revision. mehdi_amini added a comment. This revision is now accepted and ready to land. Testing is in line with the existing testing. It is not great, but at the same time is a development/debugging tool right? Repository: rL LLVM https://reviews.llvm.org/D26964 _

r290121 - Undo accidental comit

Author: probinson Date: Mon Dec 19 12:00:45 2016 New Revision: 290121 URL: http://llvm.org/viewvc/llvm-project?rev=290121&view=rev Log: Undo accidental comit Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp cfe/trunk/test/CodeGenCXX/arm-swiftcall.cpp cfe/trunk/test/CodeGenCXX/d

[PATCH] D27815: [clang-tidy] Add obvious module for obvious bugs

aaron.ballman added a comment. In https://reviews.llvm.org/D27815#626440, @Prazek wrote: > The example of this kind of check is here: > https://reviews.llvm.org/D27806 > > I am not sure if it make sense to put it as clang warning. > > After a little bit of thinking I guess name "typos" would be

r290120 - Make a few OpenMP tests "C++11 clean."

Author: probinson Date: Mon Dec 19 11:58:09 2016 New Revision: 290120 URL: http://llvm.org/viewvc/llvm-project?rev=290120&view=rev Log: Make a few OpenMP tests "C++11 clean." Reviewed by abataev (in D27794) Modified: cfe/trunk/test/CodeGenCXX/arm-swiftcall.cpp cfe/trunk/test/CodeGenCXX/

[PATCH] D27806: [clang-tidy] Add obvious-invalid-range

aaron.ballman added inline comments. Comment at: clang-tidy/obvious/InvalidRangeCheck.cpp:20-36 +"std::for_each; std::find; std::find_if; std::find_end; " +"std::find_first_of; std::adjacent_find; std::count; std::count_if;" +"std::mismatch; std::equal; std::search; s

Re: [PATCH] D27180: Testbed and skeleton of a new expression parser

David, thanks for keeping an eye on this and sorry for the breach of process. Would having Vassil approve the changelist (https://reviews.llvm.org/D27180 ) be appropriate? Let's say if he has any concerns or can't get to it by tomorrow, we revert my patches since

Re: [PATCH] D27683: Prepare PrettyStackTrace for LLDB adoption

That would require making LLDB crash and collecting the relevant crash log data out of the user's own logs. This isn't impossible – but additionally the generation of that log is asynchronous and you don't quite know when it'll land. Would you be all right with a more restricted macOS-only unit

[PATCH] D26454: Implement no_sanitize_address for global vars

dougk added a comment. I think it probably works to have the attribute appertain to any sanitizer. I did not know that it did, so I conservatively assumed that it didn't. I'll go ahead and make things consistent, and confirm that it dtrt. https://reviews.llvm.org/D26454 __

Re: [PATCH] D23921: Remove va_start diagnostic false positive with enumerations

On Fri, Dec 16, 2016 at 7:00 AM, Attila Török via Phabricator wrote: > torokati44 added a comment. > > I see this has been reverted in r281612, but I can no longer access the build > log serving as a reason linked in the message: > https://www.mail-archive.com/cfe-commits@lists.llvm.org/msg35386

[PATCH] D21298: [Clang-tidy] delete null check

aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM, with one nit. You should wait for @alexfh to sign off before committing though, since he requested changes. Comment at: clang-tidy/readability/DeleteNullPointerCheck.cpp:53 + "'if' statement is unn

Re: [PATCH] D27180: Testbed and skeleton of a new expression parser

On 19/12/16 17:55, David Blaikie wrote: On Thu, Dec 15, 2016 at 2:18 PM Sean Callanan via Phabricator via cfe-commits > wrote: spyffe updated this revision to Diff 81661. spyffe marked 2 inline comments as done. spyffe added a comment. Heral

[PATCH] D26750: [clang-tidy] Add modernize-use-default-member-init check

aaron.ballman added inline comments. Comment at: clang-tidy/modernize/UseDefaultMemberInitCheck.cpp:33-34 + case Type::STK_Floating: + case Type::STK_IntegralComplex: + case Type::STK_FloatingComplex: +return "0.0"; Do these require a literal suffix to avo

Re: [PATCH] D27763: Debug Info: Modified DIBuilder::createCompileUnit() to take DIFile instead of FileName and Directory. (Clang part)

Ah, sure - no worries. Good to mention/link to the other change, etc, in the future. (if the changes on the clang side are trivial enough to not need review, may not need to send them out for review either - or could include them as an addendum to the llvm change ("oh, and here's what it looks like

[PATCH] D27621: [clang-tidy] check to find declarations declaring more than one name

firolino updated this revision to Diff 81964. firolino added a comment. Added support for new test cases: int S::*mdpa1[2] = {&S::a, &S::a}, var1 = 0; // CHECK-MESSAGES: :[[@LINE-1]]:9: warning: declaration statement can be split // CHECK-FIXES: {{^}}int S::*mdpa1[2] = {&S::a, &S::a

RE: [PATCH] D27763: Debug Info: Modified DIBuilder::createCompileUnit() to take DIFile instead of FileName and Directory. (Clang part)

It was approved by Reid. This patch had two parts, he did not stamp the change in Clang, but he did stamp the change for LLVM. https://reviews.llvm.org/D27762 I assumed that this means a green light to commit. Did I misinterpret the rules? Thanks, Amjad From: David Blaikie [mailto:dblai...@gmai

Re: [PATCH] D27683: Prepare PrettyStackTrace for LLDB adoption

Test coverage? On Tue, Dec 13, 2016 at 2:39 PM Sean Callanan via Phabricator via cfe-commits wrote: > spyffe retitled this revision from "Fix the linkage for > __crashtracer_info__" to "Prepare PrettyStackTrace for LLDB adoption". > spyffe updated the summary for this revision. > spyffe updated

r290113 - [libclang] Revert part of r290025, "Remove the 'extern "C"' blocks from the implementation files."

Author: chapuni Date: Mon Dec 19 10:50:43 2016 New Revision: 290113 URL: http://llvm.org/viewvc/llvm-project?rev=290113&view=rev Log: [libclang] Revert part of r290025, "Remove the 'extern "C"' blocks from the implementation files." mingw32-ld complains missing symbols in exports, Cannot expo

Re: [PATCH] D27763: Debug Info: Modified DIBuilder::createCompileUnit() to take DIFile instead of FileName and Directory. (Clang part)

Was this change approved by anyone? Generally once it's sent for review, you should wait until it's approved before committing (the assumption being, if you sent it for review it's because it needed review) On Wed, Dec 14, 2016 at 12:49 PM Amjad Aboud via Phabricator via cfe-commits wrote: > Thi

Re: [PATCH] D27180: Testbed and skeleton of a new expression parser

On Thu, Dec 15, 2016 at 2:18 PM Sean Callanan via Phabricator via cfe-commits wrote: > spyffe updated this revision to Diff 81661. > spyffe marked 2 inline comments as done. > spyffe added a comment. > Herald added a subscriber: jgosnell. > > Applied Vassil and Vedant's comments. I will commit t

[PATCH] D23325: [WIP] Binding of references to packed fields

rogfer01 updated this revision to Diff 81955. rogfer01 added a comment. Rebase to current trunk https://reviews.llvm.org/D23325 Files: include/clang/AST/Decl.h include/clang/AST/Expr.h include/clang/AST/Stmt.h include/clang/Basic/DiagnosticSemaKinds.td include/clang/Basic/Specifiers.h

[PATCH] D27920: [find-all-symbols] Index partial template specializations.

hokein created this revision. hokein added a reviewer: ioeric. hokein added subscribers: bkramer, cfe-commits. Fix no std::function index. https://reviews.llvm.org/D27920 Files: include-fixer/find-all-symbols/FindAllSymbols.cpp unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cp

[PATCH] D26454: Implement no_sanitize_address for global vars

aaron.ballman added a comment. In https://reviews.llvm.org/D26454#609578, @dougk wrote: > Suppression of sanitizing is necessary if the variable is magically a > memory-mapped device I/O address. > The linker can arrange for this to be the case using fancy scripts, or even > just as simple as

[PATCH] D27641: DebugInfo: Added support for Checksum debug info feature (Clang part)

aaboud updated this revision to Diff 81953. aaboud marked 3 inline comments as done. aaboud added a comment. Addressed comments by Reid and David. https://reviews.llvm.org/D27641 Files: lib/CodeGen/CGDebugInfo.cpp lib/CodeGen/CGDebugInfo.h test/CodeGen/Inputs/debug-info-file-checksum.c

r290110 - [ARM] Add missing -backend-option for -arm-execute-only

Author: prakhar Date: Mon Dec 19 09:43:33 2016 New Revision: 290110 URL: http://llvm.org/viewvc/llvm-project?rev=290110&view=rev Log: [ARM] Add missing -backend-option for -arm-execute-only Modified: cfe/trunk/lib/Driver/Tools.cpp cfe/trunk/test/Driver/arm-execute-only.c Modified: cfe/tr

[PATCH] D27165: Add format_dynamic_key_arg attribute to improve "-Wformat" warnings for functions that load the formatting string dynamically based on a key value

arphaman updated this revision to Diff 81952. arphaman added a comment. Update to fix a test failure. Repository: rL LLVM https://reviews.llvm.org/D27165 Files: include/clang/Analysis/Analyses/FormatString.h include/clang/Basic/Attr.td include/clang/Basic/AttrDocs.td lib/Analysis/Pri

[PATCH] D27165: Add format_dynamic_key_arg attribute to improve "-Wformat" warnings for functions that load the formatting string dynamically based on a key value

arphaman added inline comments. Comment at: include/clang/Basic/Attr.td:862 +def FormatDynamicKeyArg : InheritableAttr { + let Spellings = [GCC<"format_dynamic_key_arg">]; + let Args = [IntArgument<"FormatIdx">]; aaron.ballman wrote: > Does GCC support this att

[PATCH] D27165: Add format_dynamic_key_arg attribute to improve "-Wformat" warnings for functions that load the formatting string dynamically based on a key value

arphaman updated this revision to Diff 81950. arphaman marked an inline comment as done. arphaman added a comment. The updated patch renames the attribute to `loads_format_specifier_value_using_key` and addresses Aaron's comments Repository: rL LLVM https://reviews.llvm.org/D27165 Files:

[PATCH] D27917: [OpenCL] Improve diagnostics for double type

yaxunl added inline comments. Comment at: lib/Sema/SemaType.cpp:1505 + S.getLangOpts().OpenCLVersion < 120 && + !S.getOpenCLOptions().cl_khr_fp64) { +S.Diag(DS.getTypeSpecTypeLoc(), diag::err_type_requires_extension) Plea

  1   2   >