Re: [PATCH] arc-repeated-use-of-weak should not warn about IBOutlet properties

> On May 24, 2016, at 11:46 AM, Manman Ren wrote: > >> >> On May 24, 2016, at 11:42 AM, Bob Wilson > > wrote: >> >>> >>> On May 24, 2016, at 11:21 AM, Manman >> > wrote: >>> >>>

r270665 - arc-repeated-use-of-weak should not warn about IBOutlet properties

Author: bwilson Date: Wed May 25 00:41:57 2016 New Revision: 270665 URL: http://llvm.org/viewvc/llvm-project?rev=270665=rev Log: arc-repeated-use-of-weak should not warn about IBOutlet properties Revision r211132 was supposed to disable -Warc-repeated-use-of-weak for Objective-C properties

r270666 - Rename a variable to avoid shadowing function parameter. NFC.

Author: bwilson Date: Wed May 25 00:42:00 2016 New Revision: 270666 URL: http://llvm.org/viewvc/llvm-project?rev=270666=rev Log: Rename a variable to avoid shadowing function parameter. NFC. Modified: cfe/trunk/lib/Sema/AnalysisBasedWarnings.cpp Modified:

Re: [PATCH] D20510: [PATCH] Fix for bug 27802 misc-macro-parentheses breaks variadic macros

madsravn closed this revision. madsravn added a comment. Code committed. http://reviews.llvm.org/D20510 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r270664 - [X86] Update test cases to make sure storeu builtins use the storeu instrinsics. We were previously matching on other stores in the IR from this being an -O0 test.

Author: ctopper Date: Wed May 25 00:26:23 2016 New Revision: 270664 URL: http://llvm.org/viewvc/llvm-project?rev=270664=rev Log: [X86] Update test cases to make sure storeu builtins use the storeu instrinsics. We were previously matching on other stores in the IR from this being an -O0 test.

Re: [PATCH] D18488: [OpenMP] Parsing and sema support for the from clause

ABataev added a comment. Missed a test for printing and serialization/deserialization Comment at: test/OpenMP/nesting_of_regions.cpp:1832 @@ -1831,3 +1831,3 @@ for (int i = 0; i < 10; ++i) { -#pragma omp target update to(a) +#pragma omp target update from(a) }

Re: [PATCH] D15944: [OpenMP] Parsing and sema support for target update directive

ABataev added a comment. Add a test for nesting of regions here Comment at: lib/Parse/ParseOpenMP.cpp:42 @@ -42,1 +41,3 @@ + OMPD_target_exit, + OMPD_update }; Please, add a comma after 'OMPD_update' to reduce number of changes in the next modification of

Re: [PATCH] D18597: [OpenMP] Parsing and sema support for the to clause

ABataev added inline comments. Comment at: lib/Sema/SemaOpenMP.cpp:10218-10220 @@ +10217,5 @@ +Sema , DSAStackTy *DSAS, OpenMPClauseKind CKind, +ArrayRef VarList, SmallVector , +OMPClauseMappableExprCommon::MappableExprComponentLists , +SmallVector , +

Re: [PATCH] D20089: Adding a TargetParser for AArch64

jojo added inline comments. Comment at: include/llvm/Support/TargetParser.h:173 @@ +172,3 @@ +StringRef getArchName(unsigned ArchKind); +bool getArchFeatures(unsigned ArchKind, std::vector ); +unsigned getArchAttr(unsigned ArchKind); rengolin wrote: > Nitpick,

Re: [PATCH] D20089: Adding a TargetParser for AArch64

jojo updated this revision to Diff 58381. jojo added a comment. 1.include/llvm/Support/TargetParser.h Move move the declaration of getArchFeatures to a more reasonable place. Remove unnecessary parameter for getDefaultCPU. 2.lib/Support/TargetParser.cpp Make adjustments according to

Re: [PATCH] D20112: [OpenMP] Add support for the 'private pointer' flag to signal variables captured in target regions and used in first-private clauses.

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:5630-5639 @@ -5599,6 +5629,12 @@ // type, the default is 'tofrom'. CurMapTypes.push_back(ElementType->isAggregateType() ?

Re: [PATCH] D20111: [OpenMP] Adjust map type bits according to latest spec and use zero size array sections for pointers.

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:4874-4878 @@ -4881,1 +4873,7 @@ +OMP_MAP_IS_PTR = 0x10, +/// \brief This flags signals that an argument is the first one relating to +/// a map/private clause expression. For some cases a

