[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

aaron.ballman added inline comments. Comment at: include/clang/ASTMatchers/ASTMatchers.h:1260 +/// Matches call expressions which were resolved using ADL. +/// Don't forget to regenerate the documentation and update Registry.cpp to add clang-query support.

[PATCH] D42682: [clang-tidy] Add io-functions-misused checker

hgabii marked 3 inline comments as done. hgabii added inline comments. Comment at: clang-tidy/bugprone/IoFunctionsCheck.cpp:32 +has(cxxMemberCallExpr( +on(hasType(namedDecl(hasAnyName("istream", +

[PATCH] D51211: [Sema] Emit -Wformat properly for bitfield promotions.

ebevhan added a comment. Thanks! I don't have commit access, so I would appreciate it if you could commit the change. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D51211/new/ https://reviews.llvm.org/D51211 ___ cfe-commits mailing list

[PATCH] D55066: [ASan] Minor documentation fix: clarify static linking limitation.

This revision was automatically updated to reflect the committed changes. Closed by commit rC348863: [ASan] Minor documentation fix: clarify static linking limitation. (authored by Dor1s, committed by ). Changed prior to commit: https://reviews.llvm.org/D55066?vs=176712=177724#toc Repository:

[PATCH] D55561: Stop stripping comments from AST matcher example code

aaron.ballman created this revision. aaron.ballman added reviewers: JonasToth, klimek. The AST matcher documentation dumping script was being a bit over-zealous about stripping comment markers, which ended up causing comments in example code to stop being comments. This patch fixes that by only

[PATCH] D55544: Warning: objc-encodings-larger-than=

dmaclach updated this revision to Diff 177732. dmaclach added a comment. Added some spacing around early exit as requested by theraven. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55544/new/ https://reviews.llvm.org/D55544 Files: docs/ReleaseNotes.rst

[PATCH] D53023: Prototype OpenCL BIFs using Tablegen

Nicola added a comment. This looks quite useful, thanks for taking time to do it! I can help verifying some of the tablegen'd builtins with internal tests too, if you decide to pursue this further. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D53023/new/

Re: r348858 - Revert "[PowerPC] Make no-PIC default to match GCC - CLANG"

Hi Aaron, Sorry about giving so little info. The commit was reverted because it broke two test cases on an internal buildbot. The reason this was reverted so late was because this failure was buried underneath another set of failures on that same buildbot which initially hid the problem. I'm

[PATCH] D42682: [clang-tidy] Add io-functions-misused checker

hgabii updated this revision to Diff 177713. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D42682/new/ https://reviews.llvm.org/D42682 Files: clang-tidy/bugprone/BugproneTidyModule.cpp clang-tidy/bugprone/CMakeLists.txt

r348863 - [ASan] Minor documentation fix: clarify static linking limitation.

Author: dor1s Date: Tue Dec 11 08:47:12 2018 New Revision: 348863 URL: http://llvm.org/viewvc/llvm-project?rev=348863=rev Log: [ASan] Minor documentation fix: clarify static linking limitation. Summary: ASan does not support statically linked binaries, but ASan runtime itself can be statically

r348864 - Pass PartialOverloading argument to the correct corresponding parameter

Author: ericwf Date: Tue Dec 11 08:53:25 2018 New Revision: 348864 URL: http://llvm.org/viewvc/llvm-project?rev=348864=rev Log: Pass PartialOverloading argument to the correct corresponding parameter Modified: cfe/trunk/lib/Sema/SemaOverload.cpp

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

aaron.ballman added inline comments. Comment at: docs/LibASTMatchersReference.html:2579-2581 +y(x); Matches +NS::y(x); Doesn't match +y(42); Doesn't match. EricWF wrote: > aaron.ballman wrote: > > This is not your bug to fix, but it seems the

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

riccibruno added inline comments. Comment at: lib/AST/Expr.cpp:1267 : Expr(SC, Empty), NumArgs(NumArgs) { + CallExprBits.UsesADL = false; CallExprBits.NumPreArgs = NumPreArgs; I believe that msan can cope with bit level operations just fine. What I am

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

rsmith added inline comments. Comment at: include/clang/Sema/Sema.h:2758 bool AllowExplicit = false, +bool IsADLCandidate = false, ConversionSequenceList EarlyConversions = None);

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

aaron.ballman marked an inline comment as done. aaron.ballman added a comment. A few more minor nits. Comment at: docs/LibASTMatchersReference.html:2579-2581 +y(x); Matches +NS::y(x); Doesn't match +y(42); Doesn't match. This is not your bug to

r348865 - Reuse code from CGDebugInfo::getOrCreateFile() when creating the file

Author: adrian Date: Tue Dec 11 08:58:43 2018 New Revision: 348865 URL: http://llvm.org/viewvc/llvm-project?rev=348865=rev Log: Reuse code from CGDebugInfo::getOrCreateFile() when creating the file for the DICompileUnit. This addresses post-commit feedback for D55085. Without this patch, a main

r348866 - Remove CGDebugInfo::getOrCreateFile() and use TheCU->getFile() directly.

Author: adrian Date: Tue Dec 11 08:58:46 2018 New Revision: 348866 URL: http://llvm.org/viewvc/llvm-project?rev=348866=rev Log: Remove CGDebugInfo::getOrCreateFile() and use TheCU->getFile() directly. Modified: cfe/trunk/lib/CodeGen/CGDebugInfo.cpp cfe/trunk/lib/CodeGen/CGDebugInfo.h

[PATCH] D55562: Atomics: support min/max orthogonally

t.p.northover created this revision. t.p.northover added reviewers: delena, yaxunl. Herald added subscribers: jfb, mcrosier. We seem to have been gradually growing support for atomic min/max operations (exposing longstanding IR atomicrmw instructions). But until now there have been gaps in the

Re: r348858 - Revert "[PowerPC] Make no-PIC default to match GCC - CLANG"

On Tue, Dec 11, 2018 at 12:07 PM Stefan Pintilie wrote: > > Hi Aaron, > > Sorry about giving so little info. No worries! > The commit was reverted because it broke two test cases on an internal > buildbot. The reason this was reverted so late was because this failure was > buried underneath

[PATCH] D55413: [ExprConstant] Handle compound assignment when LHS has integral type and RHS has floating point type

cpplearner updated this revision to Diff 177708. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55413/new/ https://reviews.llvm.org/D55413 Files: clang/lib/AST/ExprConstant.cpp clang/test/SemaCXX/constant-expression-cxx1y.cpp Index:

r348858 - Revert "[PowerPC] Make no-PIC default to match GCC - CLANG"

Author: stefanp Date: Tue Dec 11 07:47:57 2018 New Revision: 348858 URL: http://llvm.org/viewvc/llvm-project?rev=348858=rev Log: Revert "[PowerPC] Make no-PIC default to match GCC - CLANG" This reverts commit rL348299. Modified: cfe/trunk/lib/Driver/ToolChains/Gnu.cpp

[PATCH] D54592: [analyzer][CStringChecker] evaluate explicit_bzero

devnexen added a comment. ping just want to put it behind :-) thanks. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54592/new/ https://reviews.llvm.org/D54592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

riccibruno added inline comments. Comment at: lib/AST/Expr.cpp:1267 : Expr(SC, Empty), NumArgs(NumArgs) { + CallExprBits.UsesADL = false; CallExprBits.NumPreArgs = NumPreArgs; riccibruno wrote: > I believe that msan can cope with bit level operations

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

EricWF updated this revision to Diff 177705. EricWF marked an inline comment as done. EricWF added a comment. Update with fixes for review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55534/new/ https://reviews.llvm.org/D55534 Files: include/clang/AST/Expr.h

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

EricWF marked 12 inline comments as done. EricWF added a comment. Address more review comments. Update incoming. Comment at: include/clang/AST/Expr.h:2425 CallExpr(const ASTContext , StmtClass SC, unsigned NumPreArgs, - unsigned NumArgs, EmptyShell Empty); +

[libunwind] r348852 - [cmake] Rename append_if to avoid collision with LLVM

Author: mgorny Date: Tue Dec 11 07:30:04 2018 New Revision: 348852 URL: http://llvm.org/viewvc/llvm-project?rev=348852=rev Log: [cmake] Rename append_if to avoid collision with LLVM Rename the 'append_if' macro used in libunwind to 'unwind_append_if'. Otherwise, when used in a combined

[PATCH] D55476: [libunwind] [cmake] Rename append_if to avoid collision with LLVM

This revision was automatically updated to reflect the committed changes. Closed by commit rL348852: [cmake] Rename append_if to avoid collision with LLVM (authored by mgorny, committed by ). Herald added subscribers: llvm-commits, christof. Changed prior to commit:

[PATCH] D55413: [ExprConstant] Handle compound assignment when LHS has integral type and RHS has floating point type

riccibruno added inline comments. Comment at: clang/lib/AST/ExprConstant.cpp:3427 -if (!SubobjType->isIntegerType() || !RHS.isInt()) { +if (!SubobjType->isIntegerType() || !RHS.isInt() && !RHS.isFloat()) { // We don't support compound assignment on

Re: r348858 - Revert "[PowerPC] Make no-PIC default to match GCC - CLANG"

On Tue, Dec 11, 2018 at 10:50 AM Stefan Pintilie via cfe-commits wrote: > > Author: stefanp > Date: Tue Dec 11 07:47:57 2018 > New Revision: 348858 > > URL: http://llvm.org/viewvc/llvm-project?rev=348858=rev > Log: > Revert "[PowerPC] Make no-PIC default to match GCC - CLANG" > > This reverts

[PATCH] D55413: [ExprConstant] Handle compound assignment when LHS has integral type and RHS has floating point type

cpplearner updated this revision to Diff 177719. cpplearner added a comment. Added parentheses. Restored the original tests and add more tests to the end of this function. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55413/new/ https://reviews.llvm.org/D55413

[PATCH] D55413: [ExprConstant] Handle compound assignment when LHS has integral type and RHS has floating point type

cpplearner marked an inline comment as done. cpplearner added inline comments. Comment at: clang/test/SemaCXX/constant-expression-cxx1y.cpp:343 if (a != 7) return false; -a *= 3; if (a != 21) return false; riccibruno wrote: > Why remove `a *= 3`

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

EricWF marked 3 inline comments as done. EricWF added a comment. Address more inline comments. Comment at: docs/LibASTMatchersReference.html:2579-2581 +y(x); Matches +NS::y(x); Doesn't match +y(42); Doesn't match. aaron.ballman wrote: > This is not

[PATCH] D55544: Warning: objc-encodings-larger-than=

dmaclach added a comment. In D55544#1326956 , @theraven wrote: > It would probably be a good idea to have a similar check on properties, as > property encoding strings contain the type encoding (plus extra stuff). Properties are already picked up based

[PATCH] D55226: [Fix][StaticAnalyzer] Bug 39792 - False positive on strcpy targeting struct member

Pierre-vh added a comment. Hello! I'm pinging since it's been a week. If someone can commit this patch on my behalf, that would be great. Thank you :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55226/new/ https://reviews.llvm.org/D55226

r348848 - [analyzer] Fix a minor typo.

Author: xazax Date: Tue Dec 11 06:40:48 2018 New Revision: 348848 URL: http://llvm.org/viewvc/llvm-project?rev=348848=rev Log: [analyzer] Fix a minor typo. Modified: cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/MemRegion.h Modified:

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

riccibruno added inline comments. Comment at: lib/AST/Expr.cpp:1267 : Expr(SC, Empty), NumArgs(NumArgs) { + CallExprBits.UsesADL = false; CallExprBits.NumPreArgs = NumPreArgs; It do not really matter but there is not point initializing this bit here.

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

EricWF updated this revision to Diff 177720. EricWF marked 2 inline comments as done. EricWF added a comment. Register matcher and regenerate docs. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55534/new/ https://reviews.llvm.org/D55534 Files: docs/LibASTMatchersReference.html

Re: r348276 - [AST][NFC] Make ArrayTypeTraitExpr non polymorphic

Hi Mikael, I can indeed reproduce this with gcc 5.5.0 when doing a Release build with assertions. I cannot reproduce this with gcc 6.5.0 (also with a Release build with assertions), nor can I reproduce this with clang 7 (also with a Release build with assertions). I tried to instrument

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

Quuxplusone added inline comments. Comment at: lib/Sema/SemaTemplate.cpp:3089 + else +OS << "("; + for (CXXTemporaryObjectExpr::arg_iterator Arg = Node->arg_begin(), It might be more maintainer-proof to write this as std::pair Braces;

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

EricWF updated this revision to Diff 177716. EricWF added a comment. More tests. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55534/new/ https://reviews.llvm.org/D55534 Files: include/clang/AST/Expr.h include/clang/AST/ExprCXX.h include/clang/AST/Stmt.h

r348860 - Adding tests for -ast-dump; NFC.

Author: aaronballman Date: Tue Dec 11 08:34:59 2018 New Revision: 348860 URL: http://llvm.org/viewvc/llvm-project?rev=348860=rev Log: Adding tests for -ast-dump; NFC. This adds tests for expressions in C++. Added: cfe/trunk/test/AST/ast-dump-expr.cpp Added:

r348899 - Replace Const-Member checking with non-recursive version.

Author: erichkeane Date: Tue Dec 11 13:54:52 2018 New Revision: 348899 URL: http://llvm.org/viewvc/llvm-project?rev=348899=rev Log: Replace Const-Member checking with non-recursive version. As reported in PR39946, these two implementations cause stack overflows to occur when a type recursively

[PATCH] D55229: [COFF] Statically link certain runtime library functions

rnk added a comment. Sorry, I was out sick for a week. We should ask @smeenai about this change, since he has been doing more work in this area recently. Comment at: CodeGen/CodeGenModule.cpp:2957-2958 !getCodeGenOpts().LTOVisibilityPublicStd && -

[PATCH] D55510: [ExprConstant] Improve memchr/memcmp for type mismatch and multibyte element types

hubert.reinterpretcast updated this revision to Diff 177796. hubert.reinterpretcast added a comment. Recast representation-sensitive tests as SemaCXX tests using array bounds Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55510/new/

[PATCH] D55229: [COFF] Statically link certain runtime library functions

rnk added a comment. Thanks. I think what we really want to do here is reconsider our default for applying dllimport. Leaving things unannotated is a good safe default for every environment. In the absence of any flags, clang should assume runtime functions are statically linked. The linker

[PATCH] D53238: [Driver] Add -static-{rtlib, stdlib} and make -static-{libgcc, libstdc++} their aliases

rnk added a comment. (sub) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D53238/new/ https://reviews.llvm.org/D53238 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D55525: [Driver] Add support for -fembed-bitcode for assembly file

steven_wu added a comment. In D55525#1327742 , @compnerd wrote: > This really feels odd. Why not expect that the developer will add the > content themselves? I'm not sure I understand the motivation for this change. The main motivation for

[PATCH] D55527: Normalize GlobalDecls when used with CPUDispatch

rsmith marked an inline comment as done. rsmith added a comment. Thanks, this makes a lot of sense to me. Comment at: lib/CodeGen/CodeGenModule.cpp:907 +if (CGM.getTarget().supportsIFunc()) +Out << ".ifunc"; + } else if (CGM.getTarget().supportsIFunc())

[PATCH] D55546: [clang] Add AST matcher for block expressions 

stephanemoore added a comment. Please let me know if I missed anything as part of this proposed change. I tried my best to determine how to make this change appropriately based on previous submissions and documentation but I recognize that I could have missed things  Repository: rC Clang

Buildbot numbers for the week of 11/25/2018 - 12/01/2018

Hello everyone, Below are some buildbot numbers for the week of 11/25/2018 - 12/01/2018. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from greed

Buildbot numbers for the week of 12/02/2018 - 12/08/2018

Hello everyone, Below are some buildbot numbers for the last week of 12/02/2018 - 12/08/2018. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from

[PATCH] D55527: Normalize GlobalDecls when used with CPUDispatch

erichkeane marked 4 inline comments as done. erichkeane added inline comments. Comment at: lib/CodeGen/CodeGenModule.cpp:910 Out << ".resolver"; } rsmith wrote: > Hmm, it looks like we don't have a unique name for a `GlobalDecl` naming the > `CPUOrdinal

[PATCH] D55566: [analyzer] LiveVariables: Fix a zombie expression problem, add testing infrastructure.

Szelethus added a comment. Makes sense! I like the summary a lot, and the fact that you added a new debug checker. I feel like I'm not yet qualified to give meaningful feedback though, but if you are not in a hurry, I'll happily play around with this patch next week, and both learn a bit and

[PATCH] D55229: [COFF] Statically link certain runtime library functions

compnerd added a subscriber: theraven. compnerd added inline comments. Comment at: CodeGen/CodeGenModule.cpp:2957-2958 !getCodeGenOpts().LTOVisibilityPublicStd && - !getTriple().isWindowsGNUEnvironment()) { + !getTriple().isWindowsGNUEnvironment() &&

[PATCH] D55510: [ExprConstant] Improve memchr/memcmp for type mismatch and multibyte element types

rsmith accepted this revision. rsmith added inline comments. This revision is now accepted and ready to land. Comment at: lib/AST/ExprConstant.cpp:6147-6148 + return ZeroInitialization(E); +if (!Result.checkNullPointerForFoldAccess(Info, E, AK_Read)) + return

[PATCH] D54592: [analyzer][CStringChecker] evaluate explicit_bzero

This revision was automatically updated to reflect the committed changes. Closed by commit rL348884: [analyzer][CStringChecker] evaluate explicit_bzero (authored by devnexen, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D55527: Normalize GlobalDecls when used with CPUDispatch

erichkeane updated this revision to Diff 177767. erichkeane marked 8 inline comments as done. erichkeane added a comment. Thanks @aaron.ballman . This should fix all of your complaints. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55527/new/ https://reviews.llvm.org/D55527 Files:

[PATCH] D43871: [modules] No longer include stdlib.h from mm_malloc.h.

joerg added a comment. This header is used on systems without glibc. So please don't argue about behavior based only on that. Granted, most other libc implementation are less annoying when it comes to `free` and `malloc`, but still. CHANGES SINCE LAST ACTION

[PATCH] D55447: [Sema] Fix Modified Type in address_space AttributedType

leonardchan marked an inline comment as done. leonardchan added inline comments. Comment at: clang/tools/libclang/CXType.cpp:132 + if (!(TU->ParsingOptions & CXTranslationUnit_IncludeAttributedTypes) && + ATT->getAttrKind() != attr::AddressSpace) { return

[PATCH] D55270: [Sema] Further improvements to to static_assert diagnostics.

Quuxplusone added a comment. @courbet: On the cpplang Slack, Peter Feichtinger mentioned that defaulted template arguments should perhaps be treated differently. Is there any way to suppress the `, std::allocator` part of this diagnostic? https://godbolt.org/z/TM0UHc Before your patches:

[PATCH] D55525: [Driver] Add support for -fembed-bitcode for assembly file

compnerd added a comment. This really feels odd. Why not expect that the developer will add the content themselves? I'm not sure I understand the motivation for this change. I think that this should be easy to write a test case for as well. Repository: rC Clang CHANGES SINCE LAST ACTION

[PATCH] D55586: Basic: make `int_least64_t` and `int_fast64_t` match on Darwin

compnerd created this revision. compnerd added reviewers: ahatanak, rjmccall. The Darwin targets use `int64_t` and `uint64_t` to define the `int_least64_t` and `int_fast64_t` types. The underlying type is actually a `long long`. Match the types to allow the printf specifiers to work properly

[PATCH] D55482: [clang-tidy] Improve google-objc-function-naming diagnostics 

stephanemoore updated this revision to Diff 177795. stephanemoore marked 5 inline comments as done. stephanemoore added a comment. Changes: • Drop const on local bool variable. • Adopt the term "function in global namespace" in diagnostic messages. CHANGES SINCE LAST ACTION

[PATCH] D55482: [clang-tidy] Improve google-objc-function-naming diagnostics 

stephanemoore added inline comments. Comment at: clang-tidy/google/FunctionNamingCheck.cpp:115 diag(MatchedDecl->getLocation(), - "function name %0 not using function naming conventions described by " - "Google Objective-C style guide") - << MatchedDecl <<

[PATCH] D48753: [libcxx] Use custom allocator's `construct` in C++03 when available.

vsapsai updated this revision to Diff 177803. vsapsai added a comment. - Update `__has_construct`, `__has_destroy` according to review comments. - Tighten tests: custom allocators aren't mandated by C++03 but libc++ has the support. CHANGES SINCE LAST ACTION

[PATCH] D48753: [libcxx] Use custom allocator's `construct` in C++03 when available.

vsapsai marked 3 inline comments as done. vsapsai added a comment. Regarding the tests. I've moved most of new tests for custom allocators to test/libcxx/*. And in destroy.pass.cpp I'm just checking `_LIBCPP_VERSION` to avoid copying the test to a different file. Comment at:

Buildbot numbers for the week of 11/18/2018 - 11/24/2018

Hello everyone, Below are some buildbot numbers for the week of 11/18/2018 - 11/24/2018. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from greed

[PATCH] D54355: Use is.constant intrinsic for __builtin_constant_p

craig.topper added a comment. Here's the test case that we have https://reviews.llvm.org/P8123 gcc seems to accept it at O0 Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54355/new/ https://reviews.llvm.org/D54355

[PATCH] D43871: [modules] No longer include stdlib.h from mm_malloc.h.

teemperor added a comment. @joerg Yeah, we saw the commit explaining why the original fwd declaration patch was reverted. However, from what I can see, we only have three ways to fix the cyclic dependency between glibc and Clang's internal module: 1. We say that we don't support including

[PATCH] D55544: Warning: objc-encodings-larger-than=

dmaclach updated this revision to Diff 177744. dmaclach marked an inline comment as done. dmaclach added a comment. Full Diffs as requested. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55544/new/ https://reviews.llvm.org/D55544 Files: docs/ReleaseNotes.rst

[PATCH] D55245: [clang-tidy] Add the abseil-duration-subtraction check

hwright added inline comments. Comment at: test/clang-tidy/abseil-duration-subtraction.cpp:12 + // CHECK-FIXES: absl::ToDoubleSeconds(d - absl::Seconds(1)) + x = absl::ToDoubleSeconds(d) - absl::ToDoubleSeconds(d1); + // CHECK-MESSAGES: [[@LINE-1]]:7: warning: perform

[PATCH] D55245: [clang-tidy] Add the abseil-duration-subtraction check

hwright updated this revision to Diff 177749. hwright marked 6 inline comments as done. hwright added a comment. Rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55245/new/ https://reviews.llvm.org/D55245 Files: clang-tidy/abseil/AbseilTidyModule.cpp

[PATCH] D51211: [Sema] Emit -Wformat properly for bitfield promotions.

aaron.ballman reopened this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. I had to revert the commit as the changes caused some test failures.

[PATCH] D55410: [clang-tidy] check for flagging using declarations in headers

aaron.ballman added inline comments. Comment at: docs/clang-tidy/checks/abseil-alias-free-headers.rst:13 +The style guide http://google.github.io/styleguide/cppguide.html#Aliases discusses this +issue in more detail Eugene.Zelenko wrote: > Naysh wrote: > >

[PATCH] D54921: [analyzer] Remove memoization from RunLoopAutoreleaseLeakChecker

NoQ added a comment. I think that it is modified, at least, when `Memoization[C]` is evaluated. The object needs to be created and put into the map in order to obtain a reference to it. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54921/new/

[PATCH] D55483: Introduce the callback attribute and emit !callback metadata

aaron.ballman added inline comments. Comment at: include/clang/Basic/Attr.td:1204 + VariadicUnsignedArgument<"PayloadIndices">]; + let Subjects = SubjectList<[Function]>; + let Documentation = [Undocumented]; jdoerfert wrote: > aaron.ballman

[PATCH] D55562: Atomics: support min/max orthogonally

jfb added a comment. What does it do with floating-point inputs? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55562/new/ https://reviews.llvm.org/D55562 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D55561: Stop stripping comments from AST matcher example code

JonasToth accepted this revision. JonasToth added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55561/new/ https://reviews.llvm.org/D55561 ___ cfe-commits mailing list

[PATCH] D55488: Add utility for dumping a label with child nodes

aaron.ballman added inline comments. Comment at: lib/AST/ASTDumper.cpp:89 void dumpDecl(const Decl *D); -void dumpStmt(const Stmt *S); +void dumpStmt(const Stmt *S, const std::string = {}); steveire wrote: > aaron.ballman wrote: > > Label > > >

[PATCH] D51211: [Sema] Emit -Wformat properly for bitfield promotions.

aaron.ballman closed this revision. aaron.ballman added a comment. I've commit in r348889, thank you for the patch! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D51211/new/ https://reviews.llvm.org/D51211 ___ cfe-commits mailing list

r348889 - Emit -Wformat properly for bit-field promotions.

Author: aaronballman Date: Tue Dec 11 11:18:01 2018 New Revision: 348889 URL: http://llvm.org/viewvc/llvm-project?rev=348889=rev Log: Emit -Wformat properly for bit-field promotions. Only explicitly look through integer and floating-point promotion where the result type is actually a promotion,

r348891 - Stop stripping comments from AST matcher example code.

Author: aaronballman Date: Tue Dec 11 11:30:49 2018 New Revision: 348891 URL: http://llvm.org/viewvc/llvm-project?rev=348891=rev Log: Stop stripping comments from AST matcher example code. The AST matcher documentation dumping script was being a bit over-zealous about stripping comment markers,

[PATCH] D55561: Stop stripping comments from AST matcher example code

aaron.ballman closed this revision. aaron.ballman added a comment. Committed in r348891. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55561/new/ https://reviews.llvm.org/D55561 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

EricWF marked 13 inline comments as done. EricWF added inline comments. Comment at: include/clang/Sema/Sema.h:2758 bool AllowExplicit = false, +bool IsADLCandidate = false,

[PATCH] D55447: [Sema] Fix Modified Type in address_space AttributedType

aaron.ballman added inline comments. Comment at: clang/tools/libclang/CXType.cpp:132 + if (!(TU->ParsingOptions & CXTranslationUnit_IncludeAttributedTypes) && + ATT->getAttrKind() != attr::AddressSpace) { return MakeCXType(ATT->getModifiedType(), TU);

[PATCH] D54921: [analyzer] Remove memoization from RunLoopAutoreleaseLeakChecker

george.karpenkov added a comment. @NoQ thanks, interesting! But I thought the underlying map is not actually modified while the reference is alive? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54921/new/ https://reviews.llvm.org/D54921

[PATCH] D54921: [analyzer] Remove memoization from RunLoopAutoreleaseLeakChecker

NoQ added a comment. That is, it is modified within the recursive call. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54921/new/ https://reviews.llvm.org/D54921 ___ cfe-commits mailing list

[PATCH] D55527: Normalize GlobalDecls when used with CPUDispatch

aaron.ballman added inline comments. Comment at: include/clang/Basic/Attr.td:896 let AdditionalMembers = [{ -IdentifierInfo *getCPUName(unsigned Index) const { - return *(cpus_begin() + Index); +// gets the ordinal of the requested CPU name, or 0 if it isn't in

[PATCH] D55483: Introduce the callback attribute and emit !callback metadata

jdoerfert added a comment. Thanks for the feedback. Once we decided on the style of the annotation, I will implement that and change the tests/documentation accordingly. Comment at: include/clang/Basic/Attr.td:1204 + VariadicUnsignedArgument<"PayloadIndices">];

Re: r348889 - Emit -Wformat properly for bit-field promotions.

On Tue, Dec 11, 2018 at 2:21 PM Aaron Ballman via cfe-commits wrote: > > Author: aaronballman > Date: Tue Dec 11 11:18:01 2018 > New Revision: 348889 > > URL: http://llvm.org/viewvc/llvm-project?rev=348889=rev > Log: > Emit -Wformat properly for bit-field promotions. > > Only explicitly look

r348892 - Revert r348889; it fails some tests.

Author: aaronballman Date: Tue Dec 11 11:42:04 2018 New Revision: 348892 URL: http://llvm.org/viewvc/llvm-project?rev=348892=rev Log: Revert r348889; it fails some tests. http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/builds/40784 Removed:

[PATCH] D55483: Introduce the callback attribute and emit !callback metadata

jdoerfert updated this revision to Diff 11. jdoerfert marked 11 inline comments as done. jdoerfert added a comment. Fix and improve documentation Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55483/new/ https://reviews.llvm.org/D55483 Files:

[PATCH] D54923: [Modules] Remove non-determinism while serializing DECL_CONTEXT_LEXICAL and DECL_RECORD

bruno added a comment. Ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54923/new/ https://reviews.llvm.org/D54923 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D55482: [clang-tidy] Improve google-objc-function-naming diagnostics 

aaron.ballman added inline comments. Comment at: clang-tidy/google/FunctionNamingCheck.cpp:113 + const bool IsGlobal = MatchedDecl->getStorageClass() != SC_Static; diag(MatchedDecl->getLocation(), Drop the top-level `const` qualifier, please.

r348884 - [analyzer][CStringChecker] evaluate explicit_bzero

Author: devnexen Date: Tue Dec 11 10:57:07 2018 New Revision: 348884 URL: http://llvm.org/viewvc/llvm-project?rev=348884=rev Log: [analyzer][CStringChecker] evaluate explicit_bzero - explicit_bzero has limited scope/usage only for security/crypto purposes but is non-optimisable version of

[PATCH] D55534: [AST] Store "UsesADL" information in CallExpr.

EricWF updated this revision to Diff 177752. EricWF marked 3 inline comments as done. EricWF added a comment. Address review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55534/new/ https://reviews.llvm.org/D55534 Files: docs/LibASTMatchersReference.html

[PATCH] D54355: Use is.constant intrinsic for __builtin_constant_p

jyknight added a comment. In D54355#1327237 , @craig.topper wrote: > Here's the test case that we have https://reviews.llvm.org/P8123 gcc seems > to accept it at O0 Smaller test case: extern unsigned long long __sdt_unsp; void foo() {

[PATCH] D55544: Warning: objc-encodings-larger-than=

arphaman added a comment. Thanks for working on this! Could you please post a patch with full context (git diff -U)? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55544/new/ https://reviews.llvm.org/D55544 ___

[PATCH] D55544: Warning: objc-encodings-larger-than=

dmaclach updated this revision to Diff 177746. dmaclach added a comment. Updated to fix Stephane's good catch of Objective C vs Objective-C Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55544/new/ https://reviews.llvm.org/D55544 Files: docs/ReleaseNotes.rst

[PATCH] D55566: [analyzer] LiveVariables: Fix a zombie expression problem, add testing infrastructure.

NoQ created this revision. NoQ added reviewers: dcoughlin, xazax.hun, a_sidorin, george.karpenkov, szepet, rnkovacs, Szelethus. Herald added subscribers: cfe-commits, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, baloghadamsoftware. For the first time in years, there seems to be a bug in our

  1   2   >