r366782 - [CrossTU] Added CTU argument to diagnostic consumer create fn.

Author: balazske Date: Tue Jul 23 00:04:20 2019 New Revision: 366782 URL: http://llvm.org/viewvc/llvm-project?rev=366782&view=rev Log: [CrossTU] Added CTU argument to diagnostic consumer create fn. Summary: The PListDiagnosticConsumer needs a new CTU parameter that is passed through the create fu

[PATCH] D64635: [CrossTU] Added CTU argument to diagnostic consumer create fn.

This revision was automatically updated to reflect the committed changes. Closed by commit rL366782: [CrossTU] Added CTU argument to diagnostic consumer create fn. (authored by balazske, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit

[PATCH] D65101: [clangd] SelectionTree treats TranslationUnitDecl consistently with other containers.

hokein added inline comments. Comment at: clang-tools-extra/clangd/Selection.h:100 }; // The most specific common ancestor of all the selected nodes. + const Node &commonAncestor() const; I think it would be nice to mention "what means no selection" (TUDec

[PATCH] D64991: [analyzer][WIP] Implement a primitive reaching definitions analysis

Szelethus added a reviewer: steakhal. Szelethus added a subscriber: steakhal. Szelethus added a comment. @steakhal you some great experience with (strict) aliasing, could you chip in on this maybe? :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64991/new/ https://reviews.llvm.org/D64

[PATCH] D65092: [clang] Add isDirectlyDerivedFrom AST Matcher.

AntonBikineev updated this revision to Diff 211261. AntonBikineev added a comment. Thanks for the note, updated the diff with autogenerated LibASTMatcherReference.html. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65092/new/ https://reviews.llvm.org/D65092 Fil

[PATCH] D64739: [SVE][Inline-Asm] Add support to specify SVE registers in the clobber list

kmclaughlin updated this revision to Diff 211264. kmclaughlin added a comment. - Removed second test in aarch64-sve-inline-asm.c which didn't use //-emit-llvm// CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64739/new/ https://reviews.llvm.org/D64739 Files: clang/lib/Basic/Targets/AA

[PATCH] D64739: [SVE][Inline-Asm] Add support to specify SVE registers in the clobber list

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

[PATCH] D27918: [analyzer] OStreamChecker

gamesh411 abandoned this revision. gamesh411 added a comment. Herald added subscribers: Charusso, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware. There are multiple new additions to stream formatters, and this patch is way too old. I may consider creating a new che

[PATCH] D59934: Compare SourceLocations from different TUs by FileID

gamesh411 abandoned this revision. gamesh411 added a comment. 32f220c5fbe5 is the more sophisticated solution to the problem. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59934/new/ https://

[PATCH] D65102: [OpenCL] Rename lang mode flag for C++ mode

kpet added a comment. Thanks for doing this! Comment at: include/clang/Frontend/LangStandards.def:177 LANGSTANDARD_ALIAS_DEPR(opencl20, "CL2.0") +LANGSTANDARD_ALIAS_DEPR(opencl20, "CLC++") Shouldn't this be `openclcpp`? CHANGES SINCE LAST ACTION https://

r366792 - PlistDiagnostics Fix for compile warning (NFC).

Author: balazske Date: Tue Jul 23 03:21:42 2019 New Revision: 366792 URL: http://llvm.org/viewvc/llvm-project?rev=366792&view=rev Log: PlistDiagnostics Fix for compile warning (NFC). Modified: cfe/trunk/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp Modified: cfe/trunk/lib/StaticAnalyzer/Core/

[PATCH] D64488: [Driver] Support -fsanitize=function on Solaris/x86

ro added a comment. Ping^2? Two weeks now. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64488/new/ https://reviews.llvm.org/D64488 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[PATCH] D64487: [clang, test] Fix Clang :: Headers/max_align.c on 64-bit SPARC