Re: [PATCH] D20608: clang-cl: Treat dllimport explicit template instantiation definitions as declarations (PR27810, PR27811)

majnemer added a subscriber: majnemer. majnemer added a comment. Does this change our behavior for mingw? http://reviews.llvm.org/D20608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D18540: [Sema] Note when we've actually encountered a failure in ExprConstant, and take that into account when looking up objects.

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. LGTM with or without the `SpeculativeEvaluationRAII` refactor (which is ultimately pre-existing duplication). Please commit the rename of `keepEvaluatingAfterFailure` -> `noteFailure`

Re: [PATCH] D18488: [OpenMP] Parsing and sema support for the from clause

sfantao updated this revision to Diff 58371. sfantao added a comment. Implementation and tests only for the `from` clause. Similarly to the `to` clause, the implementation is based on the infrastructure contributed before for the `map` clause. http://reviews.llvm.org/D18488 Files:

Re: [PATCH] D18597: [OpenMP] Parsing and sema support for the to clause

sfantao updated this revision to Diff 58370. sfantao added a comment. Add implementation and tests only for the `to` clause. The implementation is based on the existing infrastructure used by the `map` clause already available upstream. http://reviews.llvm.org/D18597 Files:

Re: [PATCH] D15944: [OpenMP] Parsing and sema support for target update directive

sfantao updated this revision to Diff 58368. sfantao added a comment. Add parsing and sema support only for target update and already existing clauses (if and device). http://reviews.llvm.org/D15944 Files: include/clang-c/Index.h include/clang/AST/RecursiveASTVisitor.h

[PATCH] D20608: clang-cl: Treat dllimport explicit template instantiation definitions as declarations (PR27810, PR27811)

hans created this revision. hans added reviewers: rnk, thakis. hans added a subscriber: cfe-commits. This matches what MSVC does, and should make compiles faster by avoiding to unnecessarily emit a lot of code. http://reviews.llvm.org/D20608 Files: lib/Sema/SemaTemplate.cpp

r270642 - clang-cl: Fix unused argument warning when combining /O2 and /Ob2

Author: hans Date: Tue May 24 19:43:45 2016 New Revision: 270642 URL: http://llvm.org/viewvc/llvm-project?rev=270642=rev Log: clang-cl: Fix unused argument warning when combining /O2 and /Ob2 Modified: cfe/trunk/lib/Driver/MSVCToolChain.cpp cfe/trunk/test/Driver/cl-options.c Modified:

r270641 - [esan|wset] Add working set tool driver flags

Author: bruening Date: Tue May 24 19:41:24 2016 New Revision: 270641 URL: http://llvm.org/viewvc/llvm-project?rev=270641=rev Log: [esan|wset] Add working set tool driver flags Summary: Adds a new -fsanitize=efficiency-working-set flag to enable esan's working set tool. Adds appropriate tests

Re: [PATCH] D20602: Add .rgba syntax extension to ext_vector_type types

rsmith added a comment. In http://reviews.llvm.org/D20602#438643, @srhines wrote: > In http://reviews.llvm.org/D20602#438528, @rsmith wrote: > > > Looks like this extension was added at some point between 1.1 and 2.1. It > > would make sense to produce an `ExtWarn` for it if the OpenCL standard

LLVM buildmaster will be restarted tonight

Hello everyone, LLVM buildmaster will be updated and restarted after 6 PM Pacific time today. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20602: Add .rgba syntax extension to ext_vector_type types

srhines added a comment. In http://reviews.llvm.org/D20602#438528, @rsmith wrote: > Looks like this extension was added at some point between 1.1 and 2.1. It > would make sense to produce an `ExtWarn` for it if the OpenCL standard > version is less than the one that introduced it (whenever

Re: [PATCH] D20519: [clang-tidy] Ignore ADL-style using decls in unused-using-decls check.

alexfh requested changes to this revision. This revision now requires changes to proceed. Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:47 @@ +46,3 @@ + return; +// Ignores using-declarations defined in function definitions to avoid +// arguement-dependent

