[PATCH] D29750: [PPC] Enable -fomit-frame-pointer by default for PPC

inouehrs created this revision. This patch enables `-fomit-frame-pointer` option by default for PowerPC if -O1 or above optimization level is specified. GCC already enables this option by default for -O1 or above in all of ppc32, ppc64 and ppc64le. This option gives some performance

r294560 - [X86] Fix copy and paste bug in clzero test from r294559.

Author: ctopper Date: Thu Feb 9 00:22:43 2017 New Revision: 294560 URL: http://llvm.org/viewvc/llvm-project?rev=294560=rev Log: [X86] Fix copy and paste bug in clzero test from r294559. Modified: cfe/trunk/test/CodeGen/builtin-clzero.c Modified: cfe/trunk/test/CodeGen/builtin-clzero.c URL:

[PATCH] D29386: Clzero flag addition and inclusion under znver1

This revision was automatically updated to reflect the committed changes. Closed by commit rL294559: [X86] Clzero flag addition and inclusion under znver1 (authored by ctopper). Changed prior to commit: https://reviews.llvm.org/D29386?vs=87437=87762#toc Repository: rL LLVM

r294559 - [X86] Clzero flag addition and inclusion under znver1

Author: ctopper Date: Thu Feb 9 00:10:14 2017 New Revision: 294559 URL: http://llvm.org/viewvc/llvm-project?rev=294559=rev Log: [X86] Clzero flag addition and inclusion under znver1 1. Adds the command line flag for clzero. 2. Includes the clzero flag under znver1. 3. Defines the macro for

[PATCH] D29386: Clzero flag addition and inclusion under znver1

GGanesh added a comment. Thank you @craig.topper. Just want to check if the patch can be commited to 4.0 release branch as well. I understand that the trunk is in blocker bug-fix stage but just want to get it confirmed. Repository: rL LLVM https://reviews.llvm.org/D29386

[PATCH] D29748: [cxx1z-constexpr-lambda] Implement captures - thus completing implementation of constexpr lambdas.

faisalv added inline comments. Comment at: lib/AST/ExprConstant.cpp:5061 + APValue RVal; + // FIXME: We need to make sure we're passing the right type that + // maintains cv-qualifiers. I don't think we need this fixme - the type of

[PATCH] D29748: [cxx1z-constexpr-lambda] Implement captures - thus completing implementation of constexpr lambdas.

faisalv created this revision. faisalv added a project: clang-c. Herald added a subscriber: EricWF. This patch attempts to enable evaluation of all forms of captures (however deeply nested) within constexpr lambdas. Appreciate the feedback. Thanks! Repository: rL LLVM

[PATCH] D29386: Clzero flag addition and inclusion under znver1

craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM, but i'll make some additional testing changes before committing it. -Add __CLZERO__ checks to test/Preprocessor/predefined-arch-macros.c for znver1 -Add command line testing

[PATCH] D28445: [Analyzer] Extend taint propagation and checking

