[PATCH] D36949: [clang] Fix tests for Emitting Single Inline Remark

2017-08-20 Thread Sam Elliott via Phabricator via cfe-commits
lenary created this revision. This change depends on https://reviews.llvm.org/D36054 and should be landed at the same time. https://reviews.llvm.org/D36949 Files: test/Frontend/optimization-remark-with-hotness.c test/Frontend/optimization-remark.c Index: test/Frontend/optimization-remark

[PATCH] D35782: [C++2a][Preprocessor] Implement p0306 __VA_OPT__ (Comma omission and comma deletion)

2017-08-20 Thread David Majnemer via Phabricator via cfe-commits
majnemer added inline comments. Comment at: include/clang/Lex/TokenLexer.h:169 + bool PasteTokens(Token &Tok, + llvm::ArrayRef AltTokens = llvm::ArrayRef(), + unsigned int *const AltCurTokenIdx = nullptr); I think `llvm::Array

[PATCH] D27918: [analyzer] OStreamChecker

2017-08-20 Thread Anna Zaks via Phabricator via cfe-commits
zaks.anna added inline comments. Comment at: lib/StaticAnalyzer/Checkers/OStreamFormatChecker.cpp:513 + +bool OStreamFormatChecker::evalCall(const CallExpr *CE, +CheckerContext &C) const { gamesh411 wrote: > NoQ wrote: > > One

[PATCH] D27918: [analyzer] OStreamChecker

2017-08-20 Thread Endre Fülöp via Phabricator via cfe-commits
gamesh411 added a comment. Ping. @NoQ would you please have a look? Thanks! https://reviews.llvm.org/D27918 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D36423: [libc++] Introsort based sorting function

2017-08-20 Thread Aditya Kumar via Phabricator via cfe-commits
hiraditya added a comment. Results with the patch. Before: Run on (8 X 3900 MHz CPU s) 2017-08-20 15:11:41 --- BenchmarkTime CPU Iterations --

r311301 - Allow thiscall attribute in test/Tooling/clang-diff-ast.cpp

2017-08-20 Thread Johannes Altmanninger via cfe-commits
Author: krobelus Date: Sun Aug 20 13:13:33 2017 New Revision: 311301 URL: http://llvm.org/viewvc/llvm-project?rev=311301&view=rev Log: Allow thiscall attribute in test/Tooling/clang-diff-ast.cpp Modified: cfe/trunk/test/Tooling/clang-diff-ast.cpp Modified: cfe/trunk/test/Tooling/clang-diff-a

r311170 - [CommonOptionsParser] Expose ArgumentsAdjustingCompilationDatabase

2017-08-20 Thread Johannes Altmanninger via cfe-commits
Author: krobelus Date: Fri Aug 18 09:21:08 2017 New Revision: 311170 URL: http://llvm.org/viewvc/llvm-project?rev=311170&view=rev Log: [CommonOptionsParser] Expose ArgumentsAdjustingCompilationDatabase This is useful for tools such as clang-diff which do not use CommonOptionsParser due to the nee

[PATCH] D35948: [CommonOptionsParser] Expose ArgumentsAdjustingCompilationDatabase

2017-08-20 Thread Johannes Altmanninger via Phabricator via cfe-commits
johannes accepted this revision. johannes added a comment. This revision is now accepted and ready to land. This has already landed. https://reviews.llvm.org/D35948 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bi

[PATCH] D36186: [clang-diff] Improve and test getNodeValue

2017-08-20 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL311292: [clang-diff] Improve and test getNodeValue (authored by krobelus). Changed prior to commit: https://reviews.llvm.org/D36186?vs=110952&id=111893#toc Repository: rL LLVM https://reviews.llvm.o

r311292 - [clang-diff] Improve and test getNodeValue

2017-08-20 Thread Johannes Altmanninger via cfe-commits
Author: krobelus Date: Sun Aug 20 09:18:43 2017 New Revision: 311292 URL: http://llvm.org/viewvc/llvm-project?rev=311292&view=rev Log: [clang-diff] Improve and test getNodeValue Summary: Use qualified names if available. Reviewers: arphaman Subscribers: klimek Differential Revision: https://re

[PATCH] D36527: Implemented P0428R2 - Familiar template syntax for generic lambdas

2017-08-20 Thread Hamza Sood via Phabricator via cfe-commits
hamzasood updated this revision to Diff 111892. hamzasood added a comment. Sorry, I've just spotted a small mistake in how the auto parameter depth is recorded. This update fixes it. https://reviews.llvm.org/D36527 Files: include/clang/Basic/DiagnosticParseKinds.td include/clang/Sema/Scope

[PATCH] D36935: Enable libfuzzer on NetBSD/amd64

2017-08-20 Thread Kamil Rytarowski via Phabricator via cfe-commits
krytarowski created this revision. krytarowski added a project: Sanitizers. Enable SanitizerKind::Fuzzer and SanitizerKind::FuzzerNoLink on x86_64. Sponsored by Repository: rL LLVM https://reviews.llvm.org/D36935 Files: lib/Driver/ToolChains/NetBSD.cpp Index: lib/Driver/ToolChains/NetB

[PATCH] D36527: Implemented P0428R2 - Familiar template syntax for generic lambdas

2017-08-20 Thread Hamza Sood via Phabricator via cfe-commits
hamzasood updated this revision to Diff 111887. hamzasood added a comment. - Corrected a typo. - Made HasExplicitTemplateParams const. - Reverted the change in where template depth is incremented and recorded. - Fixed the broken test. - Keep the template scope active instead of exiting it and manu

[PATCH] D36371: [Clang][x86][Inline Asm] support for GCC style inline asm - Y constraints

2017-08-20 Thread coby via Phabricator via cfe-commits
coby updated this revision to Diff 111885. coby added a comment. Herald added a subscriber: eraman. > @rnk: tests? Add forgotten test. Repository: rL LLVM https://reviews.llvm.org/D36371 Files: lib/Basic/Targets/X86.cpp lib/Basic/Targets/X86.h lib/CodeGen/TargetInfo.cpp test/CodeGen

[PATCH] D35782: [C++2a][Preprocessor] Implement p0306 __VA_OPT__ (Comma omission and comma deletion)

2017-08-20 Thread Faisal Vali via Phabricator via cfe-commits
faisalv added a comment. *ping* Repository: rL LLVM https://reviews.llvm.org/D35782 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D36527: Implemented P0428R2 - Familiar template syntax for generic lambdas

2017-08-20 Thread Faisal Vali via Phabricator via cfe-commits
faisalv added inline comments. Comment at: lib/Parse/ParseExprCXX.cpp:1090 + TemplateParameterDepthRAII CurTemplateDepthTracker(TemplateParameterDepth); + Actions.RecordParsingTemplateParameterDepth(TemplateParameterDepth); + hamzasood wrote: > faisalv wrote: >

r311286 - [ODRHash] Move into anonymous namespace. NFC.

2017-08-20 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sun Aug 20 06:02:57 2017 New Revision: 311286 URL: http://llvm.org/viewvc/llvm-project?rev=311286&view=rev Log: [ODRHash] Move into anonymous namespace. NFC. Modified: cfe/trunk/lib/AST/ODRHash.cpp Modified: cfe/trunk/lib/AST/ODRHash.cpp URL: http://llvm.org/viewvc/llvm-pr

[PATCH] D36527: Implemented P0428R2 - Familiar template syntax for generic lambdas

2017-08-20 Thread Hamza Sood via Phabricator via cfe-commits
hamzasood added inline comments. Comment at: lib/Parse/ParseExprCXX.cpp:1090 + TemplateParameterDepthRAII CurTemplateDepthTracker(TemplateParameterDepth); + Actions.RecordParsingTemplateParameterDepth(TemplateParameterDepth); + faisalv wrote: > Since you only r

[PATCH] D36185: [clang-diff] Fix similarity computation

2017-08-20 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL311284: [clang-diff] Fix similarity computation (authored by krobelus). Changed prior to commit: https://reviews.llvm.org/D36185?vs=110951&id=111877#toc Repository: rL LLVM https://reviews.llvm.org/

r311284 - [clang-diff] Fix similarity computation

2017-08-20 Thread Johannes Altmanninger via cfe-commits
Author: krobelus Date: Sun Aug 20 05:09:07 2017 New Revision: 311284 URL: http://llvm.org/viewvc/llvm-project?rev=311284&view=rev Log: [clang-diff] Fix similarity computation Summary: Add separate tests for the top-down and the bottom-up phase, as well as one for the optimal matching. Reviewers:

[libcxx] r311283 - [NFC] remove trailing WS

2017-08-20 Thread Aditya Kumar via cfe-commits
Author: hiraditya Date: Sun Aug 20 03:38:55 2017 New Revision: 311283 URL: http://llvm.org/viewvc/llvm-project?rev=311283&view=rev Log: [NFC] remove trailing WS Modified: libcxx/trunk/include/memory Modified: libcxx/trunk/include/memory URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/

[PATCH] D36184: [clang-diff] Filter AST nodes

2017-08-20 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL311280: [clang-diff] Filter AST nodes (authored by krobelus). Changed prior to commit: https://reviews.llvm.org/D36184?vs=110687&id=111872#toc Repository: rL LLVM https://reviews.llvm.org/D36184 Fil

r311280 - [clang-diff] Filter AST nodes

2017-08-20 Thread Johannes Altmanninger via cfe-commits
Author: krobelus Date: Sun Aug 20 03:22:32 2017 New Revision: 311280 URL: http://llvm.org/viewvc/llvm-project?rev=311280&view=rev Log: [clang-diff] Filter AST nodes Summary: Ignore macros and implicit AST nodes, as well as anything outside of the main source file. Reviewers: arphaman Subscriber

Re: Buildmaster restart 08.20.2017

2017-08-20 Thread Victor Leschuk via cfe-commits
Both buildmasters were updated and currently are up and running. On 08/20/2017 10:47 AM, Victor Leschuk wrote: > Hello everyone, > LLVM buildmasters (both main and staging) will be restarted in 2 hours > (~3:00 AM PDT). > -- Best Regards, Victor Leschuk | Software Engineer |Access Softek

Buildmaster restart 08.20.2017

2017-08-20 Thread Victor Leschuk via cfe-commits
Hello everyone, LLVM buildmasters (both main and staging) will be restarted in 2 hours (~3:00 AM PDT). -- Best Regards, Victor Leschuk | Software Engineer |Access Softek ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi

[PATCH] D36764: The following functions and tests work fine for powerpc64, so enable them.

2017-08-20 Thread Nemanja Ivanovic via Phabricator via cfe-commits
nemanjai added a comment. In https://reviews.llvm.org/D36764#844692, @joerg wrote: > divtc3 and friends. Ah, OK. I see what you mean now. These builtins are for `XCmode` calculations (complex values as two `XFmode` components). Since PPC has no support for `XFmode` (i.e. 80-bit long double) i