ro added a comment. Ping^2? It's been two weeks now. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64487/new/ https://reviews.llvm.org/D64487 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llv

[PATCH] D64793: [Driver] Properly use values-X[ca].o, values-xpg[46].o on Solaris

ro added a comment. Ping? It's been a week now and this is a correctness issue on Solaris, not just a single failing test. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64793/new/ https://reviews.llvm.org/D64793 __

[PATCH] D65139: [clangd] Support extraction of binary "subexpressions" like a + [[b + c]].

sammccall created this revision. sammccall added a reviewer: SureYeaah. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. These aren't formally subexpressions in C++, in this case + is left-associative. However informally +,

[PATCH] D63126: [clangd] Implement "prepareRename"

sammccall added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:393 ; + if (Params.capabilities.RenamePrepareSupport) +Result["renameProvider"] = llvm::json::Object{{"prepareProvider", true}}; why set this only if the c

[PATCH] D65065: [clang-tidy] Possibility of displaying duplicate warnings

bruntib added a comment. Yes, LessClangTidyError would really be the best place for this. The reason of my implementation was that I wanted to bind this feature to a command line option. I don't know if there was any design decision behind the current uniquing logic and I didn't want to break i

[PATCH] D64666: [Sema] Enable -Wimplicit-float-conversion for integral to floating point precision loss

aaron.ballman requested changes to this revision. aaron.ballman added a comment. This revision now requires changes to proceed. In D64666#1596660 , @xbolva00 wrote: > I think we should warn in that case even if GCC does not warn. Strong +1. In D64666#15

[PATCH] D64737: RISCV: Add support for floating point registers in inlineasm

lenary accepted this revision. lenary added a comment. This revision is now accepted and ready to land. Looks good to me! Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64737/new/ https://reviews.llvm.org/D64737 ___

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh updated this revision to Diff 211278. lildmh added a comment. Herald added a reviewer: jdoerfert. Rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59474/new/ https://reviews.llvm.org/D59474 Files: include/clang/AST/GlobalDecl.h lib/AST/ASTContext.cpp lib/CodeGen/CGDecl

[PATCH] D64487: [clang, test] Fix Clang :: Headers/max_align.c on 64-bit SPARC

jyknight accepted this revision. jyknight added a comment. This revision is now accepted and ready to land. LGTM. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64487/new/ https://reviews.llvm.org/D64487 ___ cfe-commi

[PATCH] D64998: Improve clang-format-diff help output

krasimir accepted this revision. krasimir added inline comments. This revision is now accepted and ready to land. Comment at: clang/tools/clang-format/clang-format-diff.py:11 -r""" -ClangFormat Diff Reformatter - - +""" This script reads input from

[PATCH] D65125: clang-format: Fix namespace end comments for namespaces with attributes and macros

klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. LG, thx! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65125/new/ https://reviews.llvm.org/D65125 ___ cfe-commits mailing list cfe-commi

[PATCH] D63126: [clangd] Implement "prepareRename"

hokein updated this revision to Diff 211288. hokein marked 6 inline comments as done. hokein added a comment. Address review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63126/new/ https://reviews.llvm.org/D63126 Files: clang-tools-ex

[PATCH] D64638: [CrossTU] Fix plist macro expansion if macro in other file.

martong added a subscriber: NoQ. martong added a comment. In D64638#1593382 , @ilya-biryukov wrote: > `StaticAnalyzer/Core` does not depend on `clangFrontend` now, you can see > this by looking at `lib/StaticAnalyzer/Core/CMakeLists.txt`: > > add_clang

[PATCH] D63126: [clangd] Implement "prepareRename"

hokein added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:393 ; + if (Params.capabilities.RenamePrepareSupport) +Result["renameProvider"] = llvm::json::Object{{"prepareProvider", true}}; sammccall wrote: > why set th

[PATCH] D61466: [Rewrite][NFC] Add FIXMEs and tests for RemoveLineIfEmpty bug

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

