twoh added a comment.
@eric_niebler I just tried it on Windows machine, and it just succeeded with no
warnings/fix-it. Is that expected?
https://reviews.llvm.org/D3
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Hahnfeld added a subscriber: mgorny.
Hahnfeld added a comment.
Yes, that's the current state. We had some discussion on cfe-dev on that
matter: http://lists.llvm.org/pipermail/cfe-dev/2017-January/052512.html
Could you slightly adapt your patch so that other runtime libraries can follow
this
pirama added a comment.
In https://reviews.llvm.org/D30015#678243, @Hahnfeld wrote:
> Why is this only for OpenMP? I imagine this should be done for **all**
> runtime libraries
The other runtime libraries have the arch included in the library's name and
the driver links the correct file.
Author: rtrieu
Date: Thu Feb 16 01:09:18 2017
New Revision: 295293
URL: http://llvm.org/viewvc/llvm-project?rev=295293=rev
Log:
Revert r295284: Add better ODR checking for modules.
Fix modules build bot.
Removed:
cfe/trunk/include/clang/AST/ODRHash.h
cfe/trunk/lib/AST/ODRHash.cpp
Hahnfeld added a comment.
Why is this only for OpenMP? I imagine this should be done for **all** runtime
libraries
https://reviews.llvm.org/D30015
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
mgorny created this revision.
Herald added a subscriber: dberris.
Make the use of #pragma redefine_extname and appropriate renames of
builtins conditional to using clang. GCC used not to support it outside
Solaris and currently seems to be very restrictive on applying it.
In other words, it does
Author: ctopper
Date: Thu Feb 16 00:32:07 2017
New Revision: 295291
URL: http://llvm.org/viewvc/llvm-project?rev=295291=rev
Log:
[AVX-512] Replace 512-bit masked packss/packus builtins and replace with new
unmasked builtins.
These new unmasked builtins will enable us to easily support
The assert below is throwing a -Wsign-compare warning because std::distance
returns a signed type.
+ const size_t NumFields =
+ std::distance(ClosureClass->field_begin(),
ClosureClass->field_end());
+
+ assert(NumFields ==
+std::distance(E->capture_init_begin(), E->capture_init_end())
Author: rtrieu
Date: Wed Feb 15 23:48:25 2017
New Revision: 295286
URL: http://llvm.org/viewvc/llvm-project?rev=295286=rev
Log:
Loosen a Type check ODR checking to try to fix the build bot.
Modified:
cfe/trunk/lib/Serialization/ASTReader.cpp
Modified:
Author: phosek
Date: Wed Feb 15 23:18:08 2017
New Revision: 295285
URL: http://llvm.org/viewvc/llvm-project?rev=295285=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
Author: rtrieu
Date: Wed Feb 15 22:53:40 2017
New Revision: 295284
URL: http://llvm.org/viewvc/llvm-project?rev=295284=rev
Log:
Add better ODR checking for modules.
Recommit r293585 that was reverted in r293611 with new fixes. The previous
issue was determined to be an overly aggressive AST
Author: rsmith
Date: Wed Feb 15 22:22:56 2017
New Revision: 295281
URL: http://llvm.org/viewvc/llvm-project?rev=295281=rev
Log:
Revert r295277 to fix buildbot.
Modified:
cfe/trunk/include/clang/Basic/Diagnostic.h
cfe/trunk/include/clang/Sema/Sema.h
On 15 February 2017 at 20:12, Faisal Vali via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: faisalv
> Date: Wed Feb 15 22:12:21 2017
> New Revision: 295279
>
> URL: http://llvm.org/viewvc/llvm-project?rev=295279=rev
> Log:
> [cxx1z-constexpr-lambda] Implement captures - thus
Author: faisalv
Date: Wed Feb 15 22:12:21 2017
New Revision: 295279
URL: http://llvm.org/viewvc/llvm-project?rev=295279=rev
Log:
[cxx1z-constexpr-lambda] Implement captures - thus completing implementation of
constexpr lambdas.
Enable evaluation of captures within constexpr lambdas by using a
Author: rsmith
Date: Wed Feb 15 21:49:44 2017
New Revision: 295277
URL: http://llvm.org/viewvc/llvm-project?rev=295277=rev
Log:
Add missing "deduced A == A" check for function template partial ordering.
This appears to be the only template argument deduction context where we were
missing this
herzka updated this revision to Diff 88652.
herzka added a comment.
Rename selectors
https://reviews.llvm.org/D29967
Files:
lib/Sema/SemaExprObjC.cpp
test/SemaObjC/objc-class-property.m
Index: test/SemaObjC/objc-class-property.m
herzka updated this revision to Diff 88650.
herzka added a comment.
Added test, used auto
https://reviews.llvm.org/D29967
Files:
lib/Sema/SemaExprObjC.cpp
test/SemaObjC/objc-class-property.m
Index: test/SemaObjC/objc-class-property.m
hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D29778
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
srhines added inline comments.
Comment at: include/clang/Driver/ToolChain.h:304
+ // Returns /lib//. When -fopenmp is specified,
+ // this directory is added to the linker search path if it exists
+ const std::string getOpenMPLibPath() const;
Add a period at
dberris created this revision.
Herald added a subscriber: mehdi_amini.
We define the `__xray_customeevent` builtin that gets translated to
IR calls to the correct intrinsic. The default implementation of this is
a no-op function. The codegen side of this follows the following logic:
- When
On 15 February 2017 at 17:45, Hal Finkel via Phabricator via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> hfinkel added a comment.
>
> I don't understand why it only supports some attributes. Is there some
> handling that needs to take place (I don't see anything obvious in this
> patch)?
rsmith added a comment.
In https://reviews.llvm.org/D28510#654821, @faisalv wrote:
> In https://reviews.llvm.org/D28510#653794, @rsmith wrote:
>
> > I don't think it's possible to check this in the way you're doing so here.
> > In general, there's no way to know whether a constant expression
Author: eugenis
Date: Wed Feb 15 19:35:23 2017
New Revision: 295267
URL: http://llvm.org/viewvc/llvm-project?rev=295267=rev
Log:
Add missing regexp quantifiers in a test.
Modified:
cfe/trunk/test/CodeGen/sanitize-init-order.cpp
Modified: cfe/trunk/test/CodeGen/sanitize-init-order.cpp
URL:
hfinkel added a comment.
I don't understand why it only supports some attributes. Is there some handling
that needs to take place (I don't see anything obvious in this patch)? If most
attributes will "just work", I'd much rather opt-out the few exceptions (which
we can then explicitly
danalbert accepted this revision.
danalbert added a comment.
This revision is now accepted and ready to land.
LGTM, but should probably get signoff from someone else as well.
https://reviews.llvm.org/D30015
___
cfe-commits mailing list
On 15 February 2017 at 17:32, George Burgess IV wrote:
> I remember that we wanted to pretend that pass_object_size isn't a part of
> the FunctionType during the review that added it, though.
>
I remember we wanted to not add extra fake "parameters" to the
I remember that we wanted to pretend that pass_object_size isn't a part of
the FunctionType during the review that added it, though. Do you think that
would be better than serializing parameters before we serialize template
info? AFAICT, we only do merging after we start reading the template info,
This revision was automatically updated to reflect the committed changes.
Closed by commit rL295266: [Sema] Add lvalue-to-rvalue cast in
direct-list-initialization of enum (authored by vedantk).
Changed prior to commit:
https://reviews.llvm.org/D29723?vs=87679=88632#toc
Repository:
rL LLVM
Author: vedantk
Date: Wed Feb 15 19:20:00 2017
New Revision: 295266
URL: http://llvm.org/viewvc/llvm-project?rev=295266=rev
Log:
[Sema] Add lvalue-to-rvalue cast in direct-list-initialization of enum
After r264564, we allowed direct-list-initialization of an enum from an
integral value in C++1z
probinson added a comment.
In https://reviews.llvm.org/D28404#675687, @chandlerc wrote:
> In https://reviews.llvm.org/D28404#675616, @mehdi_amini wrote:
>
> > We're still waiting for @rsmith to comment whether it'd be better to `have
> > a LangOpts flag that basically means "pragma clang
jgravelle-google marked an inline comment as done.
jgravelle-google added a comment.
Thanks, added a comment
https://reviews.llvm.org/D29778
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
On 15 February 2017 at 14:43, George Burgess IV via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: gbiv
> Date: Wed Feb 15 16:43:27 2017
> New Revision: 295252
>
> URL: http://llvm.org/viewvc/llvm-project?rev=295252=rev
> Log:
> [Modules] Consider enable_if attrs in isSameEntity.
>
>
jgravelle-google updated this revision to Diff 88629.
jgravelle-google added a comment.
- Add comment to lgamma builtins
https://reviews.llvm.org/D29778
Files:
include/clang/Basic/Builtins.def
test/CodeGen/libcall-declarations.c
Index: test/CodeGen/libcall-declarations.c
Author: rsmith
Date: Wed Feb 15 18:36:47 2017
New Revision: 295264
URL: http://llvm.org/viewvc/llvm-project?rev=295264=rev
Log:
[c++1z] Diagnose non-deducible template parameters in deduction guide
templates, per [temp.param]p11.
Modified:
https://bugs.llvm.org/show_bug.cgi?id=31977 makes the good point that this
is warning on a certain idiomatic use of capture-by-value to extend the
lifetime of an RAII object; consider:
shared_ptr p = /*...*/;
int *q = >n;
return [=, p] { return *q++; }
Here, we'll warn that the capture of
arphaman added inline comments.
Comment at: docs/LanguageExtensions.rst:2349
+attribute is supported by the pragma by referring to the
+:doc:`individual documentation for that attribute `.
efriedma wrote:
> I'm wondering if we can tweak the approach so that we
smeenai updated this revision to Diff 88627.
smeenai edited the summary of this revision.
smeenai removed a reviewer: dexonsmith.
smeenai removed a subscriber: dexonsmith.
smeenai added a comment.
Guarding behind ABI macro
https://reviews.llvm.org/D27387
Files:
include/__config
arphaman added a comment.
I'm quite surprised by the fact that we have to change so many diagnostics in
tests. Do these diagnostics have the full qualifiers for errors that occur with
record types instead of typedefs?
https://reviews.llvm.org/D29944
efriedma added inline comments.
Comment at: docs/LanguageExtensions.rst:2344
+declaration failed to receive the attribute because of a compilation error. The
+attributes that aren't applied to any declaration are not verified
semantically.
+
I think "to each
efriedma added a comment.
(Resigning as a reviewer; I don't know enough about standard conversion
sequences off the top of my head to review this properly.)
https://reviews.llvm.org/D29723
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
vsk added a comment.
Ping.
https://reviews.llvm.org/D29723
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: hans
Date: Wed Feb 15 17:28:10 2017
New Revision: 295258
URL: http://llvm.org/viewvc/llvm-project?rev=295258=rev
Log:
[dllimport] Check for dtor references in functions
Destructor references are not modelled explicitly in the AST. This adds
checks for destructor calls due to variable
Author: hans
Date: Wed Feb 15 17:28:07 2017
New Revision: 295257
URL: http://llvm.org/viewvc/llvm-project?rev=295257=rev
Log:
[dllimport] Look through typedefs and arrays in HasNonDllImportDtor
The function is used to check whether a type is a class with
non-dllimport destructor. It needs to
Author: arphaman
Date: Wed Feb 15 17:24:15 2017
New Revision: 295256
URL: http://llvm.org/viewvc/llvm-project?rev=295256=rev
Log:
[Docs] Add missing space to an option doc
Modified:
cfe/trunk/include/clang/Driver/Options.td
Modified: cfe/trunk/include/clang/Driver/Options.td
URL:
tigerleapgorge updated this revision to Diff 88617.
tigerleapgorge added a comment.
Changed "CHECK11" to "CHECK11-NEXT".
https://reviews.llvm.org/D24812
Files:
test/CodeGenCXX/mangle-unnamed.cpp
test/CodeGenCXX/static-init.cpp
test/CodeGenCXX/volatile-1.cpp
test/CodeGenCXX/volatile.cpp
compnerd requested changes to this revision.
compnerd added a comment.
This revision now requires changes to proceed.
Please add a test for this.
Comment at: lib/Sema/SemaExprObjC.cpp:1989
+ Selector SetterSel;
+ if (ObjCPropertyDecl *PD = IFace->FindPropertyDeclaration(
+
beanz added a comment.
Thanks for all your work on these!
Repository:
rL LLVM
https://reviews.llvm.org/D26654
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arphaman created this revision.
This patch adds support for the `#pragma clang attribute` directive that was
proposed recently at
http://lists.llvm.org/pipermail/cfe-dev/2017-February/052689.html.
Initially it supports the `annotate`, `require_constant_initialization` and
Author: gbiv
Date: Wed Feb 15 16:43:27 2017
New Revision: 295252
URL: http://llvm.org/viewvc/llvm-project?rev=295252=rev
Log:
[Modules] Consider enable_if attrs in isSameEntity.
Two functions that differ only in their enable_if attributes are
considered overloads, so we should check for those
Author: rafael
Date: Wed Feb 15 16:19:04 2017
New Revision: 295245
URL: http://llvm.org/viewvc/llvm-project?rev=295245=rev
Log:
Fix the static build.
Modified:
cfe/trunk/tools/c-index-test/CMakeLists.txt
Modified: cfe/trunk/tools/c-index-test/CMakeLists.txt
URL:
zaks.anna added a comment.
Does the code you added detects array out of bounds cases without false
positives? Is it an option to just have this checkers produce a more precise
error message in the specific case.
A lot of work will probably need to be done to implement a proper array out of
zaks.anna added a comment.
Please, make sure future reviews go through cfg-dev list. See
http://llvm.org/docs/Phabricator.html.
Repository:
rL LLVM
https://reviews.llvm.org/D28297
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
rnk added inline comments.
Comment at: test/CodeGenCXX/static-init.cpp:14
+// CHECK98: @_ZZN5test414useStaticLocalEvE3obj = linkonce_odr global
%"struct.test4::HasVTable" zeroinitializer, comdat, align 8
+// CHECK11: @_ZZN5test414useStaticLocalEvE3obj = linkonce_odr global {
eric_niebler added a comment.
My question was more about whether the code is correct for absolute paths on
Windows, not about whether this particular test would pass or fail. Have you
tested an incorrectly-cased absolute path on a Windows machine?
https://reviews.llvm.org/D3
erichkeane updated this revision to Diff 88602.
erichkeane added a comment.
I was able to get the templated versions working in response to the discussion
with Akira. Note the added test file which shows off all of the combos I could
think of.
It required a little bit of surgery inside the
Hi All,
I am re-submitting my patch for Bug 30413, this time with a test case included
as well (ivar-type-encoding.m). The test case file should be added to
clang/test/CodeGenObjC. The test verifies that correct metadata is emitted by
clang for an object-valued instance variable. I've
On Wed, Feb 15, 2017 at 12:17 PM, Richard Smith wrote:
> On 15 February 2017 at 11:50, Hans Wennborg wrote:
>>
>> +Richard for risk/reward analysis.
>
>
> This is an extremely safe change, and fixes what amounts to a subtle
> miscompile. I think we
On Wed, Feb 15, 2017 at 12:14 PM, Richard Smith wrote:
> On 15 February 2017 at 11:39, Hans Wennborg wrote:
>>
>> IIUC, this is a follow-up to r291955, which was merged to 4.0. Should
>> this one be merged also?
>
>
> Yes, please.
r295233.
>
>>
>>
On 15 February 2017 at 11:50, Hans Wennborg wrote:
> +Richard for risk/reward analysis.
>
This is an extremely safe change, and fixes what amounts to a subtle
miscompile. I think we should take it.
> r274291 was also in 3.9, so this isn't strictly speaking a regression.
>
>
Hans, this would be a good candidate for Clang 4. The bug in question is
not a regression, but it is an accepts-invalid / wrong-code bug.
On 15 February 2017 at 11:57, Richard Smith via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: rsmith
> Date: Wed Feb 15 13:57:10 2017
> New
On 15 February 2017 at 11:39, Hans Wennborg wrote:
> IIUC, this is a follow-up to r291955, which was merged to 4.0. Should
> this one be merged also?
>
Yes, please.
> Cheers,
> Hans
>
> On Tue, Feb 14, 2017 at 8:18 PM, Richard Smith via cfe-commits
>
Author: rsmith
Date: Wed Feb 15 13:57:10 2017
New Revision: 295224
URL: http://llvm.org/viewvc/llvm-project?rev=295224=rev
Log:
PR24440: Do not silently discard a fold-expression appearing as the operand of
a cast-expression.
Modified:
cfe/trunk/lib/Parse/ParseExpr.cpp
+Richard for risk/reward analysis.
r274291 was also in 3.9, so this isn't strictly speaking a regression.
On Wed, Feb 15, 2017 at 11:43 AM, Akira Hatanaka wrote:
> Hans,
>
> Can this be merged to 4.0 too?
>
>> On Feb 14, 2017, at 9:15 PM, Akira Hatanaka via cfe-commits
>>
twoh updated this revision to Diff 88584.
twoh added a comment.
Make it explicit that the test doesn't support windows. @eric_niebler, my
original intention was avoiding use of platform-dependent path separator, but
now made it explicit that the test is not for windows, it should be okay to use
Hans,
Can this be merged to 4.0 too?
> On Feb 14, 2017, at 9:15 PM, Akira Hatanaka via cfe-commits
> wrote:
>
> Author: ahatanak
> Date: Tue Feb 14 23:15:28 2017
> New Revision: 295150
>
> URL: http://llvm.org/viewvc/llvm-project?rev=295150=rev
> Log:
> [Sema]
alekseyshl added a comment.
http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/2437 is
broken by this revision.
Repository:
rL LLVM
https://reviews.llvm.org/D29886
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
IIUC, this is a follow-up to r291955, which was merged to 4.0. Should
this one be merged also?
Cheers,
Hans
On Tue, Feb 14, 2017 at 8:18 PM, Richard Smith via cfe-commits
wrote:
> Author: rsmith
> Date: Tue Feb 14 22:18:23 2017
> New Revision: 295149
>
> URL:
efriedma added inline comments.
Comment at: include/clang/Basic/Builtins.def:1091
+LIBBUILTIN(lgammaf, "ff", "fn", "math.h", ALL_LANGUAGES)
+LIBBUILTIN(lgammal, "LdLd", "fn", "math.h", ALL_LANGUAGES)
Please add a comment explaining why this doesn't have an "e"
idlecode created this revision.
idlecode added a project: clang-tools-extra.
Herald added a subscriber: JDevlieghere.
Fix generated by modernize-use-override caused syntax error when method
used try-statement as a body. `override` keyword was inserted after last
declaration token which happened
eric_niebler added a subscriber: karies.
eric_niebler added inline comments.
Comment at: lib/Lex/PPDirectives.cpp:1983
+ isLeadingSeparator = false;
+else
+ Path.append(Component);
What happens on Windows for an absolute path like
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM, thank you for working on this!
https://reviews.llvm.org/D23421
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
twoh created this revision.
This is a patch for PR31836. As the bug replaces the path separators in the
included file name with the characters following them, the test script makes
sure that there's no "Ccase-insensitive-include-pr31836.h" in the warning
message.
Michal requested this to be merged in PR31965, it seems fine to me,
and Marshall OK'd it in an email to me. Merged in r295214.
On Wed, Feb 8, 2017 at 1:57 AM, Michal Gorny via cfe-commits
wrote:
> Author: mgorny
> Date: Wed Feb 8 03:57:32 2017
> New Revision: 294431
Author: hans
Date: Wed Feb 15 12:40:52 2017
New Revision: 295214
URL: http://llvm.org/viewvc/llvm-project?rev=295214=rev
Log:
Merging r294431:
r294431 | mgorny | 2017-02-08 01:57:32 -0800 (Wed, 08 Feb 2017) | 7 lines
[test]
phosek updated this revision to Diff 88575.
Repository:
rL LLVM
https://reviews.llvm.org/D26654
Files:
cmake/caches/Fuchsia-stage2.cmake
cmake/caches/Fuchsia.cmake
Index: cmake/caches/Fuchsia.cmake
===
--- /dev/null
+++
bkramer added inline comments.
Comment at: clangd/ProtocolHandlers.cpp:117
+ // starting from there.
+ StringRef Code = Store.getDocument(DOTFP->textDocument.uri);
+ size_t CursorPos = positionToOffset(Code, DOTFP->position);
This should be a std::string in
Author: jdevlieghere
Date: Wed Feb 15 11:37:58 2017
New Revision: 295207
URL: http://llvm.org/viewvc/llvm-project?rev=295207=rev
Log:
[clang-tidy] Fix test modernize-return-braced-init-list
Modified:
clang-tools-extra/trunk/test/clang-tidy/modernize-return-braced-init-list.cpp
Modified:
Author: jdevlieghere
Date: Wed Feb 15 11:19:44 2017
New Revision: 295205
URL: http://llvm.org/viewvc/llvm-project?rev=295205=rev
Log:
Fixed indentation issue in release notes
Modified:
clang-tools-extra/trunk/docs/ReleaseNotes.rst
Modified: clang-tools-extra/trunk/docs/ReleaseNotes.rst
URL:
Author: phosek
Date: Wed Feb 15 11:15:41 2017
New Revision: 295202
URL: http://llvm.org/viewvc/llvm-project?rev=295202=rev
Log:
Revert "[libunwind][CMake] Use libc++ headers when available"
This causing build failure on sanitizer bots because of the unused
argument '-nostdinc++' during linking
sdardis added a comment.
Ok, I'll have `__mips_abicalls` unconditionally defined for everyone, The other
two BSD's will have the traditional macro defined as well.
https://reviews.llvm.org/D29032
___
cfe-commits mailing list
Author: jdevlieghere
Date: Wed Feb 15 11:06:06 2017
New Revision: 295199
URL: http://llvm.org/viewvc/llvm-project?rev=295199=rev
Log:
[clang-tidy] Add check 'modernize-return-braced-init-list'
Summary:
Replaces explicit calls to the constructor in a return with a braced
initializer list. This
Author: d0k
Date: Wed Feb 15 11:04:57 2017
New Revision: 295198
URL: http://llvm.org/viewvc/llvm-project?rev=295198=rev
Log:
[clangd] Fix another use after free that I missed because COW strings.
Modified:
clang-tools-extra/trunk/clangd/ProtocolHandlers.cpp
Modified:
Author: d0k
Date: Wed Feb 15 10:58:44 2017
New Revision: 295196
URL: http://llvm.org/viewvc/llvm-project?rev=295196=rev
Log:
[clangd] Silence GCC warning about falling off a fully covered switch.
Modified:
clang-tools-extra/trunk/clangd/ASTManager.cpp
Modified:
erichkeane added a comment.
Ah, I see! Sorry for missing that. I don't see a reason why we cannot support
that, but I wasn't really considering it. In general, this attribute is a
compiler hint for some C standard library stuff in glibc.
I've been playing with it a few hours now, and it
Author: d0k
Date: Wed Feb 15 10:44:11 2017
New Revision: 295194
URL: http://llvm.org/viewvc/llvm-project?rev=295194=rev
Log:
[clangd] Synchronize logs access.
I don't think that this is necessary for correctness, but makes tsan
much more useful.
Modified:
Author: d0k
Date: Wed Feb 15 10:34:58 2017
New Revision: 295193
URL: http://llvm.org/viewvc/llvm-project?rev=295193=rev
Log:
[clangd] Initialize the thread after the mutex.
Otherwise locking the mutex yields a racy assertion failure on picky
implementations.
Modified:
JDevlieghere updated this revision to Diff 88550.
JDevlieghere added a comment.
Fixed latest comment from @aaron.ballman before landing.
Repository:
rL LLVM
https://reviews.llvm.org/D28768
Files:
clang-tidy/modernize/CMakeLists.txt
clang-tidy/modernize/ModernizeTidyModule.cpp
Author: malcolm.parsons
Date: Wed Feb 15 10:32:55 2017
New Revision: 295192
URL: http://llvm.org/viewvc/llvm-project?rev=295192=rev
Log:
[clang-tidy] Don't delay parsing of templates in test for
misc-unconventional-assign-operator
Modified:
NoQ added inline comments.
Comment at:
cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:73
+ : II(nullptr), IsLookupDone(false), FuncName(FuncName),
+RequiredArgs(RequiredArgs) {}
xazax.hun wrote:
> NoQ wrote:
> > Maybe
Author: akirtzidis
Date: Wed Feb 15 10:16:27 2017
New Revision: 295191
URL: http://llvm.org/viewvc/llvm-project?rev=295191=rev
Log:
[index] USR generation: use getTemplateArgs() instead of
getTemplateInstantiationArgs()
Otherwise we may end up creating a different USR for the definition of a
xazax.hun added inline comments.
Comment at:
cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:73
+ : II(nullptr), IsLookupDone(false), FuncName(FuncName),
+RequiredArgs(RequiredArgs) {}
NoQ wrote:
> Maybe
NoQ added inline comments.
Comment at:
cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:73
+ : II(nullptr), IsLookupDone(false), FuncName(FuncName),
+RequiredArgs(RequiredArgs) {}
Maybe `assert(FuncName.size() > 0)` here?
xazax.hun added a comment.
In https://reviews.llvm.org/D29884#677387, @NoQ wrote:
> Yep, seems that somebody has missed these issues :)
>
> I guess there's no way to test the operator case, because nobody made a
> CallDescription with an empty name for us (maybe we should even assert that).
alexfh 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)
+<<
xazax.hun updated this revision to Diff 88541.
xazax.hun added a comment.
- Do not warn for function specializations within the std namespace.
- Add a test case for swap.
https://reviews.llvm.org/D23421
Files:
clang-tidy/cert/CERTTidyModule.cpp
clang-tidy/cert/CMakeLists.txt
Author: d0k
Date: Wed Feb 15 09:56:14 2017
New Revision: 295187
URL: http://llvm.org/viewvc/llvm-project?rev=295187=rev
Log:
[clangd] Fix use after free.
Modified:
clang-tools-extra/trunk/clangd/ProtocolHandlers.cpp
Modified: clang-tools-extra/trunk/clangd/ProtocolHandlers.cpp
URL:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL295186: [analyzer] Proper caching in CallDescription
objects. (authored by xazax).
Changed prior to commit:
https://reviews.llvm.org/D29884?vs=88166=88537#toc
Repository:
rL LLVM
Author: xazax
Date: Wed Feb 15 09:35:56 2017
New Revision: 295186
URL: http://llvm.org/viewvc/llvm-project?rev=295186=rev
Log:
[analyzer] Proper caching in CallDescription objects.
During the review of D29567 it turned out the caching in CallDescription is not
implemented properly. In case an
Author: d0k
Date: Wed Feb 15 09:19:13 2017
New Revision: 295184
URL: http://llvm.org/viewvc/llvm-project?rev=295184=rev
Log:
[clangd] Add missing include.
Modified:
clang-tools-extra/trunk/clangd/DocumentStore.h
Modified: clang-tools-extra/trunk/clangd/DocumentStore.h
URL:
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
I sometimes wish ASTContext methods just didn't crash :)
Oh well, let's just see if more problems show up.
Comment at: lib/StaticAnalyzer/Checkers/CastToStructChecker.cpp:93
1 - 100 of 144 matches
Mail list logo