The test seems to be failing:
llvm/tools/clang/unittests/ASTMatchers/Dynamic/RegistryTest.cpp:533: Failure
Value of: matches("double x = 1.2f;", DoubleStmt)
Actual: false (Could not find match in "double x = 1.2f;")
Expected: true
On Thu, Jun 8, 2017 at 3:00 PM, Peter Wu via cfe-commits <
cfe-c
Author: kcc
Date: Thu Jun 8 17:58:19 2017
New Revision: 305026
URL: http://llvm.org/viewvc/llvm-project?rev=305026&view=rev
Log:
[sanitizer-coverage] one more flavor of coverage:
-fsanitize-coverage=inline-8bit-counters. Experimental so far, not documenting
yet. Reapplying revisions 304630, 304
Author: lekensteyn
Date: Thu Jun 8 17:58:12 2017
New Revision: 305025
URL: http://llvm.org/viewvc/llvm-project?rev=305025&view=rev
Log:
[ASTMatchers] temporary disable tests with floating suffix
r305022 assumed that floatLiteral(equals(1.2)) would also match 1.2f and
1.2l, but apparently that is
wash added inline comments.
Comment at: include/numeric:182
+{
+ _Tp __saved = __init;
+ for (; __first != __last; ++__first, (void) ++__result) {
If `__first == __last`, this initialization is unnecessary; I'd refactor this
so that we check for `__f
Yes, mea culpa :(
I forgot to rebuild after doing the final revision of the patch (in
which two new tests were added). The failing tests are temporary
disabled in r305025 until a proper fix is developed.
Kind regards,
Peter
On Thu, Jun 08, 2017 at 03:58:01PM -0700, Kostya Serebryany wrote:
> The
rjmccall accepted this revision.
rjmccall added inline comments.
Comment at: include/clang/AST/Decl.h:901
+/// member functions.
+unsigned ImplicitParamKind : 3;
};
aaron.ballman wrote:
> rjmccall wrote:
> > ABataev wrote:
> > > aaron.ballman wrote:
>
wash added a comment.
https://gist.github.com/brycelelbach/137f1e45b737d615134e228ec0c84f3a <- some
crude tests I wrote based on slides/notes of mine.
https://reviews.llvm.org/D34038
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://li
chh added a comment.
Android source is suppressing misc-noexcept-move-constructor warnings
because -fno-exceptions is used and Android does not like to add more
exception specific code.
It's not a big deal for Android to suppress this check one way or the other.
I don't mind reverting it, just cu
wash added a comment.
This implementation works, but performs unnecessary operations.
https://gist.github.com/brycelelbach/907ac3b8a74603cc189897ab789a65a9
The "next" result is calculated in each iteration; this means one unnecessary
application of the binary op (the "next" result on the final
Author: compnerd
Date: Thu Jun 8 19:40:30 2017
New Revision: 305039
URL: http://llvm.org/viewvc/llvm-project?rev=305039&view=rev
Log:
Represent debug information compression type fully
This is tied with the LLVM side of the change to expose the debug
information compression types to clang. We n
Author: akirtzidis
Date: Thu Jun 8 20:20:48 2017
New Revision: 305044
URL: http://llvm.org/viewvc/llvm-project?rev=305044&view=rev
Log:
[libclang] Introduce a new parsing option 'CXTranslationUnit_SingleFileParse'
that puts preprocessor in a mode for parsing a single file only.
This is useful f
EricWF added inline comments.
Comment at: test/SemaCXX/coawait_range_for.cpp:133
+
+struct ForLoopAwaiterCoawaitLookup {
+ struct promise_type {
This test is incorrect WRT ADL lookup.
https://reviews.llvm.org/D34021
Author: rsmith
Date: Thu Jun 8 20:36:10 2017
New Revision: 305045
URL: http://llvm.org/viewvc/llvm-project?rev=305045&view=rev
Log:
Remove 'Filename' parameter from BeginSourceFileAction.
No-one was using this, and it's not meaningful in general -- FrontendActions
can be run on inputs that don't
Author: akirtzidis
Date: Thu Jun 8 21:04:19 2017
New Revision: 305046
URL: http://llvm.org/viewvc/llvm-project?rev=305046&view=rev
Log:
[clangd] Update for ASTUnit API change.
Modified:
clang-tools-extra/trunk/clangd/ClangdUnit.cpp
Modified: clang-tools-extra/trunk/clangd/ClangdUnit.cpp
URL
mclow.lists added a comment.
> Should the non-parallel implementation of reduce static_assert or SFINAE away
> when these requirements are not met?
That may be desirable, but that's not what the standard says.
There's nothing there about "shall not participate in overload resolution".
//Require
mclow.lists updated this revision to Diff 101988.
mclow.lists added a comment.
Added a `_VSTD::`, made some assertions `static_assert`, and addressed Bryce's
concerns about doing an init when the input range is empty.
https://reviews.llvm.org/D34038
Files:
include/numeric
test/std/numeric
mclow.lists updated this revision to Diff 101989.
mclow.lists added a comment.
Turn the for loops into do-while loops, saving a comparison.
https://reviews.llvm.org/D34038
Files:
include/numeric
test/std/numerics/numeric.ops/exclusive.scan/exclusive_scan_iter_iter_iter.pass.cpp
test/std
sepavloff added a comment.
Thank you for explanation. Probably making new semantic action is an overkill.
The intent was to postpone setting flag `WillHaveBody` as late as possible, to
the moment when it becomes clear that the function indeed will have a body.
Otherwise parsing of deleted funct
sepavloff updated this revision to Diff 101993.
sepavloff added a comment.
Changed implementation
https://reviews.llvm.org/D30375
Files:
include/clang/AST/Decl.h
lib/Sema/SemaCUDA.cpp
lib/Sema/SemaDecl.cpp
lib/Sema/SemaDeclCXX.cpp
lib/Sema/SemaTemplateInstantiateDecl.cpp
test/SemaCX
yvvan updated this revision to Diff 101995.
yvvan added a comment.
"what kind of performance benefits do you get for the preamble load times?"
In cases where many windows headers are included preamble loading takes almost
the half of reparse time. With this fix time to load preamble goes towards
101 - 120 of 120 matches
Mail list logo