[PATCH] D65146: [clangd] Log version, cwd, args, and transport on startup. NFC

sammccall created this revision. sammccall added a reviewer: hokein. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D65146 Files: clang-tools-extra/clangd

[PATCH] D64638: [CrossTU] Fix plist macro expansion if macro in other file.

balazske added a comment. It is possible to use the `Preprocessor` instead of `ASTUnit` in `getImportedFromSourceLocation`. But this will contain less useful (even if currently not used) information. The test extension in D65064 is not possible to do if not `AS

Re: r366694 - [NFC] Relaxed regression tests for PR42665

+ cfe-commits From: Anastasia Stulova Sent: 23 July 2019 15:16 To: Hans Wennborg Cc: Marco Antognini Subject: Re: r366694 - [NFC] Relaxed regression tests for PR42665 @Hans, would it be possible to merge this commit along with r366670 to the release 9.0? FY

[PATCH] D65146: [clangd] Log version, cwd, args, and transport on startup. NFC

hokein accepted this revision. hokein added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/tool/ClangdMain.cpp:436 + { +std::string LoggableArgs; +for (int I = 0; I < argc; ++I) this variable is not

[PATCH] D63126: [clangd] Implement "prepareRename"

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:393 ; + if (Params.capabilities.RenamePrepareSupport) +Result["renameProvider"] = llvm::j

[PATCH] D65149: [Format] Add test demonstrating PR42722

modocache created this revision. modocache added reviewers: ank, klimek, acoomans. Herald added a project: clang. https://bugs.llvm.org/show_bug.cgi?id=42722 describes what I believe to be a bug in lambda formatting. If it is indeed a bug, I'd like to commit this test that reliably reproduces it.

[PATCH] D65064: [CrossTU] Add a function to retrieve original source location.

balazske marked an inline comment as done. balazske added inline comments. Comment at: clang/include/clang/CrossTU/CrossTranslationUnit.h:175 + /// returned. + llvm::Optional> + getImportedFromSourceLocation(const clang::SourceLocation &ToLoc) const; It would

[clang-tools-extra] r366811 - [clangd] Log version, cwd, args, and transport on startup. NFC

Author: sammccall Date: Tue Jul 23 07:30:28 2019 New Revision: 366811 URL: http://llvm.org/viewvc/llvm-project?rev=366811&view=rev Log: [clangd] Log version, cwd, args, and transport on startup. NFC Reviewers: hokein Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, cfe-commits

[PATCH] D65146: [clangd] Log version, cwd, args, and transport on startup. NFC

This revision was automatically updated to reflect the committed changes. Closed by commit rL366811: [clangd] Log version, cwd, args, and transport on startup. NFC (authored by sammccall, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commi

[PATCH] D60663: Time profiler: small fixes and optimizations

sammccall added inline comments. Comment at: llvm/trunk/lib/Support/TimeProfiler.cpp:27 +static cl::opt TimeTraceGranularity( +"time-trace-granularity", I know this is late, but... this shows up in the help for any tool that links in libSupport, many of wh

[PATCH] D63126: [clangd] Implement "prepareRename"

hokein updated this revision to Diff 211302. hokein marked 4 inline comments as done. hokein added a comment. Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63126/new/ https://reviews.llvm.org/D63126 Files: clang-tools-extra/clan

Re: r366694 - [NFC] Relaxed regression tests for PR42665

Merged them both in r366814. Thanks, Hans On Tue, Jul 23, 2019 at 7:20 AM Anastasia Stulova wrote: > > > + cfe-commits > > > From: Anastasia Stulova > Sent: 23 July 2019 15:16 > To: Hans Wennborg > Cc: Marco Antognini > Subject: Re: r366694 - [NFC] Relaxed regr

[PATCH] D64638: [CrossTU] Fix plist macro expansion if macro in other file.

balazske added a comment. Maybe CrossTU should not use the Frontend library? (Now the StaticAnalyzerCore is using CrossTU and CrossTU is using Frontend, in this way StaticAnalyzerCore could use Frontend too.) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64638/n

[PATCH] D65153: [clangd] Also accept flags from CLANGD_FLAGS variable.

sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. This simplifies various workflows, particularly in debugging/development. e.g. editors will tend to propagate fla

[PATCH] D64883: Add new warning -Walloca for use of builtin alloca function

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:2776 +def warn_alloca : Warning< + "use of builtin function %0">, + InGroup>, DefaultIgnore; george.burgess.iv wrote: > aaron.ballman wrote: > > george.burgess.iv

[PATCH] D63954: Add lifetime categories attributes

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/AttrDocs.td:4167 + +The attribute ``[[gsl::Owner(T)]]`` applies to structs and classes that own an +object of type ``T``: Do either of these attributes make sense on a union type? If so, m

[PATCH] D62829: [clang-tidy] Check for dynamically initialized statics in headers.

aaron.ballman added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/checks/bugprone-dynamic-static-initializers.rst:9 + +This can pose problems in certain multithreaded contexts. Eugene.Zelenko wrote: > Will be good idea to provide example. Agreed

[PATCH] D64793: [Driver] Properly use values-X[ca].o, values-xpg[46].o on Solaris

jyknight added a comment. Is this really necessary? Users don't typically pass -std= to the driver for linking anyways (what do you even pass if you've compiled both C and C++ code?) so this seems a rather odd way to control behavior. How about instead just adding "values-xpg6.o" unconditionall

[PATCH] D65154: [clangd] Reformat use of cl::opt: use unqualified name and don't bin-pack attributes. NFC

sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D65154 Files: clang-tools-extra/clangd/tool/Cl

r366818 - [ASTImporter] Fix inequivalence of ClassTemplateInstantiations

Author: martong Date: Tue Jul 23 08:46:38 2019 New Revision: 366818 URL: http://llvm.org/viewvc/llvm-project?rev=366818&view=rev Log: [ASTImporter] Fix inequivalence of ClassTemplateInstantiations Summary: We falsely state inequivalence if the template parameter is a qualified/nonquialified templ

[PATCH] D64241: [ASTImporter] Fix inequivalence of ClassTemplateInstantiations

This revision was automatically updated to reflect the committed changes. Closed by commit rL366818: [ASTImporter] Fix inequivalence of ClassTemplateInstantiations (authored by martong, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D59692: [ASTImporter] Fix name conflict handling

martong updated this revision to Diff 211312. martong added a comment. - Rebase to master Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59692/new/ https://reviews.llvm.org/D59692 Files: clang/include/clang/AST/ASTImporter.h clang/lib/AST/ASTIm

[PATCH] D64914: Implement P1771

aaron.ballman added a comment. In D64914#1595660 , @erichkeane wrote: > Rebased and did all the comments (including the www_status). @aaron.ballman > : Wasn't positive what you meant about the conversion functions, but I think > I got one? I was talki

[PATCH] D44100: [ASTImporter] Reorder fields after structure import is finished

martong updated this revision to Diff 211314. martong marked 5 inline comments as done. martong added a comment. - Address Alexei's comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D44100/new/ https://reviews.llvm.org/D44100 Files: clang/li

[PATCH] D64666: [Sema] Enable -Wimplicit-float-conversion for integral to floating point precision loss

jfb requested changes to this revision. jfb added a comment. Herald added a subscriber: dexonsmith. In D64666#1597193 , @aaron.ballman wrote: > In D64666#1596660 , @xbolva00 wrote: > > > I think we should warn in t

[PATCH] D44100: [ASTImporter] Reorder fields after structure import is finished

martong added inline comments. Comment at: clang/unittests/AST/ASTImporterTest.cpp:5244 +} + + a_sidorin wrote: > A redundant newline? Yes, I deleted that. And moved the test case closer the the last `ImportDecl` test case. Repository: rG LLVM Github Monorep

r366820 - [clang, test] Fix Clang :: Headers/max_align.c on 64-bit SPARC

Author: ro Date: Tue Jul 23 09:24:00 2019 New Revision: 366820 URL: http://llvm.org/viewvc/llvm-project?rev=366820&view=rev Log: [clang, test] Fix Clang :: Headers/max_align.c on 64-bit SPARC Clang :: Headers/max_align.c currently FAILs on 64-bit SPARC: error: 'error' diagnostics seen but not

[PATCH] D65102: [OpenCL] Rename lang mode flag for C++ mode

Anastasia updated this revision to Diff 211315. Anastasia added a comment. - Fixed `LANGSTANDARD_ALIAS_DEPR` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65102/new/ https://reviews.llvm.org/D65102 Files: include/clang/Driver/Options.td include/clang/Frontend/LangStandards.def li

[PATCH] D65102: [OpenCL] Rename lang mode flag for C++ mode

Anastasia marked an inline comment as done. Anastasia added inline comments. Comment at: include/clang/Frontend/LangStandards.def:177 LANGSTANDARD_ALIAS_DEPR(opencl20, "CL2.0") +LANGSTANDARD_ALIAS_DEPR(opencl20, "CLC++") kpet wrote: > Shouldn't this be `opencl