vlad.tsyrklevich added inline comments. Comment at: lib/StaticAnalyzer/Checkers/GenericTaintChecker.cpp:442 + +const RecordDecl *RD = RT->getDecl()->getDefinition(); +for (const auto *I : RD->fields()) { a.sidorin wrote: > NoQ wrote: > > We need to be

[libunwind] r294556 - Revert "[libunwind][CMake] Use libc++ headers when available"

Author: phosek Date: Wed Feb 8 21:56:47 2017 New Revision: 294556 URL: http://llvm.org/viewvc/llvm-project?rev=294556=rev Log: Revert "[libunwind][CMake] Use libc++ headers when available" This is causing build failure when building libc++abi with libunwind. This reverts commit

[libunwind] r294554 - [libunwind][CMake] Use libc++ headers when available

Author: phosek Date: Wed Feb 8 20:20:25 2017 New Revision: 294554 URL: http://llvm.org/viewvc/llvm-project?rev=294554=rev Log: [libunwind][CMake] Use libc++ headers when available libunwind depends on C++ library headers. When building libunwind as part of LLVM and libc++ is available, use its

[libcxx] r294553 - [libcxx][CMake] Support in-tree libunwind when building as part of runtimes

Author: phosek Date: Wed Feb 8 20:19:43 2017 New Revision: 294553 URL: http://llvm.org/viewvc/llvm-project?rev=294553=rev Log: [libcxx][CMake] Support in-tree libunwind when building as part of runtimes When building as part of runtimes, there is no predefined order in which the runtimes are

[libcxxabi] r294552 - [libcxxabi][CMake] Support in-tree libunwind when building as part of runtimes

Author: phosek Date: Wed Feb 8 20:19:30 2017 New Revision: 294552 URL: http://llvm.org/viewvc/llvm-project?rev=294552=rev Log: [libcxxabi][CMake] Support in-tree libunwind when building as part of runtimes When building as part of runtimes, there is no predefined order in which the runtimes are

Re: [cfe-commits] r164177 - in /cfe/trunk: lib/Driver/ test/Driver/ test/Driver/Inputs/freescale_ppc_tree/ test/Driver/Inputs/freescale_ppc_tree/lib/ test/Driver/Inputs/freescale_ppc_tree/usr/ test/Dr

It's blast from the past time! On Tue, Sep 18, 2012 at 3:28 PM Hal Finkel wrote: > Author: hfinkel > Date: Tue Sep 18 17:25:07 2012 > New Revision: 164177 > > URL: http://llvm.org/viewvc/llvm-project?rev=164177=rev > Log: > Add C/C++ header locations for the Freescale SDK. > >

[PATCH] D29739: Make Lit tests C++11 compatible - Objective-C++

tigerleapgorge created this revision. I am continuing to make our Lit tests C++11 compatible. This patch defaults five Objective-C++ tests to run under -std=gnu++98 This patch is essentially a continuation of Bug 24344 https://llvm.org/bugs/show_bug.cgi?id=24344 Fix r289167 resolved all tests

[PATCH] D25674: [Concepts] Class template associated constraints

hubert.reinterpretcast added inline comments. Comment at: include/clang/AST/DeclTemplate.h:373-391 +class TemplateDeclWithACBase { +protected: + TemplateDeclWithACBase() = default; + + ConstrainedTemplateDeclInfo CTDInfo; +}; + rsmith wrote: > This mechanism

[libunwind] r294547 - Creating release candidate rc2 from release_400 branch

Author: hans Date: Wed Feb 8 18:14:34 2017 New Revision: 294547 URL: http://llvm.org/viewvc/llvm-project?rev=294547=rev Log: Creating release candidate rc2 from release_400 branch Added: libunwind/tags/RELEASE_400/rc2/ (props changed) - copied from r294546,

[libcxx] r294540 - Creating release candidate rc2 from release_400 branch

Author: hans Date: Wed Feb 8 18:14:12 2017 New Revision: 294540 URL: http://llvm.org/viewvc/llvm-project?rev=294540=rev Log: Creating release candidate rc2 from release_400 branch Added: libcxx/tags/RELEASE_400/rc2/ (props changed) - copied from r294539, libcxx/branches/release_40/

[libcxxabi] r294541 - Creating release candidate rc2 from release_400 branch

Author: hans Date: Wed Feb 8 18:14:16 2017 New Revision: 294541 URL: http://llvm.org/viewvc/llvm-project?rev=294541=rev Log: Creating release candidate rc2 from release_400 branch Added: libcxxabi/tags/RELEASE_400/rc2/ (props changed) - copied from r294540,

Buildbot numbers for the last week of 01/29/2017 - 02/04/2017

Hello everyone, Below are some buildbot numbers for the last week of 01/29/2017 - 02/04/2017. 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

[PATCH] D29736: [WebAssembly] Add target specific overrides for lgamma family functions

efriedma added a comment. It doesn't make sense to do this in WebAssembly-specific code; every POSIX platform has a signgam which behaves the same way. https://reviews.llvm.org/D29736 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r294534 - Add check that "#define unix 1" is present for the PS4 target.

Author: dyung Date: Wed Feb 8 18:07:38 2017 New Revision: 294534 URL: http://llvm.org/viewvc/llvm-project?rev=294534=rev Log: Add check that "#define unix 1" is present for the PS4 target. Modified: cfe/trunk/test/Preprocessor/init.c Modified: cfe/trunk/test/Preprocessor/init.c URL:

[PATCH] D29737: Updates documentation to include command to run clang-tidy tests.

lethalantidote created this revision. Herald added a subscriber: JDevlieghere. Adds the commandline need to run clang-tidy tests. https://reviews.llvm.org/D29737 Files: clang-tools-extra/docs/clang-tidy/index.rst Index: clang-tools-extra/docs/clang-tidy/index.rst

[PATCH] D28543: Elliminates uninitialized warning for volitile varibles.

lethalantidote added a comment. Any updates on this? https://reviews.llvm.org/D28543 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D29736: [WebAssembly] Add target specific overrides for lgamma family functions

jgravelle-google created this revision. Herald added a subscriber: jfb. Current WebAssembly implementations use musl libc, which writes to signgam in its lgamma functions. We want the rest of the builtins to be const under -fno-math-errno, so overwrite lgamma's builtin definitions to be

[PATCH] D25674: [Concepts] Class template associated constraints

rsmith added inline comments. Comment at: include/clang/AST/DeclTemplate.h:373-391 +class TemplateDeclWithACBase { +protected: + TemplateDeclWithACBase() = default; + + ConstrainedTemplateDeclInfo CTDInfo; +}; + This mechanism seems unnecessary to me;

[PATCH] D29726: [Clang-tidy] Fix for bug 31838: readability-delete-null-pointer does not work for class members

alexfh requested changes to this revision. alexfh added inline comments. This revision now requires changes to proceed. Comment at: clang-tidy/readability/DeleteNullPointerCheck.cpp:46 + DeleteExpr, DeleteMemberExpr, +

[PATCH] D25674: [Concepts] Class template associated constraints

hubert.reinterpretcast added a comment. Ping 2? https://reviews.llvm.org/D25674 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D16135: Macro Debug Info support in Clang

> On Feb 8, 2017, at 2:31 PM, David Blaikie wrote: > > > > On Wed, Feb 8, 2017 at 2:25 PM Amjad Aboud via Phabricator > > wrote: > aaboud added a comment. > > > How much does the build directory grow? > > Is

[PATCH] D29630: [libcxx] Threading support: externalize sleep_for()

joerg accepted this revision. joerg added a comment. One small issue left, otherwise LGTM. Comment at: include/__threading_support:187 +_LIBCPP_THREAD_ABI_VISIBILITY +void __libcpp_thread_sleep_for(const chrono::nanoseconds& ns); + Drop the name here.

Re: [PATCH] D16135: Macro Debug Info support in Clang

On Wed, Feb 8, 2017 at 2:25 PM Amjad Aboud via Phabricator < revi...@reviews.llvm.org> wrote: > aaboud added a comment. > > > How much does the build directory grow? > > Is there any noticeable compile time regression? > > I build clang in release mode using GCC, then used that build to build >

[PATCH] D16135: Macro Debug Info support in Clang

aaboud added a comment. > How much does the build directory grow? > Is there any noticeable compile time regression? I build clang in release mode using GCC, then used that build to build clang in debug mode with "-fstandalone-debug" flag, one time with my changes and another time without my

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge updated this revision to Diff 87711. tigerleapgorge edited the summary of this revision. tigerleapgorge added a comment. Remove 2 OpenMP tests reviewed via https://reviews.llvm.org/D29725 https://reviews.llvm.org/D21626 Files: test/Modules/Inputs/merge-using-decls/a.h

r294522 - [sancov] pc_guard_init is called at least once per DSO.

Author: aizatsky Date: Wed Feb 8 15:28:39 2017 New Revision: 294522 URL: http://llvm.org/viewvc/llvm-project?rev=294522=rev Log: [sancov] pc_guard_init is called at least once per DSO. Summary: Documentation update for https://reviews.llvm.org/D29662 Differential Revision:

r294512 - Initialize builtins during modular codegen

Author: dblaikie Date: Wed Feb 8 14:51:11 2017 New Revision: 294512 URL: http://llvm.org/viewvc/llvm-project?rev=294512=rev Log: Initialize builtins during modular codegen Added: cfe/trunk/test/Modules/Inputs/codegen-opt/ cfe/trunk/test/Modules/Inputs/codegen-opt/bar.h - copied,

r294509 - More fixes for places where 'decltype(auto)' is permitted in the C++ grammar but makes no sense.

Author: rsmith Date: Wed Feb 8 14:39:08 2017 New Revision: 294509 URL: http://llvm.org/viewvc/llvm-project?rev=294509=rev Log: More fixes for places where 'decltype(auto)' is permitted in the C++ grammar but makes no sense. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td

[PATCH] D29726: [Clang-tidy] Fix for bug 31838: readability-delete-null-pointer does not work for class members

madsravn created this revision. Made a small fix to readability-delete-null-pointer check such that it includes class members. https://reviews.llvm.org/D29726 Files: clang-tidy/readability/DeleteNullPointerCheck.cpp clang-tidy/readability/DeleteNullPointerCheck.h

r294506 - Don't crash on 'decltype(auto)::'. Rather than treating it as a meaningless

Author: rsmith Date: Wed Feb 8 13:58:48 2017 New Revision: 294506 URL: http://llvm.org/viewvc/llvm-project?rev=294506=rev Log: Don't crash on 'decltype(auto)::'. Rather than treating it as a meaningless nested-name-specifier (as the standard appears to require), treat it as the type specifier

r294504 - [Lit Test] Make tests C++11 compatible - Parse OpenMP

Author: lcharles Date: Wed Feb 8 13:46:15 2017 New Revision: 294504 URL: http://llvm.org/viewvc/llvm-project?rev=294504=rev Log: [Lit Test] Make tests C++11 compatible - Parse OpenMP Differential Revision: https://reviews.llvm.org/D29725 Modified:

[PATCH] D29725: Lit C++11 Compatibility - Parse OpenMP

This revision was automatically updated to reflect the committed changes. Closed by commit rL294504: [Lit Test] Make tests C++11 compatible - Parse OpenMP (authored by lcharles). Changed prior to commit: https://reviews.llvm.org/D29725?vs=87684=87687#toc Repository: rL LLVM

[PATCH] D29725: Lit C++11 Compatibility - Parse OpenMP

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG https://reviews.llvm.org/D29725 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D19201: [clang-tidy] misc-throw-with-noexcept

Prazek added inline comments. Comment at: clang-tidy/misc/ThrowWithNoexceptCheck.cpp:54 +// FIXME use DiagnosticIDs::Level::Note +diag(NoExceptRange.getBegin(), "in a function declared no-throw here:", DiagnosticIDs::Note) +<<

[PATCH] D29725: Lit C++11 Compatibility - Parse OpenMP

tigerleapgorge created this revision. I am continuing to make out Lit tests C++11 compatible. There are two tests in this patch. Both tests verify parse errors with ill formed OpenMP expressions. test/OpenMP/declare_reduction_messages.cpp In C++11, an opening square bracket is the start of a

[PATCH] D29724: [Driver] Report available language standards on user error

idlecode created this revision. In case user did not provide valid standard name for `-std` option, available values (with short description) will be reported. To test: `clang -std=iso3103 -c source.c` https://reviews.llvm.org/D29724 Files: include/clang/Basic/DiagnosticDriverKinds.td

[libcxx] r294501 - Update info for LWG 2665 in upcoming_meeting.html

Author: ericwf Date: Wed Feb 8 13:04:18 2017 New Revision: 294501 URL: http://llvm.org/viewvc/llvm-project?rev=294501=rev Log: Update info for LWG 2665 in upcoming_meeting.html Modified: libcxx/trunk/www/upcoming_meeting.html Modified: libcxx/trunk/www/upcoming_meeting.html URL:

[PATCH] D29723: [Sema] Add lvalue-to-rvalue cast in direct-list-initialization of enum

vsk created this revision. After r264564, we allowed direct-list-initialization of an enum from an integral value in C++1z mode, so long as that value can convert to the enum's underlying type. In this kind of initialization, we need a lvalue-to-rvalue conversion for the initializer value if it

[PATCH] D29699: [clang-tidy] Add -extra-arg and -extra-arg-before to clang-tidy-diff.py

This revision was automatically updated to reflect the committed changes. Closed by commit rL294491: [clang-tidy] Add -extra-arg and -extra-arg-before to clang-tidy-diff.py (authored by ehsan). Changed prior to commit: https://reviews.llvm.org/D29699?vs=87576=87673#toc Repository: rL LLVM

[clang-tools-extra] r294491 - [clang-tidy] Add -extra-arg and -extra-arg-before to clang-tidy-diff.py

Author: ehsan Date: Wed Feb 8 11:50:24 2017 New Revision: 294491 URL: http://llvm.org/viewvc/llvm-project?rev=294491=rev Log: [clang-tidy] Add -extra-arg and -extra-arg-before to clang-tidy-diff.py Summary: These flags allow specifying extra arguments to the tool's command line which don't

[libcxx] r294483 - Docs: drop minor version

Author: hans Date: Wed Feb 8 11:28:23 2017 New Revision: 294483 URL: http://llvm.org/viewvc/llvm-project?rev=294483=rev Log: Docs: drop minor version Modified: libcxx/branches/release_40/docs/conf.py Modified: libcxx/branches/release_40/docs/conf.py URL:

[PATCH] D29630: [libcxx] Threading support: externalize sleep_for()

rmaprath added a comment. In https://reviews.llvm.org/D29630#670876, @EricWF wrote: > In https://reviews.llvm.org/D29630#670873, @rmaprath wrote: > > > Sorry for the delay, I've updated the patch with all the comments addressed. > > > > @EricWF: Got one question before I commit: The `sleep_for`

[PATCH] D29630: [libcxx] Threading support: externalize sleep_for()

EricWF added a comment. In https://reviews.llvm.org/D29630#670873, @rmaprath wrote: > Sorry for the delay, I've updated the patch with all the comments addressed. > > @EricWF: Got one question before I commit: The `sleep_for` function in the > dylib (`thread.cpp`) is now quite small. Is there a

[PATCH] D29630: [libcxx] Threading support: externalize sleep_for()

rmaprath updated this revision to Diff 87669. rmaprath added a comment. Sorry for the delay, I've updated the patch with all the comments addressed. @EricWF: Got one question before I commit: The `sleep_for` function in the dylib (`thread.cpp`) is now quite small. Is there a particular reason

r294465 - [MS] Fix C++ destructor thunk line info for a declaration

Author: rnk Date: Wed Feb 8 10:09:32 2017 New Revision: 294465 URL: http://llvm.org/viewvc/llvm-project?rev=294465=rev Log: [MS] Fix C++ destructor thunk line info for a declaration Sometimes the MS ABI needs to emit thunks for declarations that don't have bodies. Destructor thunks make calls

[clang-tools-extra] r294467 - [clang-tidy] hasErrorOccurred() -> hasUncompilableErrorOccurred()

Author: alexfh Date: Wed Feb 8 10:11:22 2017 New Revision: 294467 URL: http://llvm.org/viewvc/llvm-project?rev=294467=rev Log: [clang-tidy] hasErrorOccurred() -> hasUncompilableErrorOccurred() hasErrorOccurred() -> hasUncompilableErrorOccurred(), since we only care about errors that lead to

[PATCH] D29621: Add ASTMatchRefactorer and ReplaceNodeWithTemplate to RefactoringCallbacks

ioeric added inline comments. Comment at: lib/Tooling/RefactoringCallbacks.cpp:160 +llvm::Expected +ReplaceNodeWithTemplate::create(StringRef FromId, StringRef ToTemplate) { + std::vector ParsedTemplate; Is this covered in the test?

[PATCH] D29718: [libclang] [OpenCL] Expose half type

svenvh created this revision. Expose the half type (fp16) through libclang and the python bindings. It seems CXType_LastBuiltin was not updated in b2ea6d9 ("Enable support for __float128 in Clang", 2016-04-13), so update it now. Add an Index test for OpenCL types; in the future we will add

[PATCH] D28973: Supresses misc-move-constructor-init warning for const fields.

This revision was automatically updated to reflect the committed changes. Closed by commit rL294459: [clang-tidy] Supresses misc-move-constructor-init warning for const fields. (authored by alexfh). Changed prior to commit: https://reviews.llvm.org/D28973?vs=87484=87655#toc Repository: rL

[clang-tools-extra] r294459 - [clang-tidy] Supresses misc-move-constructor-init warning for const fields.

Author: alexfh Date: Wed Feb 8 08:56:16 2017 New Revision: 294459 URL: http://llvm.org/viewvc/llvm-project?rev=294459=rev Log: [clang-tidy] Supresses misc-move-constructor-init warning for const fields. Patch by CJ DiMeglio! Differential revision: https://reviews.llvm.org/D28973 Modified:

[PATCH] D29716: [clang-format] Move OriginalPrefix from base to subclass.

This revision was automatically updated to reflect the committed changes. Closed by commit rL294457: [clang-format] Move OriginalPrefix from base to subclass. (authored by krasimir). Changed prior to commit: https://reviews.llvm.org/D29716?vs=87643=87653#toc Repository: rL LLVM

[PATCH] D29716: [clang-format] Move OriginalPrefix from base to subclass.

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

r294457 - [clang-format] Move OriginalPrefix from base to subclass.

Author: krasimir Date: Wed Feb 8 08:45:19 2017 New Revision: 294457 URL: http://llvm.org/viewvc/llvm-project?rev=294457=rev Log: [clang-format] Move OriginalPrefix from base to subclass. Summary: OriginalPrefix is only needed for line comment sections. Moved from the base class to the child

[PATCH] D28973: Supresses misc-move-constructor-init warning for const fields.

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. I'll commit the patch for you. https://reviews.llvm.org/D28973 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D29713: [clang-format] Move comment tests to their own file.

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

[PATCH] D29708: [compiler-rt] [test] #ifdef new builtin tests for __arm__ platform

This revision was automatically updated to reflect the committed changes. Closed by commit rL294438: [test] #ifdef new builtin tests for __arm__ platform (authored by mgorny). Changed prior to commit: https://reviews.llvm.org/D29708?vs=87617=87629#toc Repository: rL LLVM

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

This revision was automatically updated to reflect the committed changes. Closed by commit rL294435: [clang-format] Break before a sequence of line comments aligned with the next… (authored by krasimir). Changed prior to commit: https://reviews.llvm.org/D29626?vs=87615=87622#toc Repository:

r294435 - [clang-format] Break before a sequence of line comments aligned with the next line.

Author: krasimir Date: Wed Feb 8 04:30:44 2017 New Revision: 294435 URL: http://llvm.org/viewvc/llvm-project?rev=294435=rev Log: [clang-format] Break before a sequence of line comments aligned with the next line. Summary: Make the comment alignment respect sections of line comments originally

[PATCH] D29692: [clang-tidy] add check modernize-use-const-instead-of-define

malcolm.parsons added inline comments. Comment at: clang-tidy/modernize/UseConstInsteadOfDefineCheck.cpp:20 +namespace { +class MacroParenthesesPPCallbacks : public PPCallbacks { +public: s/MacroParentheses/ConstantValues/ ? Comment at:

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg. I'd probably still call it consumeComments or something, as we require a flush afterwards for the unconsumed comments, but I don't feel too strongly about that.

[PATCH] D29706: [libcxx] [test] Fix hard_link_count test to account for fs with dir nlink==1

This revision was automatically updated to reflect the committed changes. Closed by commit rL294431: [test] Fix hard_link_count test to account for fs with dir nlink==1 (authored by mgorny). Changed prior to commit: https://reviews.llvm.org/D29706?vs=87613=87618#toc Repository: rL LLVM

[libcxx] r294431 - [test] Fix hard_link_count test to account for fs with dir nlink==1

Author: mgorny Date: Wed Feb 8 03:57:32 2017 New Revision: 294431 URL: http://llvm.org/viewvc/llvm-project?rev=294431=rev Log: [test] Fix hard_link_count test to account for fs with dir nlink==1 Filesystems are not required to maintain a hard link count consistent with number of subdirectories.

[PATCH] D29708: [compiler-rt] [test] #ifdef new builtin tests for __arm__ platform

mgorny created this revision. Herald added subscribers: dberris, aemerson. Add an #if that excludes the newly added aeabi* tests on non-ARM platforms. This is consistent with other ARM tests, and aims to make running builtin tests easier. Lacking a proper infrastructure to run tests selectively,

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

krasimir added inline comments. Comment at: lib/Format/UnwrappedLineParser.h:121-123 + // Comments specifies the sequence of comment tokens to analyze. They get + // either pushed to the current line or added to the comments before the next + // token. klimek

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

krasimir updated this revision to Diff 87615. krasimir marked an inline comment as done. krasimir added a comment. - Updated call sites. https://reviews.llvm.org/D29626 Files: lib/Format/UnwrappedLineParser.cpp lib/Format/UnwrappedLineParser.h unittests/Format/FormatTest.cpp Index:

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

krasimir updated this revision to Diff 87614. krasimir added a comment. - Invent better name for comment analysis. https://reviews.llvm.org/D29626 Files: lib/Format/UnwrappedLineParser.cpp lib/Format/UnwrappedLineParser.h unittests/Format/FormatTest.cpp Index:

[PATCH] D29707: Fix improper microsoft-pure-definition warning on template class

Ilod created this revision. Clang emits a warning when using pure specifier =0 in function definition at class scope, which is a MS-specific construct, when using -fms-extensions. However, to detect this, it was using FD->isCanonicalDecl() on function declaration, which was also detecting

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

klimek added inline comments. Comment at: lib/Format/UnwrappedLineParser.h:121-123 + // Comments specifies the sequence of comment tokens to analyze. They get + // either pushed to the current line or added to the comments before the next + // token. krasimir

[PATCH] D29706: [libcxx] [test] Fix hard_link_count test to account for fs with dir nlink==1

mgorny created this revision. Filesystems are not required to maintain a hard link count consistent with number of subdirectories. For example, on btrfs all directories have nlink==1. Account for that in the test. Repository: rL LLVM https://reviews.llvm.org/D29706 Files:

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

krasimir added inline comments. Comment at: lib/Format/UnwrappedLineParser.h:121-123 + // Comments specifies the sequence of comment tokens to analyze. They get + // either pushed to the current line or added to the comments before the next + // token. klimek

[PATCH] D29469: Fix PR31843: Clang-4.0 crashes/assert while evaluating __builtin_object_size

mehdi_amini added a comment. Ping? This is hitting 4.0 https://reviews.llvm.org/D29469 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D29699: [clang-tidy] Add -extra-arg and -extra-arg-before to clang-tidy-diff.py

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

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

klimek added inline comments. Comment at: lib/Format/UnwrappedLineParser.h:121-123 + // Comments specifies the sequence of comment tokens to analyze. They get + // either pushed to the current line or added to the comments before the next + // token. krasimir

[PATCH] D29628: [compiler-rt] [test] Enable the strace_test only if strace is installed

This revision was automatically updated to reflect the committed changes. Closed by commit rL294425: [test] Enable the strace_test only if strace is installed (authored by mgorny). Changed prior to commit: https://reviews.llvm.org/D29628?vs=87392=87603#toc Repository: rL LLVM

[PATCH] D20689: [clang-tidy] Suspicious Call Argument checker

varjujan added a comment. @xazax.hun Yes I do. Obviously some of them seem to be better than the others so I can remove a couple if needed. https://reviews.llvm.org/D20689 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r294424 - [X86] Add -mprefetchwt1/-mno-prefetchwt1 command line options and __PREFETCHWT1__ define to match gcc.

Author: ctopper Date: Wed Feb 8 02:23:40 2017 New Revision: 294424 URL: http://llvm.org/viewvc/llvm-project?rev=294424=rev Log: [X86] Add -mprefetchwt1/-mno-prefetchwt1 command line options and __PREFETCHWT1__ define to match gcc. Modified: cfe/trunk/docs/ClangCommandLineReference.rst

r294423 - [X86] Add -msgx/-mno-sgx command line options and __SGX__ define to match gcc.

Author: ctopper Date: Wed Feb 8 02:23:17 2017 New Revision: 294423 URL: http://llvm.org/viewvc/llvm-project?rev=294423=rev Log: [X86] Add -msgx/-mno-sgx command line options and __SGX__ define to match gcc. Modified: cfe/trunk/docs/ClangCommandLineReference.rst

[PATCH] D26244: [Driver] Add CLANG_PREFER_GCC_LIBRARIES which can be disabled

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

r294420 - [X86] Update command line documentation for -mclwb and -mmovbe which I forgot in my previous commits.

Author: ctopper Date: Wed Feb 8 01:57:01 2017 New Revision: 294420 URL: http://llvm.org/viewvc/llvm-project?rev=294420=rev Log: [X86] Update command line documentation for -mclwb and -mmovbe which I forgot in my previous commits. Modified: cfe/trunk/docs/ClangCommandLineReference.rst

[PATCH] D29626: [clang-format] Break before a sequence of line comments aligned with the next line.

krasimir marked an inline comment as done. krasimir added inline comments. Comment at: lib/Format/UnwrappedLineParser.h:121-123 + // Comments specifies the sequence of comment tokens to analyze. They get + // either pushed to the current line or added to the comments before

r294419 - [X86] Add -mmpx/-mno-mpx command line options and __MPX__ define to match gcc.

Author: ctopper Date: Wed Feb 8 01:56:42 2017 New Revision: 294419 URL: http://llvm.org/viewvc/llvm-project?rev=294419=rev Log: [X86] Add -mmpx/-mno-mpx command line options and __MPX__ define to match gcc. Modified: cfe/trunk/docs/ClangCommandLineReference.rst