r345370 - Reapply: [Driver] Use forward slashes in most linker arguments

Author: mstorsjo Date: Fri Oct 26 00:01:59 2018 New Revision: 345370 URL: http://llvm.org/viewvc/llvm-project?rev=345370&view=rev Log: Reapply: [Driver] Use forward slashes in most linker arguments libtool inspects the output of $CC -v to detect what object files and libraries are linked in by de

[PATCH] D53066: [Driver] Use forward slashes in most linker arguments

This revision was automatically updated to reflect the committed changes. Closed by commit rL345370: Reapply: [Driver] Use forward slashes in most linker arguments (authored by mstorsjo, committed by ). Changed prior to commit: https://reviews.llvm.org/D53066?vs=170989&id=171253#toc Repository

r345371 - Update the example of BS_Stroustrup to match what is done by clang-format

Author: sylvestre Date: Fri Oct 26 00:25:37 2018 New Revision: 345371 URL: http://llvm.org/viewvc/llvm-project?rev=345371&view=rev Log: Update the example of BS_Stroustrup to match what is done by clang-format Summary: reported here https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=911561 clang-

[PATCH] D53520: Update the example of BS_Stroustrup to match what is done by clang-format

This revision was automatically updated to reflect the committed changes. Closed by commit rL345371: Update the example of BS_Stroustrup to match what is done by clang-format (authored by sylvestre, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.ll

[PATCH] D53654: [clang] Improve ctor initializer completions.

kadircet updated this revision to Diff 171259. kadircet added a comment. - Add a generic constructor if found none. - format && add comments. - Add generic constructors to tests. Repository: rC Clang https://reviews.llvm.org/D53654 Files: include/clang/Sema/CodeCompleteConsumer.h lib/Sem

r345372 - Revert "Reapply: [Driver] Use forward slashes in most linker arguments"

Author: mstorsjo Date: Fri Oct 26 01:33:29 2018 New Revision: 345372 URL: http://llvm.org/viewvc/llvm-project?rev=345372&view=rev Log: Revert "Reapply: [Driver] Use forward slashes in most linker arguments" This reverts commit r345370, as it uncovered even more issues in tests with partial/incons

[PATCH] D53066: [Driver] Use forward slashes in most linker arguments

mstorsjo reopened this revision. mstorsjo added a comment. This revision is now accepted and ready to land. Nope, still no really working properly with all the tests out there (I had only run the Driver subdirectory since I'm only testing in a very underpowered VM, and it had skipped the cuda/hi

[PATCH] D53751: [ASTImporter] Added Import functions for transition to new API.

balazske created this revision. Herald added subscribers: cfe-commits, Szelethus, martong, dkrupp. Herald added a reviewer: a.sidorin. These Import_New functions should be used in the ASTImporter, and the old Import functions should not be used. Later the Import_New should be renamed to Import aga

[PATCH] D53655: [ASTImporter] Fix redecl chain of classes and class templates

balazske added inline comments. Comment at: lib/AST/ASTImporter.cpp:5054 +if (!ToTemplated->getPreviousDecl()) { + auto *PrevTemplated = FoundByLookup->getTemplatedDecl()->getMostRecentDecl(); + if (ToTemplated != PrevTemplated) This is a long line

[PATCH] D53738: [Fixed Point Arithmetic] Fixed Point Addition

ebevhan added a comment. I think I've already expressed that I'm not at all a fan of encoding the full-precision logic in the operations themselves and not explicitly in the AST. We already have (well, not yet, but we will) routines for emitting conversions to/from/between fixed-point types of

[PATCH] D53751: [ASTImporter] Added Import functions for transition to new API.

balazske updated this revision to Diff 171278. balazske added a comment. - Corrected Import_New(const Attr *) Repository: rC Clang https://reviews.llvm.org/D53751 Files: include/clang/AST/ASTImporter.h lib/AST/ASTImporter.cpp Index: lib/AST/ASTImporter.cpp ==

[PATCH] D52670: [clang-tidy] Add new 'readability-uppercase-literal-suffix' check (CERT DCL16-C, MISRA C:2012, 7.3, MISRA C++:2008, 2-13-4)

lebedev.ri updated this revision to Diff 171279. lebedev.ri added a comment. This revision is now accepted and ready to land. Finally finished creducing the issue i wasn't able to understand even after https://reviews.llvm.org/D53719+https://reviews.llvm.org/D53723, and it turned out to be `hasPa

Re: r345330 - Add MS ABI mangling for operator<=>.

This commit, specifically the changes made to CodeGenCXX/cxx2a-three-way-comparison.cpp, are failing on all the Arm and AArch64 builders that run check-clang and some of the other non-X86 builders as well like S390 and PPC. It seems to be the // RUN: not %clang_cc1 -std=c++2a -emit-llvm %s -o - -t

r345344 - [AArch64] Implement FP16FML intrinsics

Author: bryanpkc Date: Thu Oct 25 16:47:00 2018 New Revision: 345344 URL: http://llvm.org/viewvc/llvm-project?rev=345344&view=rev Log: [AArch64] Implement FP16FML intrinsics Generate the FP16FML intrinsics into arm_neon.h (AArch64 only for now). Add two new type modifiers to NeonEmitter to handle

[PATCH] D53754: [Analyzer] Skip symbolic regions based on conjured symbols in comparison of the containers of iterators

baloghadamsoftware created this revision. baloghadamsoftware added a reviewer: NoQ. baloghadamsoftware added a project: clang. Herald added subscribers: donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, whisperity. Herald added a reviewer: george.karpenkov. Checking whether two

[PATCH] D53755: [ASTImporter] Remove import of definition from GetAlreadyImportedOrNull

martong created this revision. Herald added subscribers: cfe-commits, Szelethus, dkrupp, rnkovacs. Herald added a reviewer: a.sidorin. a_sidorin Repository: rC Clang https://reviews.llvm.org/D53755 Files: include/clang/AST/ASTImporter.h lib/AST/ASTImporter.cpp lib/AST/ExternalASTMerger

[PATCH] D53025: [clang-tidy] implement new check for const return types.

ymandel added a comment. Thanks! What's the next step in the process? Does someone need to approve this change? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53025 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://li

[PATCH] D53755: [ASTImporter] Remove import of definition from GetAlreadyImportedOrNull

martong added a comment. This is about to make `GetAlreadyImportedOrNull` to be a const function, as it should be naturally. This is a query function which should not initiate other imports. Repository: rC Clang https://reviews.llvm.org/D53755

[PATCH] D53025: [clang-tidy] implement new check for const return types.

lebedev.ri added inline comments. Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:54 + +namespace { + Extend the namespace /\ above, so that function is also covered? Comment at: clang-tidy/readability/ConstValueReturnCheck.h:19-20

[PATCH] D52615: Handle -fsanitize-address-poison-class-member-array-new-cookie in the driver and propagate it to cc1

filcab added a comment. In https://reviews.llvm.org/D52615#1276938, @rjmccall wrote: > In https://reviews.llvm.org/D52615#1275946, @filcab wrote: > > > In https://reviews.llvm.org/D52615#1272725, @rjmccall wrote: > > > > > In https://reviews.llvm.org/D52615#1266320, @filcab wrote: > > > > > > > I

[PATCH] D53757: [ASTImporter] Changed use of Import to Import_New in ASTNodeImporter.

balazske created this revision. Herald added subscribers: cfe-commits, Szelethus, martong, dkrupp. Herald added a reviewer: a.sidorin. Herald added a reviewer: a.sidorin. Repository: rC Clang https://reviews.llvm.org/D53757 Files: lib/AST/ASTImporter.cpp Index: lib/AST/ASTImporter.cpp =

[PATCH] D53754: [Analyzer] Skip symbolic regions based on conjured symbols in comparison of the containers of iterators

Szelethus added a comment. I wonder whether a method in `MemRegion` called `isSameRegion` or `isSurelySameRegion` would be better. I think it's likely that there are (or will be) checkers that would do similar things. Maybe something like this? bool MemRegion::isSurelySameRegion(const MemReg

[PATCH] D53025: [clang-tidy] implement new check for const return types.

Eugene.Zelenko added inline comments. Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:54 + +namespace { + lebedev.ri wrote: > Extend the namespace /\ above, so that function is also covered? Anonymous namespaces are only for class/struct declarations

r345380 - Revert r345330 "Add MS ABI mangling for operator<=>."

Author: hans Date: Fri Oct 26 06:05:45 2018 New Revision: 345380 URL: http://llvm.org/viewvc/llvm-project?rev=345380&view=rev Log: Revert r345330 "Add MS ABI mangling for operator<=>." The generated MS manglings differ between 32- and 64-bit, and the test only expects the latter. See also the com

[PATCH] D52670: [clang-tidy] Add new 'readability-uppercase-literal-suffix' check (CERT DCL16-C, MISRA C:2012, 7.3, MISRA C++:2008, 2-13-4)

lebedev.ri added a comment. Rerun on clang-tools-extra+lld+clang codebase, nothing broken. I'm gonna reland. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52670 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

Re: r345330 - Add MS ABI mangling for operator<=>.

It seems the generated MS mangling differs between 32- and 64-bit, and the test expects the latter. Let's revert in the meantime.. r345380 On Fri, Oct 26, 2018 at 1:10 PM, Peter Smith via cfe-commits wrote: > This commit, specifically the changes made to > CodeGenCXX/cxx2a-three-way-comparison.

[clang-tools-extra] r345381 - [clang-tidy] Re-commit: Add new 'readability-uppercase-literal-suffix' check (CERT DCL16-C, MISRA C:2012, 7.3, MISRA C++:2008, 2-13-4)

Author: lebedevri Date: Fri Oct 26 06:09:27 2018 New Revision: 345381 URL: http://llvm.org/viewvc/llvm-project?rev=345381&view=rev Log: [clang-tidy] Re-commit: Add new 'readability-uppercase-literal-suffix' check (CERT DCL16-C, MISRA C:2012, 7.3, MISRA C++:2008, 2-13-4) Summary: Detects when the

[PATCH] D53025: [clang-tidy] implement new check for const return types.

aaron.ballman added inline comments. Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107 +diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified " + "hindering compiler optimizations") +<< Def->getRetur

[PATCH] D52670: [clang-tidy] Add new 'readability-uppercase-literal-suffix' check (CERT DCL16-C, MISRA C:2012, 7.3, MISRA C++:2008, 2-13-4)

This revision was automatically updated to reflect the committed changes. Closed by commit rL345381: [clang-tidy] Re-commit: Add new 'readability-uppercase-literal-suffix' check… (authored by lebedevri, committed by ). Changed prior to commit: https://reviews.llvm.org/D52670?vs=171279&id=17129

[PATCH] D53755: [ASTImporter] Remove import of definition from GetAlreadyImportedOrNull

balazske added inline comments. Comment at: lib/AST/ASTImporter.cpp:7716 } } This can be simplified by removing brace characters and removing `ToD`. Repository: rC Clang https://reviews.llvm.org/D53755 ___

[PATCH] D53751: [ASTImporter] Added Import functions for transition to new API.

balazske updated this revision to Diff 171292. balazske added a comment. - Added missing Import_New with Selector and DeclarationName. Repository: rC Clang https://reviews.llvm.org/D53751 Files: include/clang/AST/ASTImporter.h lib/AST/ASTImporter.cpp Index: lib/AST/ASTImporter.cpp =

[PATCH] D53621: Support for groups of attributes in #pragma clang attribute

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from some small nits. Comment at: clang/docs/LanguageExtensions.rst:2666 - #pragma clang attribute push(__attribute__((annotate("custom"))), apply

[PATCH] D53717: [AST] Only store the needed data in ForStmt.

riccibruno added a comment. In https://reviews.llvm.org/D53717#1276388, @rsmith wrote: > In https://reviews.llvm.org/D53717#1276245, @rsmith wrote: > > > Do you have evidence that this complexity is worthwhile? (I wouldn't > > imagine we have very many `ForStmt`s per translation unit, so saving

[PATCH] D53025: [clang-tidy] implement new check for const return types.

ymandel updated this revision to Diff 171293. ymandel marked 4 inline comments as done. ymandel added a comment. Reword comment on the check, for clarity. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53025 Files: clang-tidy/readability/CMakeLists.txt clang-tidy/readabilit

[PATCH] D53025: [clang-tidy] implement new check for const return types.

ymandel added inline comments. Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107 +diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified " + "hindering compiler optimizations") +<< Def->getReturnType(

[PATCH] D53457: clang-cl: Add "/clang:" pass-through arg support.

hans added a comment. One note about flag ordering: the /clang: flags are concatenated to the end of the argument list, so in cases where the last flag wins, the /clang: flags will be chosen regardless of their order relative to other flags on the driver command line. This seems a little

[PATCH] D51949: [clang-tidy] new check 'readability-isolate-declaration'

lebedev.ri added inline comments. Comment at: test/clang-tidy/readability-isolate-declaration.cpp:233 + int member1, member2; + // TODO: Transform FieldDecl's as well +}; Comment is misleading. Transform == fixit, at least for me. But they are not even diagnose

[PATCH] D51949: [clang-tidy] new check 'readability-isolate-declaration'

aaron.ballman added a comment. Mostly minor nits that clean up wording and comments. Comment at: clang-tidy/readability/IsolateDeclarationCheck.cpp:52 + const LangOptions &LangOpts) { + assert(Indirections >= 0 && "Indirections must

[PATCH] D53200: [OpenCL] Fix serialization of OpenCLExtensionDecls

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! Comment at: test/SemaOpenCL/extension-begin.cl:43 g(0); // expected-error {{no matching function for call to 'g'}} -// expected-note@-26 {{cand

[PATCH] D52742: [analyzer][PlistMacroExpansion] Part 1.: New expand-macros flag

Szelethus planned changes to this revision. Szelethus added a comment. @xazax.hun observed that the way `diagnostics` looks like is this: diagnostics report 1 notes macro_expansions path executed_lines report 2 ... Bt, if I didn't insist on this struct

[PATCH] D53025: [clang-tidy] implement new check for const return types.

aaron.ballman added inline comments. Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107 +diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified " + "hindering compiler optimizations") +<< Def->getRetur

[PATCH] D53763: [libc++] [test] Fix logic error in tests; enable for MSVC previews

CaseyCarter created this revision. CaseyCarter added reviewers: EricWF, mclow.lists. Fairly straightforward: these tests were written without an implementation of `<=>`, and they're incorrectly testing that `0 <=> foo` has the behavior that is required for `foo <=> 0`. https://reviews.llvm.org

[PATCH] D53764: [OpenCL] Enable address spaces for references in C++

Anastasia created this revision. Anastasia added reviewers: rjmccall, yaxunl. I first enabled AS deduction for references that allowed to inherit the right conversion diagnostics based on qualification conversion rules implemented earlier for the pointer type. Then in order to tests the deducti

[PATCH] D53764: [OpenCL] Enable address spaces for references in C++

Anastasia added inline comments. Comment at: lib/AST/Expr.cpp:1609 case CK_AddressSpaceConversion: -assert(getType()->isPointerType() || getType()->isBlockPointerType()); -assert(getSubExpr()->getType()->isPointerType() || - getSubExpr()->getType()->isBlockPo

[PATCH] D53448: [OpenMP][NVPTX] Use single loops when generating code for distribute parallel for

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG, with a nit Comment at: lib/Sema/SemaOpenMP.cpp:5308 + +// Build IV <= PrevEUB to be used in parallel for is in combination with +// a distribute directive with

[PATCH] D51484: [OpenCL] Add support of cl_intel_device_side_avc_motion_estimation extension

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! Comment at: test/SemaOpenCL/intel-subgroup-avc-ext-types.cl:5 + +// All intel_sub_group_avc_* types can only be used as argument or return value +// of bu

[PATCH] D53023: Prototype OpenCL BIFs using Tablegen

asavonic added a comment. > TODO > Measure the performance / size impact I looks like the total size of OpenCLBuiltinDecl table should be ~450KB (for ~15000 functions). I guess it can be reduced furthermore by: 1. Replacing return type with an index (you've mentioned this in TODO). 2. Replace e

[PATCH] D44100: [ASTImporter] Reorder fields after structure import is finished

martong added a comment. Ping. This is still not committed, however, essential for ctu analyse any C++ project. Aleksei, I am happy to commit it for you if you don't have time, just let me know. Repository: rC Clang https://reviews.llvm.org/D44100

[PATCH] D53023: Prototype OpenCL BIFs using Tablegen

asavonic added inline comments. Comment at: lib/Sema/SemaExpr.cpp:2247 if (R.empty() && HasTrailingLParen && II && !getLangOpts().CPlusPlus) { -NamedDecl *D = ImplicitlyDefineFunction(NameLoc, *II, S); -if (D) R.addDecl(D); +if (getLangOpts().OpenCL) { + auto

[PATCH] D53443: [OpenMP][NVPTX] Enable default scheduling for parallel for in non-SPMD cases.

gtbercea updated this revision to Diff 171310. gtbercea added a comment. Add test. Repository: rC Clang https://reviews.llvm.org/D53443 Files: lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp test/OpenMP/nvptx_parallel_for_codegen.cpp Index: test/OpenMP/nvptx_parallel_for_codegen.cpp

[PATCH] D53693: [ASTImporter] Typedef import brings in the complete type

a_sidorin added a comment. Hi Gabor, Thank you for the patch. I looks reasonable but I have some questions. Comment at: lib/AST/ASTImporter.cpp:2310 +D->getUnderlyingType(), FoundTypedef->getUnderlyingType())) { + QualType FromUT = D->getUnderlyingType(

[PATCH] D53443: [OpenMP][NVPTX] Enable default scheduling for parallel for in non-SPMD cases.

ABataev added inline comments. Comment at: test/OpenMP/nvptx_parallel_for_codegen.cpp:44 +// CHECK: call void @__kmpc_kernel_prepare_parallel( +// CHECK: call void @__kmpc_begin_sharing_variables(i8*** %shared_arg_refs, i64 2) +// CHECK: call void @llvm.nvvm.barrier0() -

[PATCH] D44100: [ASTImporter] Reorder fields after structure import is finished

a_sidorin added a comment. Hi Gabor, I have tried to rebase it last week but there were some test failures. I hope to fix them this weekend and finally commit the patch. Repository: rC Clang https://reviews.llvm.org/D44100 ___ cfe-commits mailin

r345398 - [Fixed Point Arithmetic] Refactor fixed point casts

Author: bjope Date: Fri Oct 26 09:12:12 2018 New Revision: 345398 URL: http://llvm.org/viewvc/llvm-project?rev=345398&view=rev Log: [Fixed Point Arithmetic] Refactor fixed point casts Summary: - Added names for some emitted values (such as "tobool" for the result of a cast to boolean). - Replac

[PATCH] D53738: [Fixed Point Arithmetic] Fixed Point Addition

bjope added inline comments. Comment at: clang/lib/CodeGen/CGExprScalar.cpp:3208 + if (LHSWidth < CommonWidth) +LHS = LHSSign ? Builder.CreateSExt(LHS, CommonTy) + : Builder.CreateZExt(LHS, CommonTy); `LHS = Builder.CreateIntCast(LHS, Common

[PATCH] D53707: [Fixed Point Arithmetic] Refactor fixed point casts

This revision was automatically updated to reflect the committed changes. Closed by commit rC345398: [Fixed Point Arithmetic] Refactor fixed point casts (authored by bjope, committed by ). Changed prior to commit: https://reviews.llvm.org/D53707?vs=171113&id=171311#toc Repository: rC Clang

[PATCH] D52676: [clang-format] tweaked another case of lambda formatting

oleg.smolsky added a comment. Folks, are there any other comments/suggestions? Repository: rC Clang https://reviews.llvm.org/D52676 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D53200: [OpenCL] Fix serialization of OpenCLExtensionDecls

AlexeySachkov added inline comments. Comment at: test/SemaOpenCL/extension-begin.cl:43 g(0); // expected-error {{no matching function for call to 'g'}} -// expected-note@-26 {{candidate unavailable as it requires OpenCL extension 'my_ext' to be disabled}} -//

[PATCH] D53768: Add VerboseOutputStream to CompilerInstance

scott.linder created this revision. scott.linder added a reviewer: rsmith. Herald added a subscriber: cfe-commits. An attempt at removing one instance of a hardcoded output stream in `CompilerInstance::ExecuteAction`. There are still other cases of output being hard-coded to standard streams in

[PATCH] D53417: [Clang][Sema][PowerPC] Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error

hubert.reinterpretcast added inline comments. Comment at: clang/lib/Sema/SemaOverload.cpp:3913 +for (auto Type : Types) { + if (S.Context.getCanonicalType(Type)->getTypeClass() != Type::Vector) +return false; wuzish wrote: > hubert.re

[PATCH] D53770: Support g++ headers in include/g++

greened created this revision. greened added reviewers: danalbert, dlj, atanasyan, phosek. Herald added a subscriber: cfe-commits. Some gcc installations put C++ headers in PREFIX/include/g++ without indicating a gcc version at all. Typically this is because the version is encoded somewhere in

[PATCH] D53771: [clang-tidy] Avoid C arrays check

lebedev.ri created this revision. lebedev.ri added reviewers: aaron.ballman, JonasToth, alexfh, hokein, xazax.hun. lebedev.ri added a project: clang-tools-extra. Herald added subscribers: rnkovacs, mgorny. lebedev.ri edited the summary of this revision. PR39224

[PATCH] D53025: [clang-tidy] implement new check for const return types.

ymandel marked 2 inline comments as done. ymandel added inline comments. Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107 +diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified " + "hindering compiler optim

[PATCH] D53417: [Clang][Sema][PowerPC] Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error

hubert.reinterpretcast added inline comments. Comment at: clang/lib/Sema/SemaOverload.cpp:3913 +for (auto Type : Types) { + if (S.Context.getCanonicalType(Type)->getTypeClass() != Type::Vector) +return false; hubert.reinterpretcast wr

[PATCH] D53771: [clang-tidy] Avoid C arrays check

JonasToth added a comment. `modernize` would be a fitting module, wouldn't it? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53771 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listin

[PATCH] D53771: [clang-tidy] Avoid C arrays check

JonasToth added inline comments. Comment at: docs/clang-tidy/checks/list.rst:13 abseil-redundant-strcat-calls - abseil-string-find-startswith abseil-str-cat-append spurious change Comment at: docs/clang-tidy/checks/list.rst:154 hi

[PATCH] D50860: [libc++][test] Remove non-portable assumption that thread's constructor allocates with ::new

CaseyCarter added a comment. I'll put this explanation in the comments and push a change. Comment at: test/std/thread/thread.threads/thread.thread.class/thread.thread.constr/F.pass.cpp:121 //allocations. +int numAllocs; + EricWF wrote: > Why? `main` non-p

[PATCH] D53200: [OpenCL] Fix serialization of OpenCLExtensionDecls

Anastasia added inline comments. Comment at: test/SemaOpenCL/extension-begin.cl:43 g(0); // expected-error {{no matching function for call to 'g'}} -// expected-note@-26 {{candidate unavailable as it requires OpenCL extension 'my_ext' to be disabled}} -// expe

[PATCH] D53206: Allow padding checker to traverse simple class hierarchies

tekknolagi added inline comments. Comment at: lib/StaticAnalyzer/Checkers/PaddingChecker.cpp:78-81 +// We need to be looking at a definition, not just any pointer to the +// declaration. +if (!(RD = RD->getDefinition())) + return; tekknolagi wrot

[PATCH] D53206: Allow padding checker to traverse simple class hierarchies

NoQ added inline comments. Comment at: lib/StaticAnalyzer/Checkers/PaddingChecker.cpp:78-81 +// We need to be looking at a definition, not just any pointer to the +// declaration. +if (!(RD = RD->getDefinition())) + return; tekknolagi wrote: > te

[PATCH] D53772: [NFC][OpenMP] Add new test for parallel for code generation.

gtbercea created this revision. gtbercea added reviewers: ABataev, caomhin. Herald added subscribers: cfe-commits, guansong. This is a simple test of the parallel for code generation. It will be used to showcase the change introduced by patch https://reviews.llvm.org/D53443. Repository: rC Cl

[PATCH] D53772: [NFC][OpenMP] Add new test for parallel for code generation.

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG Repository: rC Clang https://reviews.llvm.org/D53772 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bi

r345417 - [NFC][OpenMP] Add new test for parallel for code generation.

Author: gbercea Date: Fri Oct 26 11:59:52 2018 New Revision: 345417 URL: http://llvm.org/viewvc/llvm-project?rev=345417&view=rev Log: [NFC][OpenMP] Add new test for parallel for code generation. Summary: This is a simple test of the parallel for code generation. It will be used to showcase the c

[PATCH] D53772: [NFC][OpenMP] Add new test for parallel for code generation.

This revision was automatically updated to reflect the committed changes. Closed by commit rC345417: [NFC][OpenMP] Add new test for parallel for code generation. (authored by gbercea, committed by ). Repository: rC Clang https://reviews.llvm.org/D53772 Files: test/OpenMP/nvptx_parallel_for_

[PATCH] D53025: [clang-tidy] implement new check for const return types.

aaron.ballman added inline comments. Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107 +diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified " + "hindering compiler optimizations") +<< Def->getRetur

[PATCH] D53771: [clang-tidy] Avoid C arrays check

Eugene.Zelenko added a comment. May be this check belongs to modernize? Comment at: docs/clang-tidy/checks/misc-avoid-c-arrays.rst:6 + +Find C-style array delarations and recommend to use ``std::array<>``. +All types of C arrays are diagnosed. Finds.

[clang-tools-extra] r345418 - [clang-doc] Switch to default to all-TUs executor

Author: juliehockett Date: Fri Oct 26 12:11:34 2018 New Revision: 345418 URL: http://llvm.org/viewvc/llvm-project?rev=345418&view=rev Log: [clang-doc] Switch to default to all-TUs executor Since we generally want to document a whole project, not just one file. Differential Revision: https://revi

[PATCH] D53170: [clang-doc] Switch to default to all-TUs executor

This revision was automatically updated to reflect the committed changes. Closed by commit rCTE345418: [clang-doc] Switch to default to all-TUs executor (authored by juliehockett, committed by ). Changed prior to commit: https://reviews.llvm.org/D53170?vs=169414&id=171331#toc Repository: rCT

[PATCH] D53771: [clang-tidy] Avoid C arrays check

lebedev.ri added inline comments. Comment at: test/clang-tidy/misc-avoid-c-arrays.cpp:14 + +template +class array { JonasToth wrote: > Please add a case with templates, where `T` itself is the array type, maybe > there are other fancy template tricks that could

[PATCH] D53605: [AST] Refactor PredefinedExpr

riccibruno updated this revision to Diff 171326. riccibruno marked 4 inline comments as done. riccibruno retitled this revision from "[AST] Pack PredefinedExpr" to "[AST] Refactor PredefinedExpr". riccibruno edited the summary of this revision. riccibruno added a comment. Address rjmcall's commen

[PATCH] D53607: [AST] Only store the needed data in IfStmt.

riccibruno updated this revision to Diff 171332. riccibruno added a reviewer: rsmith. riccibruno added a comment. Add a flag to the output of -ast-dump indicating which sub-statement `IfStmt` is storing. This removes the ambiguity in the output of -ast-dump and addresses rsmith's comment in https:

r345419 - PR26547: alignof should return ABI alignment, not preferred alignment

Author: rsmith Date: Fri Oct 26 12:26:45 2018 New Revision: 345419 URL: http://llvm.org/viewvc/llvm-project?rev=345419&view=rev Log: PR26547: alignof should return ABI alignment, not preferred alignment Summary: - Add `UETT_PreferredAlignOf` to account for the difference between `__alignof` and

[PATCH] D53207: Fix bug 26547 - alignof should return ABI alignment, not preferred alignment

This revision was automatically updated to reflect the committed changes. Closed by commit rL345419: PR26547: alignof should return ABI alignment, not preferred alignment (authored by rsmith, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.

[PATCH] D53770: Support g++ headers in include/g++

rsmith added a comment. This needs a test. Comment at: lib/Driver/ToolChains/Linux.cpp:927-928 LibDir.str() + "/../include/c++", + // Some gcc installations put headers under "g++" without a + // version suffix. + LibDir.str() + "/../include/g++",

r345421 - Fix typo.

Author: rsmith Date: Fri Oct 26 12:35:39 2018 New Revision: 345421 URL: http://llvm.org/viewvc/llvm-project?rev=345421&view=rev Log: Fix typo. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td URL: http://llvm.org/v

r345423 - Fix test expectation to match reality.

Author: rsmith Date: Fri Oct 26 12:42:43 2018 New Revision: 345423 URL: http://llvm.org/viewvc/llvm-project?rev=345423&view=rev Log: Fix test expectation to match reality. Modified: cfe/trunk/test/SemaCXX/alignof.cpp Modified: cfe/trunk/test/SemaCXX/alignof.cpp URL: http://llvm.org/viewvc/l

[PATCH] D53609: [AST] Don't store data for GNU range case statement if not needed.

riccibruno updated this revision to Diff 171336. riccibruno added a reviewer: rsmith. riccibruno added a comment. Add a flag in the output of -ast-dump to indicate that a `CaseStmt` is a GNU range case statement, following rsmith's comment in https://reviews.llvm.org/D53717 Repository: rC Clan

[PATCH] D53684: [COFF, ARM64] Change setjmp for AArch64 Windows to use Intrinsic.sponentry

mstorsjo accepted this revision. mstorsjo added a comment. This revision is now accepted and ready to land. Looks good to me, once the dependency patch is done. Repository: rC Clang https://reviews.llvm.org/D53684 ___ cfe-commits mailing list cfe

[PATCH] D53714: [AST] Only store the needed data in SwitchStmt.

riccibruno updated this revision to Diff 171337. riccibruno added a reviewer: rsmith. riccibruno added a comment. Add a flag to the output of -ast-dump indicating which sub-statement a `SwitchStmt` is storing. This removes the ambiguity in the output of -ast-dump as per rsmith's comment in https:/

[PATCH] D53715: [AST] Only store the needed data in WhileStmt.

riccibruno updated this revision to Diff 171338. riccibruno added a reviewer: rsmith. riccibruno added a comment. Add a flag to the output of -ast-dump indicating which sub-statement a `WhileStmt` is storing. Repository: rC Clang https://reviews.llvm.org/D53715 Files: include/clang/AST/Stm

[PATCH] D53717: [AST] Only store the needed data in ForStmt.

riccibruno updated this revision to Diff 171340. riccibruno marked 6 inline comments as done. riccibruno added a comment. Add a flag in the output of -ast-dump indicating which sub-statement the `ForStmt` is storing. This removes the ambiguity in the output of -ast-dump. Repository: rC Clang

[PATCH] D53771: [clang-tidy] Avoid C arrays check

JonasToth added inline comments. Comment at: test/clang-tidy/misc-avoid-c-arrays.cpp:14 + +template +class array { lebedev.ri wrote: > JonasToth wrote: > > Please add a case with templates, where `T` itself is the array type, maybe > > there are other fancy tem

[PATCH] D53717: [AST] Only store the needed data in ForStmt.

rsmith added inline comments. Comment at: include/clang/AST/Stmt.h:1863 + Expr *getCond() { +return reinterpret_cast(getTrailingObjects()[condOffset()]); + } riccibruno wrote: > rsmith wrote: > > Please `static_cast` rather than `reinterpret_cast` throughou

[PATCH] D50860: [libc++][test] Remove non-portable assumption that thread's constructor allocates with ::new

CaseyCarter updated this revision to Diff 171344. CaseyCarter added a comment. Clarify use of `numAllocs`. https://reviews.llvm.org/D50860 Files: test/std/thread/thread.threads/thread.thread.class/thread.thread.constr/F.pass.cpp Index: test/std/thread/thread.threads/thread.thread.class/th

[PATCH] D51949: [clang-tidy] new check 'readability-isolate-declaration'

JonasToth added inline comments. Comment at: clang-tidy/readability/IsolateDeclarationCheck.cpp:52 + const LangOptions &LangOpts) { + assert(Indirections >= 0 && "Indirections must be non-negative"); + if (Indirections == 0) -

[PATCH] D51949: [clang-tidy] new check 'readability-isolate-declaration'

JonasToth updated this revision to Diff 171346. JonasToth marked 16 inline comments as done. JonasToth added a comment. - address review comments Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D51949 Files: clang-tidy/readability/CMakeLists.txt clang-tidy/readability/Isolate

[PATCH] D53621: Support for groups of attributes in #pragma clang attribute

arphaman accepted this revision. arphaman added a comment. LGTM too when Aaron's comments are addressed https://reviews.llvm.org/D53621 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[PATCH] D53443: [OpenMP][NVPTX] Enable default scheduling for parallel for in non-SPMD cases.

gtbercea updated this revision to Diff 171352. gtbercea added a comment. Add test. Repository: rC Clang https://reviews.llvm.org/D53443 Files: lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp test/OpenMP/nvptx_parallel_for_codegen.cpp Index: test/OpenMP/nvptx_parallel_for_codegen.cpp ===

[PATCH] D51949: [clang-tidy] new check 'readability-isolate-declaration'

JonasToth updated this revision to Diff 171361. JonasToth added a comment. - [Fix] wrong condition in matcher coming from incorrect code change Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D51949 Files: clang-tidy/readability/CMakeLists.txt clang-tidy/readability/IsolateDe

[PATCH] D53780: Fix bitcast to address space cast for coerced load/stores

david-salinas created this revision. david-salinas added reviewers: yaxunl, scchan. Herald added a subscriber: cfe-commits. Coerced load/stores through memory do not take into account potential address space differences when it creates its bitcasts. Repository: rC Clang https://reviews.llvm

  1   2   >