[PATCH] D64487: [clang, test] Fix Clang :: Headers/max_align.c on 64-bit SPARC

This revision was automatically updated to reflect the committed changes. Closed by commit rG2da6eea07cde: [clang, test] Fix Clang :: Headers/max_align.c on 64-bit SPARC (authored by ro). Changed prior to commit: https://reviews.llvm.org/D64487?vs=208946&id=211316#toc Repository: rG LLVM Git

[PATCH] D65102: [OpenCL] Rename lang mode flag for C++ mode

kpet added a comment. Hmm, maybe we need to make sure that one of the tests is using a C++ feature and building with `CLC++`. This would have caught the mistake. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65102/new/ https://reviews.llvm.org/D65102

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

ABataev added inline comments. Comment at: lib/CodeGen/CGDecl.cpp:2533 + CodeGenFunction *CGF) { + if (!LangOpts.OpenMP || LangOpts.OpenMPSimd || + (!LangOpts.EmitAllDecls && !D->isUsed())) Why do we need to emit it f

Re: r366694 - [NFC] Relaxed regression tests for PR42665

Great! Thanks! From: Hans Wennborg Sent: 23 July 2019 15:58 To: Anastasia Stulova Cc: Marco Antognini ; Clang Commits ; nd Subject: Re: r366694 - [NFC] Relaxed regression tests for PR42665 Merged them both in r366814. Thanks, Hans On Tue, Jul 23, 2019 at 7:

[PATCH] D65102: [OpenCL] Rename lang mode flag for C++ mode

Anastasia updated this revision to Diff 211320. Anastasia added a comment. Use -cl-std=CLC++ spelling for test/CodeGenOpenCLCXX/addrspace-with-class.cl CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65102/new/ https://reviews.llvm.org/D65102 Files: include/clang/Driver/Options.td in

[PATCH] D65102: [OpenCL] Rename lang mode flag for C++ mode

Anastasia added a comment. In D65102#1597651 , @kpet wrote: > Hmm, maybe we need to make sure that one of the tests is using a C++ feature > and building with `CLC++`. This would have caught the mistake. That's right! The only test we had wasn't using a

r366823 - [clang][NFCI] Fix random typos

Author: jkorous Date: Tue Jul 23 09:54:11 2019 New Revision: 366823 URL: http://llvm.org/viewvc/llvm-project?rev=366823&view=rev Log: [clang][NFCI] Fix random typos Modified: cfe/trunk/include/clang/ASTMatchers/ASTMatchersInternal.h cfe/trunk/lib/Index/IndexSymbol.cpp cfe/trunk/lib/Se

[PATCH] D64644: Fixes an assertion failure while instantiation a template with an incomplete typo corrected type

lebedev.ri added reviewers: aaron.ballman, erichkeane. lebedev.ri added a comment. Test looks good, adding a few more potential reviewers.. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64644/new/ https://reviews.llvm.org/D64644 ___ cfe-com

[PATCH] D64838: [Attr] Support _attribute__ ((fallthrough))

Nathan-Huckleberry added a comment. In D64838#1593516 , @aaron.ballman wrote: > In D64838#1592520 , > @Nathan-Huckleberry wrote: > > > void foo() { > > __attribute__((address_space(0))) *x; > > *y; > >

[PATCH] D64666: [Sema] Enable -Wimplicit-float-conversion for integral to floating point precision loss

ziangwan added a comment. In D64666#1597627 , @jfb wrote: > In D64666#1597193 , @aaron.ballman > wrote: > > > In D64666#1596660 , @xbolva00 > > wrote: > > > > > I think we

[PATCH] D64883: Add new warning -Walloca for use of builtin alloca function

george.burgess.iv added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:2776 +def warn_alloca : Warning< + "use of builtin function %0">, + InGroup>, DefaultIgnore; aaron.ballman wrote: > george.burgess.iv wrote: > > aaron.ballman

r366828 - Improve clang-format-diff help output

Author: nico Date: Tue Jul 23 10:34:18 2019 New Revision: 366828 URL: http://llvm.org/viewvc/llvm-project?rev=366828&view=rev Log: Improve clang-format-diff help output The description in clang-format-diff.py is more useful than the one in `clang-format-diff -h`, so use the same description in bo

[PATCH] D64998: Improve clang-format-diff help output

This revision was automatically updated to reflect the committed changes. Closed by commit rL366828: Improve clang-format-diff help output (authored by nico, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/

[PATCH] D65043: [Format] Add C++20 standard to style options

modocache added a comment. Friendly ping! I'm wondering, from the clang-format maintainers' point of view, whether a diff like this and https://reviews.llvm.org/D65044 make sense to add? I do think that it is useful for users to specify whether they wish to use C++11 or C++20 constructs, but I'

[PATCH] D64666: [Sema] Enable -Wimplicit-float-conversion for integral to floating point precision loss

xbolva00 added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:3279 + "implicit conversion from %2 to %3 changes value from %0 to %1">, + InGroup, DefaultIgnore; + Drop ‘DefaultIgnore‘ here CHANGES SINCE LAST ACTION https://rev

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh added inline comments. Comment at: lib/CodeGen/CGDecl.cpp:2533 + CodeGenFunction *CGF) { + if (!LangOpts.OpenMP || LangOpts.OpenMPSimd || + (!LangOpts.EmitAllDecls && !D->isUsed())) ABataev wrote: > Why do we n

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh updated this revision to Diff 211333. lildmh marked 4 inline comments as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59474/new/ https://reviews.llvm.org/D59474 Files: include/clang/AST/GlobalDecl.h lib/AST/ASTContext.cpp lib/CodeGen/CGDecl.cpp lib/CodeGen/CGOpenMPR

r366831 - clang-format: Fix namespace end comments for namespaces with attributes and macros.

Author: nico Date: Tue Jul 23 10:49:45 2019 New Revision: 366831 URL: http://llvm.org/viewvc/llvm-project?rev=366831&view=rev Log: clang-format: Fix namespace end comments for namespaces with attributes and macros. Fixes PR39247. While here, also make C++20 `namespace A::inline B::inline C` nes

[PATCH] D65125: clang-format: Fix namespace end comments for namespaces with attributes and macros

This revision was automatically updated to reflect the committed changes. Closed by commit rL366831: clang-format: Fix namespace end comments for namespaces with attributes and… (authored by nico, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior

[PATCH] D64793: [Driver] Properly use values-X[ca].o, values-xpg[46].o on Solaris

ro marked an inline comment as done. ro added a comment. In D64793#1597550 , @jyknight wrote: > Is this really necessary? Users don't typically pass -std= to the driver for > linking anyways (what do you even pass if you've compiled both C and C++ > code

[PATCH] D64838: [Attr] Support _attribute__ ((fallthrough))

xbolva00 added a comment. >> they parse the attributes first then attempt to parse a declaration; if >> that fails, they fall back to parsing a statement Well, I don’t think this reparsing is ideal in terms of compile time either. If we really care about attributes on implicit ints: I don’t t

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

ABataev added inline comments. Comment at: lib/CodeGen/CGDecl.cpp:2533 + CodeGenFunction *CGF) { + if (!LangOpts.OpenMP || LangOpts.OpenMPSimd || + (!LangOpts.EmitAllDecls && !D->isUsed())) lildmh wrote: > ABataev wro

[PATCH] D64696: Adds a warning when an inline Doxygen comment has no argument

Mordante marked 2 inline comments as done. Mordante added a comment. All inline commands defined in include/clang/AST/CommentCommands.td require an argument. The escape commands, like \&, are handled in the switch starting at lib/AST/CommentLexer.cpp:366. They are stored as Text and not as an I

[PATCH] D64696: Adds a warning when an inline Doxygen comment has no argument

Mordante updated this revision to Diff 211340. Mordante added a comment. Addresses @gribozavr comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64696/new/ https://reviews.llvm.org/D64696 Files: clang/include/clang/Basic/DiagnosticCommentKinds.td clang/lib/AST/CommentParser.cp

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh marked 3 inline comments as done. lildmh added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:7116-7124 + /// Get the offset of the OMP_MAP_MEMBER_OF field. + static unsigned getFlagMemberOffset() { +unsigned Offset = 0; +for (uint64_t Remain = OMP_

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.h:2110 + /// Emit code for the user defined mapper construct. + void emitUserDefinedMapper(const OMPDeclareMapperDecl *D, + CodeGenFunction *CGF = nullptr) override; -

[PATCH] D65154: [clangd] Reformat use of cl::opt: use unqualified name and don't bin-pack attributes. NFC

kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. That LGTM, but i didn't have any bad feelings regarding the previous state. Since we tend to read those options in isolation and don't care about how they look like in general, but that mi

[PATCH] D64638: [CrossTU] Fix plist macro expansion if macro in other file.

NoQ added a comment. The preprocessor is defined in `Lex`, so i guess i'm curious about a more precise description of what's impossible to do without `Frontend`. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64638/new/ https://reviews.llvm.org/D64638

[PATCH] D64914: Implement P1771

erichkeane marked 2 inline comments as done. erichkeane added a comment. Ugg... well conversion functions are an interesting bit, as well as Type{}; with no constructors. It ends up being a function style cast to an init-list-expr, so its going to require a bit more work. Stay tuned :)

[PATCH] D64914: Implement P1771

aaron.ballman marked an inline comment as done. aaron.ballman added inline comments. Comment at: clang/lib/Sema/SemaDeclAttr.cpp:2835 + D->getFunctionType()->getReturnType()->isVoidType() && + !isa(D)) { S.Diag(AL.getLoc(), diag::warn_attribute_void_function_method

[PATCH] D64744: #pragma clang loop vectorize_predicate(enable|disable)

SjoerdMeijer added a comment. Apologies for the early ping! Bu I'm off next weeks, so it would be nice to get this in before that if there are no further comments. Tomorrow, I will upload another diff that builds on top D64916 , which enables code-generation with

[PATCH] D64539: [clang-doc] Add stylesheet to generated html docs

juliehockett accepted this revision. juliehockett added a comment. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64539/new/ https://reviews.llvm.org/D64539 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/

[PATCH] D64938: [clang-doc] Add option for user provided stylesheets

juliehockett added inline comments. Comment at: clang-tools-extra/unittests/clang-doc/HTMLGeneratorTest.cpp:26 + ClangDocContext CDCtx; + CDCtx.UserStylesheets = {"../share/clang/clang-doc-default-stylesheet.css"}; + return CDCtx; Can you write a test for the

[PATCH] D65107: [clang-doc] Fix html entities in rendered text

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

[PATCH] D64454: [clang-tidy] Adding static analyzer check to list of clang-tidy checks

Nathan-Huckleberry added a comment. In D64454#1587102 , @aaron.ballman wrote: > I think this looks reasonable to me, though I am still not certain if the > relative path in the python script will work with both the svn in-tree > directory layout as well

[PATCH] D65110: [NewPM] Run avx*-builtins.c tests under the new pass manager only

RKSimon added inline comments. Comment at: clang/test/CodeGen/avx512vlbw-builtins.c:904 + // CHECK: [[TMP:%.*]] = bitcast [[SRCTY:<.*>]] [[SEL]] to [[DSTTY:<.*>]] + // CHECK: [[SEL:%.*]] = bitcast [[DSTTY]] [[TMP]] to [[SRCTY]] // CHECK: select <16 x i1> %{{.*}}, <16 x i8> [

[PATCH] D65050: [SemaTemplate] Mark a function type as dependent when its parameter list contains pack expansion

cpplearner updated this revision to Diff 211358. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65050/new/ https://reviews.llvm.org/D65050 Files: clang/lib/AST/Type.cpp clang/test/SemaTemplate/alias-templates.cpp Index: clang/test/SemaTemplate/alias-templates.cpp ===

[PATCH] D65050: [SemaTemplate] Mark a function type as dependent when its parameter list contains pack expansion

cpplearner added a comment. In D65050#1596514 , @efriedma wrote: > Is this the only place where a compound type can contain a PackExpansionType? Yes AFAIK. No other place can contain a list or a pack expansion. The closest thing is dynamic exception spe

[PATCH] D64753: [CrossTU][NFCI] Refactor loadExternalAST function

gamesh411 updated this revision to Diff 211361. gamesh411 added a comment. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Refactor functionality into local classes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64753/new/ htt

[PATCH] D64938: [clang-doc] Add option for user provided stylesheets

DiegoAstiazaran updated this revision to Diff 211362. DiegoAstiazaran marked an inline comment as done. DiegoAstiazaran added a comment. Add a second CSS file to one of the tests in HTMLGeneratorTest.cpp CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64938/new/ https://reviews.llvm.org/D

[PATCH] D64678: [Sema] Fix -Wuninitialized for struct assignment from GNU C statement expression

nickdesaulniers accepted this revision. nickdesaulniers added a comment. This revision is now accepted and ready to land. Thanks for the patch and following up on the review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64678/new/ https://reviews.

[PATCH] D64753: [CrossTU][NFCI] Refactor loadExternalAST function

gamesh411 updated this revision to Diff 211363. gamesh411 added a comment. Too much autoformat fixed Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64753/new/ https://reviews.llvm.org/D64753 Files: clang/include/clang/CrossTU/CrossTranslationUnit

[PATCH] D64666: [Sema] Enable -Wimplicit-float-conversion for integral to floating point precision loss

ziangwan marked an inline comment as done. ziangwan added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:3279 + "implicit conversion from %2 to %3 changes value from %0 to %1">, + InGroup, DefaultIgnore; + xbolva00 wrote: > Drop ‘

[PATCH] D65169: [WIP] LLVM Optimization Remark for Attributes

wsmoses created this revision. Herald added subscribers: llvm-commits, cfe-commits, hiraditya, mgorny, mehdi_amini. Herald added projects: clang, LLVM. This is a *work in progress* patch illustrating how an optimization remark that prints out derived function/argument attributes could be impleme

  1   2   >