Re: [PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

hans added a comment. In http://reviews.llvm.org/D20576#438581, @Ilod wrote: > It seems the new test was commited as .cc instead of .c, which changes the > name mangling, making // NOINLINE-LABEL: @foo fail. > Either the file should be renamed, or -x c should be added to clang > invocation,

r270634 - docs: Document how safestack handles setjmp and exceptions.

Author: pcc Date: Tue May 24 18:38:02 2016 New Revision: 270634 URL: http://llvm.org/viewvc/llvm-project?rev=270634=rev Log: docs: Document how safestack handles setjmp and exceptions. Modified: cfe/trunk/docs/SafeStack.rst Modified: cfe/trunk/docs/SafeStack.rst URL:

r270633 - Rename test/CodeGen/inline-optim.cc to .c and provide a triple

Author: hans Date: Tue May 24 18:37:56 2016 New Revision: 270633 URL: http://llvm.org/viewvc/llvm-project?rev=270633=rev Log: Rename test/CodeGen/inline-optim.cc to .c and provide a triple Added: cfe/trunk/test/CodeGen/inline-optim.c - copied, changed from r270615,

Re: [PATCH] D20510: [PATCH] Fix for bug 27802 misc-macro-parentheses breaks variadic macros

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. lg http://reviews.llvm.org/D20510 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D18369: [OpenCL] Upstreaming khronos OpenCL header file.

yaxunl added inline comments. Comment at: lib/Headers/opencl-c.h:143 @@ +142,3 @@ +#if __OPENCL_C_VERSION__ >= CL_VERSION_2_0 +#define NULL ((void*)0) +#endif Anastasia wrote: > indentation seems wrong! fixed.

Re: [PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

Ilod added a comment. Hello, It seems the new test was commited as .cc instead of .c, which changes the name mangling, making // NOINLINE-LABEL: @foo fail. Either the file should be renamed, or -x c should be added to clang invocation, or LABEL check updated. Repository: rL LLVM

Re: [PATCH] D20597: Speed up check by using a recursive visitor.

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. Thank you! Comment at: clang-tidy/readability/FunctionSizeCheck.cpp:25 @@ +24,3 @@ + bool TraverseStmt(Stmt* Node) { +if (!Node) return Base::TraverseStmt(Node); +

Re: [PATCH] D20423: [Clang][LLVMGold] Passing LLVM arguments to gold plugin

llvm-commits added a subscriber: llvm-commits. llvm-commits added a comment. This seems strange to me. For example, it breaks the otherwise very convenient: $ clang -flto t.c -mllvm -some-internal-option-for-cc1

Re: [PATCH] D20602: Add .rgba syntax extension to ext_vector_type types

rsmith added a comment. Looks like this extension was added at some point between 1.1 and 2.1. It would make sense to produce an `ExtWarn` for it if the OpenCL standard version is less than the one that introduced it (whenever that was) -- that would match what we do for extensions in other

Re: [PATCH] D20563: Add load/store co-processor intrinsics

t.p.northover accepted this revision. t.p.northover added a reviewer: t.p.northover. t.p.northover added a comment. This revision is now accepted and ready to land. Thanks Ranjeet. LGTM! Tim. http://reviews.llvm.org/D20563 ___ cfe-commits mailing

r270625 - Make the altivec intrinsics that require immediate constant propagation

Author: echristo Date: Tue May 24 17:25:06 2016 New Revision: 270625 URL: http://llvm.org/viewvc/llvm-project?rev=270625=rev Log: Make the altivec intrinsics that require immediate constant propagation macros rather than functions. Unfortunately couldn't come up with a simple testcase that

[PATCH] D20602: Add .rgba syntax extension to ext_vector_type types

pirama created this revision. pirama added subscribers: cfe-commits, srhines. This patch enables .rgba accessors to ext_vector_type types and adds tests for syntax validation and code generation. 'a' and 'b' can appear either in the point access mode or the numeric access mode (for indices 10

Re: [PATCH] D20573: [libcxx] Allow explicit pthread opt-in

bcraig marked 2 inline comments as done. bcraig added a comment. http://reviews.llvm.org/D20573 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20573: [libcxx] Allow explicit pthread opt-in

bcraig updated this revision to Diff 58320. http://reviews.llvm.org/D20573 Files: CMakeLists.txt include/__config include/__config_site.in include/__threading_support Index: include/__threading_support === ---

Re: [PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

hans added a subscriber: majnemer. hans added a comment. majnemer reminded me that /O options can be combined, so e.g. /Odb2 would be a valid combination. I've followed up with r270614 to move the /Ob flags into that mechanism. Repository: rL LLVM http://reviews.llvm.org/D20576

Re: [PATCH] D20500: [ms] Allow more unqualified lookup of types in dependent base classes

This revision was automatically updated to reflect the committed changes. Closed by commit rL270615: [ms] Allow more unqualified lookup of types in dependent base classes (authored by rnk). Changed prior to commit: http://reviews.llvm.org/D20500?vs=58316=58319#toc Repository: rL LLVM

r270615 - [ms] Allow more unqualified lookup of types in dependent base classes

Author: rnk Date: Tue May 24 16:23:54 2016 New Revision: 270615 URL: http://llvm.org/viewvc/llvm-project?rev=270615=rev Log: [ms] Allow more unqualified lookup of types in dependent base classes Summary: In dependent contexts where we know a type name is required, such as a new expression, we

Re: [PATCH] D16948: [libcxx] Filesystem TS -- Complete

EricWF added a comment. For reference here's the current test coverage: http://efcs.ca/filesystem-coverage/ http://reviews.llvm.org/D16948 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r270614 - clang-cl: Bake /Ob0 and /Ob2 into the general /O option handling

Author: hans Date: Tue May 24 16:23:29 2016 New Revision: 270614 URL: http://llvm.org/viewvc/llvm-project?rev=270614=rev Log: clang-cl: Bake /Ob0 and /Ob2 into the general /O option handling This is a follow-up to r270609, wherein I forgot that /O options can be combined, and e.g. /Odb2 is a

Re: [PATCH] D20500: [ms] Allow more unqualified lookup of types in dependent base classes

rnk updated this revision to Diff 58316. rnk added a comment. - fix the diff http://reviews.llvm.org/D20500 Files: include/clang/Sema/Sema.h lib/Parse/ParseDecl.cpp lib/Sema/SemaDecl.cpp test/SemaTemplate/ms-delayed-default-template-args.cpp

Re: [PATCH] D20574: [libcxxabi] Allow explicit pthread opt-in

bcraig marked an inline comment as done. bcraig added a comment. http://reviews.llvm.org/D20574 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20574: [libcxxabi] Allow explicit pthread opt-in

bcraig updated this revision to Diff 58317. http://reviews.llvm.org/D20574 Files: CMakeLists.txt Index: CMakeLists.txt === --- CMakeLists.txt +++ CMakeLists.txt @@ -113,6 +113,7 @@ option(LIBCXXABI_ENABLE_WERROR "Fail and stop

Re: [PATCH] D20563: Add load/store co-processor intrinsics

rs marked an inline comment as done. rs added a comment. Tim thanks for reviewing this patch. I've uploaded a new one with your suggested change. If I don't respond to any further comments after today it'll be because I'm on holiday and won't be back till next week.

Re: [PATCH] D20500: [ms] Allow more unqualified lookup of types in dependent base classes

rnk updated this revision to Diff 58315. rnk added a comment. Herald added a subscriber: jyknight. - nits http://reviews.llvm.org/D20500 Files: include/clang-c/Index.h include/clang/AST/RecordLayout.h include/clang/Basic/BuiltinsX86.def include/clang/Driver/CLCompatOptions.td

Re: [PATCH] D20563: Add load/store co-processor intrinsics

rs updated this revision to Diff 58314. http://reviews.llvm.org/D20563 Files: include/clang/Basic/BuiltinsARM.def test/CodeGen/builtins-arm.c test/Sema/builtins-arm.c Index: test/Sema/builtins-arm.c === ---

Re: [PATCH] D18821: Add bugprone-bool-to-integer-conversion

Prazek updated this revision to Diff 58309. Prazek added a comment. Some small bugfixes afeter running it on llvm http://reviews.llvm.org/D18821 Files: clang-tidy/CMakeLists.txt clang-tidy/bugprone/BoolToIntegerConversionCheck.cpp clang-tidy/bugprone/BoolToIntegerConversionCheck.h

Re: [PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

This revision was automatically updated to reflect the committed changes. Closed by commit rL270609: [Driver] Add support for -finline-functions and /Ob2 flags (authored by hans). Changed prior to commit: http://reviews.llvm.org/D20576?vs=58302=58308#toc Repository: rL LLVM

r270609 - [Driver] Add support for -finline-functions and /Ob2 flags

Author: hans Date: Tue May 24 15:40:51 2016 New Revision: 270609 URL: http://llvm.org/viewvc/llvm-project?rev=270609=rev Log: [Driver] Add support for -finline-functions and /Ob2 flags -finline-functions and /Ob2 are currently ignored by Clang. The only way to enable inlining is to use the

Re: [PATCH] D20328: [libcxx] Externally threaded libc++ variant

rmaprath added inline comments. Comment at: include/__threading_support:201 @@ +200,3 @@ +// Mutex +#define _LIBCPP_MUTEX_INITIALIZER nullptr +struct __libcpp_platform_mutex_t; bcraig wrote: > rmaprath wrote: > > bcraig wrote: > > > rmaprath wrote: > > > > bcraig

[PATCH] D20596: [libcxx] Refactor locale switching, creation, and destruction

bcraig created this revision. bcraig added reviewers: mclow.lists, EricWF, joerg, jroelofs. bcraig added a subscriber: cfe-commits. Herald added a subscriber: jfb. This patch cleans up libcxx's usage of newlocale, freelocale, uselocale, and locale_t. First, libcxx no longer defines the posix

Re: [PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

Ilod added a comment. I don't have commit access, so you can do it for me, thanks. http://reviews.llvm.org/D20576 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. Looks good to me! Do you have commit access, or would you like me to commit this for you? http://reviews.llvm.org/D20576 ___ cfe-commits mailing

Re: [PATCH] D17462: Fix a codegen bug for variadic functions with pass_object_size params

george.burgess.iv added a comment. Ping :) http://reviews.llvm.org/D17462 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D18540: [Sema] Note when we've actually encountered a failure in ExprConstant, and take that into account when looking up objects.

george.burgess.iv added a comment. Ping :) http://reviews.llvm.org/D18540 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

Ilod updated this revision to Diff 58302. Ilod added a comment. Thanks! Updated to respect the 80-columns limit. Updated the tests to have a clearer checks. (I don't know well the FileCheck, so I took example from test/CodeGen/noinline.c, but this is indeed more comprehensible).

Re: [PATCH] D20444: [OpenCL] Include opencl-c.h by default as a module

yaxunl updated this revision to Diff 58297. yaxunl added a comment. Fix regressions in unit test due to ConstructJob disables builtin include path. http://reviews.llvm.org/D20444 Files: include/clang/Basic/LangOptions.def include/clang/Driver/CC1Options.td

Re: [PATCH] D20328: [libcxx] Externally threaded libc++ variant

rmaprath added inline comments. Comment at: include/__threading_support:201 @@ +200,3 @@ +// Mutex +#define _LIBCPP_MUTEX_INITIALIZER nullptr +struct __libcpp_platform_mutex_t; bcraig wrote: > rmaprath wrote: > > bcraig wrote: > > > I'm not sure I like taking the

Re: [PATCH] D20447: [OpenCL] Fixup extension list

jvesely added inline comments. Comment at: test/SemaOpenCL/extension-version.cl:11 @@ +10,3 @@ +#endif +#pragma OPENCL EXTENSION cl_clang_storage_class_specifiers: enable + Anastasia wrote: > jvesely wrote: > > Anastasia wrote: > > > Could you use standard

Re: [PATCH] D20328: [libcxx] Externally threaded libc++ variant

bcraig added inline comments. Comment at: include/__threading_support:201 @@ +200,3 @@ +// Mutex +#define _LIBCPP_MUTEX_INITIALIZER nullptr +struct __libcpp_platform_mutex_t; rmaprath wrote: > bcraig wrote: > > I'm not sure I like taking the freedom to define

Re: [PATCH] D20537: clang-rename: fix renaming non-members variables when referenced as macro arguments

This revision was automatically updated to reflect the committed changes. Closed by commit rL270599: clang-rename: fix renaming non-members variables when referenced as macro… (authored by vmiklos). Changed prior to commit: http://reviews.llvm.org/D20537?vs=58136=58290#toc Repository: rL

[clang-tools-extra] r270599 - clang-rename: fix renaming non-members variables when referenced as macro arguments

Author: vmiklos Date: Tue May 24 14:08:53 2016 New Revision: 270599 URL: http://llvm.org/viewvc/llvm-project?rev=270599=rev Log: clang-rename: fix renaming non-members variables when referenced as macro arguments The second check failed, FOO(C::X) wasn't renamed to FOO(C::Y). Reviewers: klimek

RE: r267590 - [OpenCL] Add predefined macros.

Did clang accept -std=CL2.0 before this change? According to this diff, the only accepted option for -std= is c99 when compiling OpenCL programs. Sam -Original Message- From: Anastasia Stulova [mailto:anastasia.stul...@arm.com] Sent: Tuesday, May 24, 2016 2:48 PM To: Liu, Yaxun (Sam)

RE: r267590 - [OpenCL] Add predefined macros.

Hi Sam, I think this commit broke Clang. It seems we are no longer able to pass the -std=CL2.0, which is important for the standalone Clang OpenCL users as -cl-std is frontend only option. clang -std=CL2.0 test.cl error: invalid argument '-std=CL2.0' not allowed with 'OpenCL' We might have

Re: [PATCH] arc-repeated-use-of-weak should not warn about IBOutlet properties

> On May 24, 2016, at 11:42 AM, Bob Wilson wrote: > >> >> On May 24, 2016, at 11:21 AM, Manman wrote: >> >> >>> On May 23, 2016, at 8:15 PM, Bob Wilson wrote: >>> >>> Revision r211132 was supposed to disable

Re: [PATCH] arc-repeated-use-of-weak should not warn about IBOutlet properties

> On May 24, 2016, at 11:21 AM, Manman wrote: > > >> On May 23, 2016, at 8:15 PM, Bob Wilson wrote: >> >> Revision r211132 was supposed to disable -Warc-repeated-use-of-weak for >> Objective-C properties marked with the IBOutlet attribute. Those

Re: [PATCH] D20328: [libcxx] Externally threaded libc++ variant

rmaprath added inline comments. Comment at: include/__threading_support:201 @@ +200,3 @@ +// Mutex +#define _LIBCPP_MUTEX_INITIALIZER nullptr +struct __libcpp_platform_mutex_t; bcraig wrote: > I'm not sure I like taking the freedom to define

Re: [PATCH] D20447: [OpenCL] Fixup extension list

Anastasia added inline comments. Comment at: test/SemaOpenCL/extension-version.cl:11 @@ +10,3 @@ +#endif +#pragma OPENCL EXTENSION cl_clang_storage_class_specifiers: enable + jvesely wrote: > Anastasia wrote: > > Could you use standard diagnostic check please: >

Re: [PATCH] D20574: [libcxxabi] Allow explicit pthread opt-in

rmaprath added inline comments. Comment at: CMakeLists.txt:278 @@ -277,1 +277,3 @@ +if (LIBCXXABI_HAS_PTHREAD_API) + add_definitions(-D_LIBCPP_HAS_THREAD_API_PTHREAD) Shouldn't there be an `option()` line for this new build option? Also a check for

Re: [clang-tools-extra] r261991 - [clang-tidy] Fix a crash issue when clang-tidy runs with compilation database.

Ping, who's going to merge? I have no commit access. Cheers, Edward-san 2016-05-20 18:34 GMT+02:00 Tom Stellard : > Hi, > > This looks fine to me, go ahead and merge. > > -Tom > > On Thu, May 19, 2016 at 08:29:14PM +0200, Alexander Kornienko wrote: >> On Thu, May 19, 2016 at

Re: [PATCH] D20573: [libcxx] Allow explicit pthread opt-in

rmaprath added a comment. Couple of minor comments. LGTM otherwise. / Asiri Comment at: CMakeLists.txt:387 @@ -386,2 +386,3 @@ +config_define_if(LIBCXX_HAS_PTHREAD_API _LIBCPP_HAS_THREAD_API_PTHREAD) config_define_if(LIBCXX_HAS_MUSL_LIBC _LIBCPP_HAS_MUSL_LIBC)

Re: [PATCH] arc-repeated-use-of-weak should not warn about IBOutlet properties

> On May 23, 2016, at 8:15 PM, Bob Wilson wrote: > > Revision r211132 was supposed to disable -Warc-repeated-use-of-weak for > Objective-C properties marked with the IBOutlet attribute. Those properties > are supposed to be weak but they are only accessed from the main

r270591 - [RecordLayout] Use an ASTVector instead of using a separate pointer and counter

Author: majnemer Date: Tue May 24 13:10:50 2016 New Revision: 270591 URL: http://llvm.org/viewvc/llvm-project?rev=270591=rev Log: [RecordLayout] Use an ASTVector instead of using a separate pointer and counter No functional change is intended. Modified:

Re: [PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

hans added a comment. Thanks! Some comments below: Comment at: lib/Driver/Tools.cpp:5335 @@ -5334,3 +5334,3 @@ - if (Args.hasArg(options::OPT_fno_inline_functions)) -CmdArgs.push_back("-fno-inline-functions"); + if (Arg* InlineArg =

Re: [PATCH] D20334: [libcxx] Fix a bug in strstreambuf::overflow

bcraig added a subscriber: bcraig. bcraig added a comment. I don't believe this is a libcxx bug, but it is a bug in the test code. oss.str(); isn't required to return a null terminated string. std::cout << (char *) requires a null terminated string though. http://reviews.llvm.org/D20334

Re: [PATCH] D20334: [libcxx] Fix a bug in strstreambuf::overflow

ping. > On May 17, 2016, at 12:20 PM, Akira Hatanaka via cfe-commits > wrote: > > ahatanak created this revision. > ahatanak added reviewers: mclow.lists, EricWF, howard.hinnant. > ahatanak added a subscriber: cfe-commits. > > The end pointer should point to one

[PATCH] D20581: [include-fixer] Simplify the code since we won't handle multiple includes at once.

hokein created this revision. hokein added a reviewer: bkramer. hokein added a subscriber: cfe-commits. http://reviews.llvm.org/D20581 Files: include-fixer/IncludeFixer.cpp Index: include-fixer/IncludeFixer.cpp === ---

Re: r270580 - [Lex] Support more type-traits in __has_feature

Done with r270583, I added a note in a comment to avoid this situation in the future. On Tue, May 24, 2016 at 10:26 AM, Richard Smith wrote: > On 24 May 2016 10:15 a.m., "Richard Smith" wrote: > > > > As I recall, this was intentional. The

r270583 - Revert "[Lex] Support more type-traits in __has_feature"

Author: majnemer Date: Tue May 24 12:21:42 2016 New Revision: 270583 URL: http://llvm.org/viewvc/llvm-project?rev=270583=rev Log: Revert "[Lex] Support more type-traits in __has_feature" This reverts commit r270580. Using __has_feature to test for type-traits is deprecated. Modified:

Re: r270580 - [Lex] Support more type-traits in __has_feature

On 24 May 2016 10:15 a.m., "Richard Smith" wrote: > > As I recall, this was intentional. The supported way to check for these is has_builtin. The has_feature support is provided only for backwards compatibility. See documentation here:

Re: [clang-tools-extra] r270575 - [clang-tidy] modernize-pass-by-value bugfix. Reverting lit-style test

I'm glad. I'm finally getting the hang of this (I think) :) Best regards, Mads Ravn On Tue, May 24, 2016 at 7:01 PM Renato Golin wrote: > On 24 May 2016 at 17:09, Mads Ravn via cfe-commits > wrote: > > Author: madsravn > > Date: Tue May 24

[PATCH] D20576: [Driver] Add support for -finline-functions and /Ob2 flags

Ilod created this revision. Ilod added a reviewer: hans. Ilod added a subscriber: cfe-commits. -finline-functions and /Ob2 are currently ignored by Clang. The only way to enable inlining is to use the global O flags, which also enable other options, or to emit LLVM bitcode using Clang, then

Re: r270580 - [Lex] Support more type-traits in __has_feature

As I recall, this was intentional. The supported way to check for these is has_builtin. The has_feature support is provided only for backwards compatibility. On 24 May 2016 9:59 a.m., "David Majnemer via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > Author: majnemer > Date: Tue May 24

Re: [PATCH] D18369: [OpenCL] Upstreaming khronos OpenCL header file.

Anastasia added a comment. Sam, there are a few small comments, otherwise it seems to be in a good shape. @rsmith, do you think you could take a look here? The OpenCL side is fine, but I was just wondering if you see any issue with us adding a header of ~17K lines. It is all part of OpenCL

Re: [PATCH] D20328: [libcxx] Externally threaded libc++ variant

bcraig added a comment. Note: You'll want to look at http://reviews.llvm.org/D20573, as there will be confilicts for whoever submits second. Comment at: include/__threading_support:201 @@ +200,3 @@ +// Mutex +#define _LIBCPP_MUTEX_INITIALIZER nullptr +struct

Re: [clang-tools-extra] r270575 - [clang-tidy] modernize-pass-by-value bugfix. Reverting lit-style test

On 24 May 2016 at 17:09, Mads Ravn via cfe-commits wrote: > Author: madsravn > Date: Tue May 24 11:09:24 2016 > New Revision: 270575 > > URL: http://llvm.org/viewvc/llvm-project?rev=270575=rev > Log: > [clang-tidy] modernize-pass-by-value bugfix. Reverting lit-style

[clang-tools-extra] r270581 - [clang-tidy] Don't rely on being available.

Author: d0k Date: Tue May 24 11:54:26 2016 New Revision: 270581 URL: http://llvm.org/viewvc/llvm-project?rev=270581=rev Log: [clang-tidy] Don't rely on being available. The STL isn't necessarily around when running tests. Make our own fake std::array to test this. Modified:

r270580 - [Lex] Support more type-traits in __has_feature

Author: majnemer Date: Tue May 24 11:53:13 2016 New Revision: 270580 URL: http://llvm.org/viewvc/llvm-project?rev=270580=rev Log: [Lex] Support more type-traits in __has_feature It looks like we forgot to update the __has_feature support when we added some of the type traits. Modified:

[PATCH] D20574: [libcxxabi] Allow explicit pthread opt-in

bcraig created this revision. bcraig added reviewers: rmaprath, mclow.lists, EricWF. bcraig added a subscriber: cfe-commits. Companion to http://reviews.llvm.org/D20573 When building libcxxabi in tree (i.e. in llvm/projects/libcxxabi, along with llvm/projects/libcxx), libcxx's __config_site.in

[PATCH] D20573: [libcxx] Allow explicit pthread opt-in

bcraig created this revision. bcraig added reviewers: rmaprath, mclow.lists, EricWF. bcraig added a subscriber: cfe-commits. The existing pthread detection code in __config is pretty good for common operating systems. It doesn't allow cmake-time choices to be made for uncommon operating

Re: [PATCH] D20563: Add load/store co-processor intrinsics

t.p.northover added a subscriber: t.p.northover. t.p.northover added a comment. Just one suggestion for the tests here: Comment at: test/Sema/builtins-arm.c:50 @@ -49,2 +49,3 @@ void test6(int a, int b, int c) { + __builtin_arm_ldc(a, 2, ); // expected-error {{argument to

Re: [clang-tools-extra] r270567 - [clang-tidy] modernize-pass-by-value bugfix. Reverting lit-style test

On 24 May 2016 at 16:13, Mads Ravn via cfe-commits wrote: > Author: madsravn > Date: Tue May 24 10:13:44 2016 > New Revision: 270567 > > URL: http://llvm.org/viewvc/llvm-project?rev=270567=rev > Log: > [clang-tidy] modernize-pass-by-value bugfix. Reverting lit-style

Re: [PATCH] D20112: [OpenMP] Add support for the 'private pointer' flag to signal variables captured in target regions and used in first-private clauses.

sfantao updated this revision to Diff 58253. sfantao marked 2 inline comments as done. sfantao added a comment. - Rebase after last changes related to first private. http://reviews.llvm.org/D20112 Files: lib/CodeGen/CGOpenMPRuntime.cpp test/OpenMP/target_firstprivate_codegen.cpp

[clang-tools-extra] r270578 - [include-fixer][find-all-symbols] added missing dependencies.

Author: ioeric Date: Tue May 24 11:35:24 2016 New Revision: 270578 URL: http://llvm.org/viewvc/llvm-project?rev=270578=rev Log: [include-fixer][find-all-symbols] added missing dependencies. Modified: clang-tools-extra/trunk/include-fixer/find-all-symbols/CMakeLists.txt Modified:

Re: [PATCH] D20111: [OpenMP] Adjust map type bits according to latest spec and use zero size array sections for pointers.

sfantao updated this revision to Diff 58252. sfantao added a comment. - Rebase after last changes related to first private. http://reviews.llvm.org/D20111 Files: lib/CodeGen/CGOpenMPRuntime.cpp test/OpenMP/target_codegen.cpp test/OpenMP/target_codegen_registration.cpp

Re: [PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

rnk added a comment. lgtm http://reviews.llvm.org/D18953 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] r270575 - [clang-tidy] modernize-pass-by-value bugfix. Reverting lit-style test

Author: madsravn Date: Tue May 24 11:09:24 2016 New Revision: 270575 URL: http://llvm.org/viewvc/llvm-project?rev=270575=rev Log: [clang-tidy] modernize-pass-by-value bugfix. Reverting lit-style test Adding to revision 270567. The lit-style test was wrong. This is being fixed by this commit.

  1   2   >