r360995 - Refactor constant evaluation of typeid(T) to track a symbolic type_info

Author: rsmith Date: Fri May 17 00:06:46 2019 New Revision: 360995 URL: http://llvm.org/viewvc/llvm-project?rev=360995&view=rev Log: Refactor constant evaluation of typeid(T) to track a symbolic type_info object rather than tracking the originating expression. This is groundwork for supporting po

r360996 - [ClangFormat] Editor integrations inherit default style from clang-format binary

Author: sammccall Date: Fri May 17 00:22:55 2019 New Revision: 360996 URL: http://llvm.org/viewvc/llvm-project?rev=360996&view=rev Log: [ClangFormat] Editor integrations inherit default style from clang-format binary Summary: This allows downstream customizations to the default style to work with

[PATCH] D49719: [ClangFormat] Editor integrations inherit default style from clang-format binary

This revision was automatically updated to reflect the committed changes. Closed by commit rC360996: [ClangFormat] Editor integrations inherit default style from clang-format binary (authored by sammccall, committed by ). Herald added a project: clang. Changed prior to commit: https://reviews.l

r360997 - Fix alignment check to check the alignment of the intended type.

Author: rsmith Date: Fri May 17 00:28:41 2019 New Revision: 360997 URL: http://llvm.org/viewvc/llvm-project?rev=360997&view=rev Log: Fix alignment check to check the alignment of the intended type. Modified: cfe/trunk/lib/AST/APValue.cpp Modified: cfe/trunk/lib/AST/APValue.cpp URL: http://l

r360998 - Fix crash if, during evaluation of __builtin_object_size, we try to load

Author: rsmith Date: Fri May 17 01:01:34 2019 New Revision: 360998 URL: http://llvm.org/viewvc/llvm-project?rev=360998&view=rev Log: Fix crash if, during evaluation of __builtin_object_size, we try to load through an invalid base. Modified: cfe/trunk/lib/AST/ExprConstant.cpp cfe/trunk/tes

r360999 - Suppress false-positive GCC -Wreturn-type warning.

Author: rsmith Date: Fri May 17 01:06:49 2019 New Revision: 360999 URL: http://llvm.org/viewvc/llvm-project?rev=360999&view=rev Log: Suppress false-positive GCC -Wreturn-type warning. Modified: cfe/trunk/include/clang/AST/Expr.h Modified: cfe/trunk/include/clang/AST/Expr.h URL: http://llvm.

[PATCH] D61522: Added an assertion to constant evaluation enty points that prohibits dependent expressions

gribozavr added a comment. @ABataev @rsmith Ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61522/new/ https://reviews.llvm.org/D61522 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:

[PATCH] D61522: Added an assertion to constant evaluation enty points that prohibits dependent expressions

rsmith accepted this revision. rsmith added inline comments. Comment at: clang/lib/Sema/SemaOverload.cpp:6369 // very difficult. Ideally, we should handle them more gracefully. -if (!EIA->getCond()->EvaluateWithSubstitution( +if (EIA->getCond()->isValueDependent() ||

[PATCH] D62056: Use ASTDumper to dump the AST from clang-query

steveire created this revision. steveire added a reviewer: aaron.ballman. Herald added a project: clang. Herald added a subscriber: cfe-commits. This way, the output is not limited by the various API differences between the dump() member functions. For example, all dumps are now in color, while t

[PATCH] D59885: [Lex] Allow to consume tokens while preprocessing

ilya-biryukov updated this revision to Diff 11. ilya-biryukov added a comment. - Fix compilation of a clang-tidy check, add a FIXME to stop reporting those tokens Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59885/new/ https://reviews.llvm.or

[PATCH] D61834: Add a Visit overload for DynTypedNode to ASTNodeTraverser

steveire updated this revision to Diff 12. steveire added a comment. Update Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61834/new/ https://reviews.llvm.org/D61834 Files: include/clang/AST/ASTNodeTraverser.h unittests/AST/ASTTraverserTest.cpp unittest

[PATCH] D61816: [CFG] [analyzer] pr41300: Add a branch to skip virtual base initializers when they are handled by the superclass.

Szelethus added inline comments. Comment at: clang/lib/Analysis/CFG.cpp:1434-1438 + // For C++ constructor add initializers to CFG. Virtual base classes may have + // already been initialized by the superclass. Make a branch so that it was + // possible to jump straight to non

[PATCH] D61774: [LibTooling] Add RangeSelector library for defining source ranges based on bound AST nodes.

ilya-biryukov marked an inline comment as done. ilya-biryukov added inline comments. Comment at: clang/include/clang/Tooling/Refactoring/RangeSelector.h:29 + +namespace range_selector { +inline RangeSelector charRange(CharSourceRange R) { ymandel wrote: > ilya-bi

[PATCH] D61522: Added an assertion to constant evaluation enty points that prohibits dependent expressions

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61522/new/ https://reviews.llvm.org/D61522 ___

[PATCH] D59555: [analyzer] Add yaml parser to GenericTaintChecker

boga95 updated this revision to Diff 10. boga95 edited the summary of this revision. boga95 added a comment. I changed the parsing, therefore the return value index is represented by -1. I added a test configuration file and parse it when testing to prove the configuration doesn't break the c

[PATCH] D61335: [LibTooling] Add support to Transformer for composing rules as an ordered choice.

ilya-biryukov added inline comments. Comment at: clang/include/clang/Tooling/Refactoring/Transformer.h:278 +/// Builds the matcher needed for registration. +ast_matchers::internal::DynTypedMatcher buildMatcher(const RewriteRule &Rule); + ymandel wrote: > ilya-bir

[PATCH] D41005: Reuse preamble even if an unsaved file does not exist

ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. Thanks! LGTM Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D41005/new/ https://reviews.llvm.org/D41005

[PATCH] D60763: Prototype OpenCL BIFs using Tablegen

Pierre updated this revision to Diff 18. Pierre marked 14 inline comments as done. Pierre added a comment. Corrections from the comments on the previous version. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60763/new/ https://reviews.llvm.org/D60763 Files: clang/include/clang/Ba

[PATCH] D60763: Prototype OpenCL BIFs using Tablegen

Pierre added inline comments. Comment at: clang/include/clang/Basic/OpenCLBuiltins.td:17 +// +// Builtin functions are instances of the Builtin class. +// Prototypes are defined as a list of Type classes (or its subclasses). Anastasia wrote: > Can you explain thi

r361004 - [NFC] Fix line endings in OpenCL tests

Author: svenvh Date: Fri May 17 02:25:38 2019 New Revision: 361004 URL: http://llvm.org/viewvc/llvm-project?rev=361004&view=rev Log: [NFC] Fix line endings in OpenCL tests Modified: cfe/trunk/test/CodeGenOpenCL/no-signed-zeros.cl cfe/trunk/test/SemaOpenCL/array-init.cl Modified: cfe/trun

r361006 - [analyzer] Insert checker options into AnalyzerOption::ConfigTable

Author: szelethus Date: Fri May 17 02:29:44 2019 New Revision: 361006 URL: http://llvm.org/viewvc/llvm-project?rev=361006&view=rev Log: [analyzer] Insert checker options into AnalyzerOption::ConfigTable The more entries we have in AnalyzerOptions::ConfigTable, the more helpful debug.ConfigDumper

[PATCH] D57922: [analyzer] Insert checker options into AnalyzerOption::ConfigTable

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Szelethus marked an inline comment as done. Closed by commit rL361006: [analyzer] Insert checker options into AnalyzerOption::ConfigTable (au

r361007 - [Lex] Allow to consume tokens while preprocessing

Author: ibiryukov Date: Fri May 17 02:32:05 2019 New Revision: 361007 URL: http://llvm.org/viewvc/llvm-project?rev=361007&view=rev Log: [Lex] Allow to consume tokens while preprocessing Summary: By adding a hook to consume all tokens produced by the preprocessor. The intention of this change is t

[clang-tools-extra] r361007 - [Lex] Allow to consume tokens while preprocessing

Author: ibiryukov Date: Fri May 17 02:32:05 2019 New Revision: 361007 URL: http://llvm.org/viewvc/llvm-project?rev=361007&view=rev Log: [Lex] Allow to consume tokens while preprocessing Summary: By adding a hook to consume all tokens produced by the preprocessor. The intention of this change is t

[PATCH] D57922: [analyzer] Insert checker options into AnalyzerOption::ConfigTable

Szelethus added a comment. I committed but changed the moves to copies. Singel `CheckerRegisrty::CmdLineOption` is only filled with `StringRef`s, it shouldn't really matter. Comment at: lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp:314 insertOptionToCollection(StringRef Fu

[PATCH] D59885: [Lex] Allow to consume tokens while preprocessing

This revision was automatically updated to reflect the committed changes. Closed by commit rC361007: [Lex] Allow to consume tokens while preprocessing (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D59885?vs=11&id=22#toc Repository: rC Clang

r361011 - [analyzer] Validate checker option names and values

Author: szelethus Date: Fri May 17 02:51:59 2019 New Revision: 361011 URL: http://llvm.org/viewvc/llvm-project?rev=361011&view=rev Log: [analyzer] Validate checker option names and values Validate whether the option exists, and also whether the supplied value is of the correct type. With this pat

[PATCH] D60672: [libclang] visit c++14 lambda capture init expressions

milianw added a comment. @nik odd, it works for me, I just switched to master (previously I used the 8x release branch) and applied this patch and build it all. Then I run: $ make && ./bin/llvm-lit ~/projects/src/llvm-project/clang/test/Index/cxx14-lambdas.cpp makeobj[0]: Entering directory

[PATCH] D57860: [analyzer] Validate checker option names and values

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL361011: [analyzer] Validate checker option names and values (authored by Szelethus, committed by ). Herald added a project

[PATCH] D61974: [ObjC] Fix encoding of ObjC pointer types that are pointers to typedefs

theraven added inline comments. Comment at: lib/AST/ASTContext.cpp:6973 QualType PointeeTy = OPT->getPointeeType(); -if (!Options.EncodingProperty() && +if (getLangOpts().ObjCRuntime.isGNUFamily() && +!Options.EncodingProperty() && ahatanak w

[PATCH] D59887: [Syntax] Introduce TokenBuffer, start clangToolingSyntax library

ilya-biryukov updated this revision to Diff 200019. ilya-biryukov added a comment. - Remove the function that maps tokens to offsets Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59887/new/ https://reviews.llvm.org/D59887 Files: clang/include/cl

r360984 - [ELF] Implement Dependent Libraries Feature

Author: bd1976llvm Date: Thu May 16 20:44:15 2019 New Revision: 360984 URL: http://llvm.org/viewvc/llvm-project?rev=360984&view=rev Log: [ELF] Implement Dependent Libraries Feature This patch implements a limited form of autolinking primarily designed to allow either the --dependent-library compi

[PATCH] D62009: [clang] perform semantic checking in constant context

Tyker added a comment. I checked how we can propagate the information about constant context through semantic checking. there are issues with using ConstantExpr to mark expressions as constant for semantic checking: - #1 multpile Expr::Ignore* operation remove ConstantExpr from the expression.

Re: r360946 - Add Clang shared library with C++ exports

Thanks! Bots are happy with the reland. Thanks for taking the time to explain the object library stuff! *From: *Chris Bieneman *Date: *Thu, May 16, 2019 at 11:24 PM *To: *Nico Weber *Cc: *cfe-commits I’ll re-land disabling if PIC is off. > > This doesn’t replace the static libraries with object

[PATCH] D61837: Make it possible control matcher traversal kind with ASTContext

steveire marked an inline comment as done. steveire added inline comments. Comment at: lib/ASTMatchers/ASTMatchersInternal.cpp:240 + + assert(RestrictKind.isBaseOf(NodeKind)); + if (Implementation->dynMatches(N, Finder, Builder)) { aaron.ballman wrote: > Add an

[PATCH] D62046: [OpenMP][bugfix] Add missing math functions variants for log and abs.

jdoerfert added a comment. This LGTM me, @tra or @efriedma any objections? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62046/new/ https://reviews.llvm.org/D62046 ___ cfe-commits mailing list cfe-commits@lists.llvm.

[PATCH] D61835: Extract ASTDumper to a header file

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. In D61835#1505388 , @steveire wrote: > In D61835#1505314 , @aaron.ballman > wrote: > > > In D618

[PATCH] D61335: [LibTooling] Add support to Transformer for composing rules as an ordered choice.

ymandel marked 2 inline comments as done. ymandel added inline comments. Comment at: clang/include/clang/Tooling/Refactoring/Transformer.h:278 +/// Builds the matcher needed for registration. +ast_matchers::internal::DynTypedMatcher buildMatcher(const RewriteRule &Rule); + --

[PATCH] D61335: [LibTooling] Add support to Transformer for composing rules as an ordered choice.

ymandel updated this revision to Diff 200026. ymandel marked 2 inline comments as done. ymandel added a comment. updated comments; moved some decls. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61335/new/ https://reviews.llvm.org/D61335 Files:

Re: r361011 - [analyzer] Validate checker option names and values

It looks like this change is making gcc-7 crash on these (and other http://lab.llvm.org:8011/console) bots: http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick/builds/18639 http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/33837/ [100/212] Building CXX object tools/clang/unitte

[PATCH] D59887: [Syntax] Introduce TokenBuffer, start clangToolingSyntax library

ilya-biryukov updated this revision to Diff 200030. ilya-biryukov added a comment. - Skip annotation tokens, some of them are reported by the preprocessor but we don't want them in the final expanded stream. - Add functions to compute FileRange of tokens, add tests for it. Repository: rG LLVM

[PATCH] D61837: Make it possible control matcher traversal kind with ASTContext

aaron.ballman added inline comments. Comment at: lib/AST/ASTContext.cpp:120 +ASTContext::TraverseIgnored(const ast_type_traits::DynTypedNode &N) { + if (auto E = N.get()) { +return ast_type_traits::DynTypedNode::create(*TraverseIgnored(E)); aaron.ballman wro

[PATCH] D61333: [ASTImporter] Fix LLDB lookup in transparent ctx and with ext src

martong updated this revision to Diff 200031. martong added a comment. - Rebase to master - Rebase to D62061 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61333/new/ https://reviews.llvm.org/D61333 Files: clang/

[PATCH] D61333: [ASTImporter] Fix LLDB lookup in transparent ctx and with ext src

martong marked an inline comment as done. martong added inline comments. Comment at: lldb/source/Symbol/ClangASTImporter.cpp:922 + + Log *log_ast(lldb_private::GetLogIfAllCategoriesSet(LIBLLDB_LOG_AST)); + if (log_ast) { shafik wrote: > I am going to sa

[PATCH] D61333: [ASTImporter] Fix LLDB lookup in transparent ctx and with ext src

martong updated this revision to Diff 200033. martong added a comment. - se -> so Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61333/new/ https://reviews.llvm.org/D61333 Files: clang/lib/AST/ASTImporter.cpp clang/unittests/AST/ASTImporterTest

[PATCH] D61743: New clang option -MD-filter=prefix to filter files from make dependencies

mibintc added a comment. @dexonsmith Can you take a look at this patch or recommend someone who can review it? Many thanks. --Melanie CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61743/new/ https://reviews.llvm.org/D61743 ___ cfe-commits

[PATCH] D61834: Add a Visit overload for DynTypedNode to ASTNodeTraverser

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. In D61834#1505418 , @steveire wrote: > In D61834#1505124 , @aaron.ballman > wrote: > > > In D618

r361032 - Fix Wdocumentation warnings. NFCI.

Author: rksimon Date: Fri May 17 06:42:16 2019 New Revision: 361032 URL: http://llvm.org/viewvc/llvm-project?rev=361032&view=rev Log: Fix Wdocumentation warnings. NFCI. Modified: cfe/trunk/include/clang/Sema/Sema.h Modified: cfe/trunk/include/clang/Sema/Sema.h URL: http://llvm.org/viewvc/ll

[PATCH] D59628: Add support for __attribute__((objc_class_stub))

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/Attr.td:292 + // "LangOpts" bound. + string CustomCode = customCode; } If this is code, should it be using a `code` type rather than a `string` type? Comment at: clan

r361033 - Add a Visit overload for DynTypedNode to ASTNodeTraverser

Author: steveire Date: Fri May 17 06:55:28 2019 New Revision: 361033 URL: http://llvm.org/viewvc/llvm-project?rev=361033&view=rev Log: Add a Visit overload for DynTypedNode to ASTNodeTraverser Reviewers: aaron.ballman Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews

[PATCH] D61834: Add a Visit overload for DynTypedNode to ASTNodeTraverser

steveire marked 5 inline comments as done. steveire added inline comments. Comment at: unittests/AST/ASTTraverserTest.cpp:75 + +template std::string dumpASTString(NodeType &&... N) { + std::string Buffer; aaron.ballman wrote: > Did clang-format produce this for

[PATCH] D61834: Add a Visit overload for DynTypedNode to ASTNodeTraverser

This revision was automatically updated to reflect the committed changes. steveire marked an inline comment as done. Closed by commit rL361033: Add a Visit overload for DynTypedNode to ASTNodeTraverser (authored by steveire, committed by ). Herald added a project: LLVM. Herald added a subscriber:

r361034 - Extract ASTDumper to a header file

Author: steveire Date: Fri May 17 06:59:15 2019 New Revision: 361034 URL: http://llvm.org/viewvc/llvm-project?rev=361034&view=rev Log: Extract ASTDumper to a header file Summary: This class has member APIs which are useful to clients. Make it possible to use those APIs without adding them to dum

Re: r361011 - [analyzer] Validate checker option names and values

I'll take a look. From: Nico Weber Sent: 17 May 2019 15:09:18 To: Kristof Umann Cc: cfe-commits Subject: Re: r361011 - [analyzer] Validate checker option names and values It looks like this change is making gcc-7 crash on these (and other http://lab.llvm.org:801

[PATCH] D61508: [clang-tidy] bugprone-header-guard : a simple version of llvm-header-guard

aaron.ballman added inline comments. Comment at: clang-tidy/bugprone/HeaderGuardCheck.cpp:50 + + } else { +if (OldGuard.size()) I think this should be an `else if` rather than an `else`. I'd like to see us diagnose unknown guard styles so that we only accep

[PATCH] D61835: Extract ASTDumper to a header file

steveire added a comment. Great, thanks. I'm going to investigate whether we can move the `dump` method implementations to their respective class files, and then look into a rename for this to `StreamNodeDumper` or so (name tbd). Repository: rC Clang CHANGES SINCE LAST ACTION https://rev

[PATCH] D61835: Extract ASTDumper to a header file

This revision was automatically updated to reflect the committed changes. Closed by commit rC361034: Extract ASTDumper to a header file (authored by steveire, committed by ). Changed prior to commit: https://reviews.llvm.org/D61835?vs=199179&id=200038#toc Repository: rC Clang CHANGES SINCE

[PATCH] D61834: Add a Visit overload for DynTypedNode to ASTNodeTraverser

aaron.ballman added inline comments. Comment at: unittests/AST/ASTTraverserTest.cpp:75 + +template std::string dumpASTString(NodeType &&... N) { + std::string Buffer; steveire wrote: > aaron.ballman wrote: > > Did clang-format produce this formatting? (If not,

[PATCH] D61335: [LibTooling] Add support to Transformer for composing rules as an ordered choice.

ymandel updated this revision to Diff 200042. ymandel marked an inline comment as done. ymandel added a comment. Synced to HEAD in preparation for committing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61335/new/ https://reviews.llvm.org/D61335

[PATCH] D62064: [ASTImporter] Fix unhandled cases in ASTImporterLookupTable

martong created this revision. martong added a reviewer: a_sidorin. Herald added subscribers: cfe-commits, gamesh411, Szelethus, dkrupp, rnkovacs. Herald added a reviewer: a.sidorin. Herald added a reviewer: shafik. Herald added a project: clang. In most cases the FriendDecl contains the declarati

[PATCH] D58035: [clang/DIVar] Emit flag for params that have unchanged values

djtodoro marked 3 inline comments as done. djtodoro added inline comments. Comment at: lib/CodeGen/CGDebugInfo.cpp:4537 + CGM.getLangOpts().Optimize) { +for (auto &SP : DeclCache) { + auto *D = SP.first; aprantl wrote: > djtodoro wrote: > > aprantl

r361037 - [LibTooling] Add support to Transformer for composing rules as an ordered choice.

Author: ymandel Date: Fri May 17 07:23:33 2019 New Revision: 361037 URL: http://llvm.org/viewvc/llvm-project?rev=361037&view=rev Log: [LibTooling] Add support to Transformer for composing rules as an ordered choice. This revision updates `RewriteRule` to support multiple subrules that are interp

[PATCH] D61335: [LibTooling] Add support to Transformer for composing rules as an ordered choice.

This revision was automatically updated to reflect the committed changes. Closed by commit rC361037: [LibTooling] Add support to Transformer for composing rules as an ordered… (authored by ymandel, committed by ). Changed prior to commit: https://reviews.llvm.org/D61335?vs=200042&id=200044#toc

[PATCH] D59887: [Syntax] Introduce TokenBuffer, start clangToolingSyntax library

sammccall accepted this revision. sammccall added inline comments. Comment at: clang/include/clang/Tooling/Syntax/Tokens.h:49 + +/// A half-open range inside a particular file, the start offset is included and +/// the end offset is excluded from the range. nit:

[PATCH] D62065: Move dump method implementations to their respective class files

steveire created this revision. steveire added a reviewer: aaron.ballman. Herald added a project: clang. Herald added a subscriber: cfe-commits. There is no need to group them together. Repository: rC Clang https://reviews.llvm.org/D62065 Files: lib/AST/ASTDumper.cpp lib/AST/Comment.cpp

[PATCH] D62066: [ASTImporter] Enable disabled but passing tests

martong created this revision. martong added a reviewer: a_sidorin. Herald added subscribers: cfe-commits, gamesh411, Szelethus, dkrupp, rnkovacs. Herald added a reviewer: a.sidorin. Herald added a reviewer: shafik. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://revi

[PATCH] D61837: Make it possible control matcher traversal kind with ASTContext

steveire marked an inline comment as done. steveire added inline comments. Comment at: lib/ASTMatchers/ASTMatchersInternal.cpp:240 + + assert(RestrictKind.isBaseOf(NodeKind)); + if (Implementation->dynMatches(N, Finder, Builder)) { aaron.ballman wrote: > stevei

[PATCH] D60568: [OpenMP] Add support for registering requires directives with the runtime

gtbercea marked an inline comment as done. gtbercea added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:10364 + if (const auto *FD = dyn_cast(D)) { +if (OMPDeclareTargetDeclAttr::isDeclareTargetDeclaration(FD)) { + HasEmittedDeclareTargetRegion = true; --

[PATCH] D37813: clang-format: better handle namespace macros

Typz added a comment. Herald added a project: clang. ping ? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D37813/new/ https://reviews.llvm.org/D37813 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[PATCH] D60953: [clangd] Respect clang-tidy suppression comments

nridge added a comment. Could someone merge this and D61841 now that they're approved? Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60953/new/ https://reviews.llvm.org/D60953 __

Re: r361011 - [analyzer] Validate checker option names and values

http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick seems to be doing fine now, http://lab.llvm.org:8011/builders/clang-ppc64be-linux seems to crash on files totally unrelated to mine. From: Kristof Umann Sent: 17 May 2019 15:59:28 To: Nico Weber Cc: cfe-

[PATCH] D60568: [OpenMP] Add support for registering requires directives with the runtime

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:10364 + if (const auto *FD = dyn_cast(D)) { +if (OMPDeclareTargetDeclAttr::isDeclareTargetDeclaration(FD)) { + HasEmittedDeclareTargetRegion = true; gtbercea wrote: > ABataev wro

[PATCH] D62046: [OpenMP][bugfix] Add missing math functions variants for log and abs.

gtbercea updated this revision to Diff 200048. gtbercea added a comment. - update patch Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62046/new/ https://reviews.llvm.org/D62046 Files: lib/Headers/__clang_cuda_cmath.h lib/Headers/__clang_cuda_math_forward_dec

[PATCH] D62046: [OpenMP][bugfix] Add missing math functions variants for log and abs.

gtbercea added a comment. @tra I eliminated the long double definition for log and only left the declaration. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62046/new/ https://reviews.llvm.org/D62046 ___ cfe-commits

[PATCH] D60568: [OpenMP] Add support for registering requires directives with the runtime

gtbercea marked an inline comment as done. gtbercea added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:10364 + if (const auto *FD = dyn_cast(D)) { +if (OMPDeclareTargetDeclAttr::isDeclareTargetDeclaration(FD)) { + HasEmittedDeclareTargetRegion = true; --

r361042 - [analyzer] Remove the default value arg from getChecker*Option

Author: szelethus Date: Fri May 17 08:52:13 2019 New Revision: 361042 URL: http://llvm.org/viewvc/llvm-project?rev=361042&view=rev Log: [analyzer] Remove the default value arg from getChecker*Option Since D57922, the config table contains every checker option, and it's default value, so having it

[PATCH] D59195: [analyzer] Remove the default value arg from getChecker*Option

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL361042: [analyzer] Remove the default value arg from getChecker*Option (authored by Szelethus, committed by ). Herald adde

[PATCH] D60568: [OpenMP] Add support for registering requires directives with the runtime

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:9291 + if (CGM.getLangOpts().OpenMPSimd || CGM.getLangOpts().OpenMPIsDevice || + (OffloadEntriesInfoManager.empty() && !HasEmittedDeclareTargetRegion)) +return nullptr; Missed ch

Re: r361033 - Add a Visit overload for DynTypedNode to ASTNodeTraverser

Looks like this caused a breakage: http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick/builds/18641/steps/ninja%20check%201/logs/stdio On Fri, May 17, 2019 at 9:52 AM Stephen Kelly via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: steveire > Date: Fri May 17 06:55:28 2019 > N

Re: r361011 - [analyzer] Validate checker option names and values

Looks like your r361042 fixed the ppc64be bot nonetheless. Maybe it was related after all :) *From: *Kristof Umann *Date: *Fri, May 17, 2019 at 11:10 AM *To: *Nico Weber *Cc: *cfe-commits http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick seems to be > doing fine now, http://lab.llvm.or

[PATCH] D62046: [OpenMP][bugfix] Add missing math functions variants for log and abs.

tra accepted this revision. tra added a comment. This revision is now accepted and ready to land. I'd add a comment with a brief explanation for the const variant and a TODO() to remove it. Looks OK to me otherwise. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D59887: [Syntax] Introduce TokenBuffer, start clangToolingSyntax library

ilya-biryukov added inline comments. Comment at: clang/include/clang/Tooling/Syntax/Tokens.h:115 + /// expansion. + llvm::Optional range(const SourceManager &SM) const; + sammccall wrote: > I think this might need a more explicit name. It's reasonably obvious t

[PATCH] D61774: [LibTooling] Add RangeSelector library for defining source ranges based on bound AST nodes.

ymandel updated this revision to Diff 200061. ymandel marked 4 inline comments as done. ymandel added a comment. Restructured tests to simplify. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61774/new/ https://reviews.llvm.org/D61774 Files: clan

[PATCH] D61774: [LibTooling] Add RangeSelector library for defining source ranges based on bound AST nodes.

ymandel marked 7 inline comments as done. ymandel added inline comments. Comment at: clang/include/clang/Tooling/Refactoring/RangeSelector.h:29 + +namespace range_selector { +inline RangeSelector charRange(CharSourceRange R) { ilya-biryukov wrote: > ymandel wrote

[PATCH] D58035: [clang/DIVar] Emit flag for params that have unchanged values

aprantl added inline comments. Comment at: lib/CodeGen/CGDebugInfo.cpp:4537 + CGM.getLangOpts().Optimize) { +for (auto &SP : DeclCache) { + auto *D = SP.first; djtodoro wrote: > aprantl wrote: > > djtodoro wrote: > > > aprantl wrote: > > > > Just lo

[PATCH] D59887: [Syntax] Introduce TokenBuffer, start clangToolingSyntax library

sammccall added inline comments. Comment at: clang/include/clang/Tooling/Syntax/Tokens.h:115 + /// expansion. + llvm::Optional range(const SourceManager &SM) const; + ilya-biryukov wrote: > sammccall wrote: > > I think this might need a more explicit name. It's

[PATCH] D61522: Added an assertion to constant evaluation enty points that prohibits dependent expressions

This revision was automatically updated to reflect the committed changes. Closed by commit rL361050: Added an assertion to constant evaluation enty points that prohibits dependent… (authored by gribozavr, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Chang

Re: r360998 - Fix crash if, during evaluation of __builtin_object_size, we try to load

The test fails on Windows: *From: *Richard Smith via cfe-commits *Date: *Fri, May 17, 2019 at 3:58 AM *To: * Author: rsmith > Date: Fri May 17 01:01:34 2019 > New Revision: 360998 > > URL: http://llvm.org/viewvc/llvm-project?rev=360998&view=rev > Log: > Fix crash if, during evaluation of __bui

Re: r360998 - Fix crash if, during evaluation of __builtin_object_size, we try to load

-- Testing: 49966 tests, 32 threads -- Testing: 0 .. 10.. 20.. FAIL: Clang :: SemaCXX/builtin-object-size-cxx14.cpp (14324 of 49966) TEST 'Clang :: SemaCXX/builtin-object-size-cxx14.cpp' FAILED Script: -- : 'RUN: at line 1'; c:\b\s\w\ir\k\src\third_party\ll

r361050 - Added an assertion to constant evaluation enty points that prohibits dependent expressions

Author: gribozavr Date: Fri May 17 10:16:53 2019 New Revision: 361050 URL: http://llvm.org/viewvc/llvm-project?rev=361050&view=rev Log: Added an assertion to constant evaluation enty points that prohibits dependent expressions Summary: Constant evaluator does not work on value-dependent or type-

Re: r360998 - Fix crash if, during evaluation of __builtin_object_size, we try to load

Also on the LLVM waterfall: http://lab.llvm.org:8011/builders/clang-x64-windows-msvc http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/6937/steps/stage%201%20check/logs/stdio (Same error.) *From: *Nico Weber *Date: *Fri, May 17, 2019 at 1:14 PM *To: *Richard Smith *Cc: *cfe-commits

r361054 - Attempt to fix test on Windows after r360998

Author: nico Date: Fri May 17 10:33:54 2019 New Revision: 361054 URL: http://llvm.org/viewvc/llvm-project?rev=361054&view=rev Log: Attempt to fix test on Windows after r360998 Modified: cfe/trunk/test/SemaCXX/builtin-object-size-cxx14.cpp Modified: cfe/trunk/test/SemaCXX/builtin-object-size-

Re: r360998 - Fix crash if, during evaluation of __builtin_object_size, we try to load

I attempted to fix this in 361054. *From: *Nico Weber *Date: *Fri, May 17, 2019 at 1:16 PM *To: *Richard Smith *Cc: *cfe-commits Also on the LLVM waterfall: > http://lab.llvm.org:8011/builders/clang-x64-windows-msvc > > http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/6937/steps/s

Re: r361033 - Add a Visit overload for DynTypedNode to ASTNodeTraverser

Hmm, thanks for letting me know! Is that an old compiler? I'm not near a computer for the weekend. Can someone fix or revert? Thanks, Stephen. On Fri 17 May 2019, 17:11 Yitzhak Mandelbaum, wrote: > Looks like this caused a breakage: > http://lab.llvm.org:8011/builders/clang-cmake-aarch64-qu

Re: r361033 - Add a Visit overload for DynTypedNode to ASTNodeTraverser

i'll see if i can repro on my build and fix if so. On Fri, May 17, 2019 at 1:56 PM Stephen Kelly wrote: > Hmm, thanks for letting me know! > > Is that an old compiler? > > I'm not near a computer for the weekend. > > Can someone fix or revert? > > Thanks, > > Stephen. > > On Fri 17 May 2019, 17:

Re: r360998 - Fix crash if, during evaluation of __builtin_object_size, we try to load

Looks good, thanks :) On Fri, 17 May 2019, 10:31 Nico Weber via cfe-commits, < cfe-commits@lists.llvm.org> wrote: > I attempted to fix this in 361054. > > *From: *Nico Weber > *Date: *Fri, May 17, 2019 at 1:16 PM > *To: *Richard Smith > *Cc: *cfe-commits > > Also on the LLVM waterfall: >> http:/

[PATCH] D62049: [clang-tidy] Add a close-on-exec check on pipe2() in Android module.

jcai19 updated this revision to Diff 200066. jcai19 added a comment. Update based on comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62049/new/ https://reviews.llvm.org/D62049 Files: clang-tools-extra/clang-tidy/android/AndroidTidyModule

[PATCH] D61967: [clang-tidy] Add a close-on-exec check on pipe() in Android module.

jcai19 updated this revision to Diff 200068. jcai19 marked an inline comment as done. jcai19 added a comment. Update based on comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61967/new/ https://reviews.llvm.org/D61967 Files: clang-tools-e

[PATCH] D61909: Add Clang shared library with C++ exports

sylvestre.ledru added a comment. @beanz @tstellar I am wondering what to do wrt apt.llvm.org should it be part of libclang or create a libclang++ package what do you think? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61909/new/ https://reviews.llvm.org/D61909

[PATCH] D61909: Add Clang shared library with C++ exports

beanz added a comment. I would leave it out of any distribution (at least for now). Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61909/new/ https://reviews.llvm.org/D61909 ___ cfe-commits mailing list cfe-commits@li

  1   2   >