[clang] c83602f - Recommit "[DWARF5][clang]: Added support for DebugInfo generation for auto return type for C++ member functions."

Author: Awanish Pandey Date: 2020-01-24T14:50:17+05:30 New Revision: c83602fdf51b2692e3bacb06bf861f20f74e987f URL: https://github.com/llvm/llvm-project/commit/c83602fdf51b2692e3bacb06bf861f20f74e987f DIFF:

[PATCH] D73320: [WebAssembly] Add reference types target feature

aheejin marked 2 inline comments as done. aheejin added inline comments. Comment at: llvm/test/CodeGen/WebAssembly/reference-types.ll:4 +; This currently only tests if the command line argument for reference types +; feature is accepted by llc. + sbc100 wrote: >

[PATCH] D73320: [WebAssembly] Add reference types target feature

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62151 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D73334: [Format] Fix 'auto x(T&&, T &&)->F' with PAS_Left.

This revision was automatically updated to reflect the committed changes. Closed by commit rGb3b68c0f802e: [Format] Fix auto x(T, T )-F with PAS_Left. (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73334/new/

[PATCH] D73320: [WebAssembly] Add reference types target feature

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62151 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D73098: [clang-tidy] readability-identifier-naming disregards parameters restrictions on main like functions

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62153 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D73320: [WebAssembly] Add reference types target feature

aheejin updated this revision to Diff 240124. aheejin marked an inline comment as done. aheejin added a comment. - Test target feature section Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73320/new/ https://reviews.llvm.org/D73320 Files:

[PATCH] D72222: [Driver][CodeGen] Add -fpatchable-function-entry=N[,0]

nsz added a comment. In D7#1837536 , @MaskRay wrote: > A -DLLVM_ENABLE_ASSERTIONS=on build is required to trigger the assertion > failure. My `make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- HOSTGCC=gcc > CC=~/llvm/ReleaseAssert/bin/clang

[PATCH] D73218: [clangd] Show background index status using LSP 3.15 work-done progress notifications

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon times-circle color=red} clang-tidy: fail. clang-tidy found 8 errors and 12 warnings . 12 of them are added as

[PATCH] D73199: [clangd] Errors in TestTU cause test failures unless suppressed with error-ok.

This revision was automatically updated to reflect the committed changes. Closed by commit rGd3260bf5b2f7: [clangd] Errors in TestTU cause test failures unless suppressed with error-ok. (authored by sammccall). Changed prior to commit: https://reviews.llvm.org/D73199?vs=239594=240129#toc

[PATCH] D73334: [Format] Fix 'auto x(T&&, T &&)->F' with PAS_Left.

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62136 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D73261: [dwarf5] Support DebugInfo for constexpr for C++ variables and functions

awpandey updated this revision to Diff 240122. awpandey marked 29 inline comments as done. awpandey added a comment. Hi @probinson, I have changed the patch according to your comments. Is it necessary to use DIFlags? I am willing to do that but generally, it is not welcomed because we have a

[PATCH] D73300: [clang-tidy] Add library for clang-tidy main function

gribozavr2 added a reviewer: gribozavr2. gribozavr2 accepted this revision. gribozavr2 added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clang-tidy/tool/ClangTidyMain.h:9 +/// +/// \file This file declares clang-tidy tool main

[PATCH] D73199: [clangd] Errors in TestTU cause test failures unless suppressed with error-ok.

sammccall marked an inline comment as done. sammccall added inline comments. Comment at: clang-tools-extra/clangd/unittests/TestTU.cpp:84 + if (llvm::none_of(Files, [](const auto ) { +return llvm::StringRef(KV.second).contains("error-ok"); + })) {

[PATCH] D67537: [clangd] Client-side support for inactive regions

hokein added a comment. > Do you mean naming the scope something like `clangd.meta.disable` to make it > clear it's a clangd extension? > > (In that case, we might as well make it something more descriptive, like > `clangd.preprocessor.inactive`. `meta` doesn't convey much information.) yes,

[clang] b3b68c0 - [Format] Fix 'auto x(T&&, T &&)->F' with PAS_Left.

Author: Sam McCall Date: 2020-01-24T11:03:46+01:00 New Revision: b3b68c0f802ebd9892f66bec0e7be96b3fb06e86 URL: https://github.com/llvm/llvm-project/commit/b3b68c0f802ebd9892f66bec0e7be96b3fb06e86 DIFF: https://github.com/llvm/llvm-project/commit/b3b68c0f802ebd9892f66bec0e7be96b3fb06e86.diff

[PATCH] D73320: [WebAssembly] Add reference types target feature

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62156 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D72705: [clang][checkers] Added new checker 'alpha.unix.ErrorReturn'.

balazske updated this revision to Diff 240111. balazske added a comment. Using check::DeadSymbols. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72705/new/ https://reviews.llvm.org/D72705 Files:

[PATCH] D72829: Implement -fsemantic-interposition

serge-sans-paille updated this revision to Diff 240110. serge-sans-paille added a comment. Formatting nits + rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72829/new/ https://reviews.llvm.org/D72829 Files:

[PATCH] D73334: [Format] Fix 'auto x(T&&, T &&)->F' with PAS_Left.

sammccall marked an inline comment as done. sammccall added inline comments. Comment at: clang/lib/Format/TokenAnnotator.cpp:1804 // "&&(" is quite unlikely to be two successive unary "&". -if (Tok.is(tok::ampamp) && NextToken && NextToken->is(tok::l_paren)) +if

[PATCH] D73334: [Format] Fix 'auto x(T&&, T &&)->F' with PAS_Left.

sammccall created this revision. sammccall added a reviewer: hokein. Herald added a project: clang. Herald added a subscriber: cfe-commits. sammccall marked an inline comment as done. sammccall added inline comments. Comment at: clang/lib/Format/TokenAnnotator.cpp:1804 //

[PATCH] D73218: [clangd] Show background index status using LSP 3.15 work-done progress notifications

sammccall updated this revision to Diff 240121. sammccall marked 3 inline comments as done. sammccall added a comment. address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73218/new/ https://reviews.llvm.org/D73218 Files:

[PATCH] D73218: [clangd] Show background index status using LSP 3.15 work-done progress notifications

sammccall added inline comments. Comment at: clang-tools-extra/clangd/Protocol.cpp:383 + +llvm::json::Value toJSON(const WorkDoneProgressBegin ) { + llvm::json::Object Result{ kadircet wrote: > why not have a single struct with that has a required `kind` field

[PATCH] D73320: [WebAssembly] Add reference types target feature

aheejin updated this revision to Diff 240127. aheejin added a comment. rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73320/new/ https://reviews.llvm.org/D73320 Files: clang/lib/Basic/Targets/WebAssembly.cpp

[PATCH] D73320: [WebAssembly] Add reference types target feature

aheejin updated this revision to Diff 240125. aheejin added a comment. - Remove a stray attribute number Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73320/new/ https://reviews.llvm.org/D73320 Files: clang/lib/Basic/Targets/WebAssembly.cpp

[PATCH] D73218: [clangd] Show background index status using LSP 3.15 work-done progress notifications

kadircet accepted this revision. kadircet added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/Protocol.cpp:383 + +llvm::json::Value toJSON(const WorkDoneProgressBegin ) { + llvm::json::Object Result{

[PATCH] D72829: Implement -fsemantic-interposition

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62155 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[clang] 326c39b - [clang][NFC] Remove redundant cast

Author: Raphael Isemann Date: 2020-01-24T11:07:33+01:00 New Revision: 326c39b569e385b35cc117a31a3b20919c799352 URL: https://github.com/llvm/llvm-project/commit/326c39b569e385b35cc117a31a3b20919c799352 DIFF:

[PATCH] D73097: [AArch64][SVE] Add intrinsics for FFR manipulation

kmclaughlin added a comment. Thanks for your input on this @efriedma, I will submit this patch for now and we will revisit the modelling of the FFR as also discussed in D71698 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73099: [Alignment][NFC] Deprecate Align::None()

This revision was automatically updated to reflect the committed changes. Closed by commit rG805c157e8aee: [Alignment][NFC] Deprecate Align::None() (authored by gchatelet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73099/new/

[PATCH] D73203: [clang-tidy] Prevent a remove only fixit causing a conflict when enclosed by another fixit

njames93 updated this revision to Diff 240163. njames93 added a comment. - Small edge case tweak Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73203/new/ https://reviews.llvm.org/D73203 Files:

[PATCH] D73346: [clangd] Rename DiagnosticsConsumer -> ClangdServer::Callbacks, and make it optional

sammccall created this revision. sammccall added a reviewer: hokein. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous, MaskRay, javed.absar, ilya-biryukov. Herald added a project: clang. This reflects its current function better and avoids confusion with

[PATCH] D73335: clang-format: [JS] options for arrow functions.

krasimir added a comment. > FYI I've started a conversation on whether SLS_All should be the default > Google style. But the option I think is useful in either case. If that's the case, consider adding the option to the style in a follow-up patch: GoogleStyle.AllowShortLambdasOnASingleLine

[PATCH] D73322: [WebAssembly] Update bleeding-edge CPU features

aheejin updated this revision to Diff 240132. aheejin added a comment. Herald added subscribers: llvm-commits, jfb, hiraditya. Herald added a project: LLVM. rebase + test fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73322/new/

[PATCH] D73322: [WebAssembly] Update bleeding-edge CPU features

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62155 tests passed, 6 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D72705: [clang][checkers] Added new checker 'alpha.unix.ErrorReturn'.

balazske added a comment. I am still unsure about how this checker works if the function to check is "modeled" or evaluated by another checker. Then the function may have already a constrained value at the PostCall event (for example if a `malloc` fails) and the basic idea of this checker that

[clang] ba545c8 - [Sema] Try 2: Attempt to perform call-size-specific `__attribute__((alloc_align(param_idx)))` validation

Author: Roman Lebedev Date: 2020-01-24T14:42:45+03:00 New Revision: ba545c814bc19c3b1ec9ce88a7f13575e0e4ce1d URL: https://github.com/llvm/llvm-project/commit/ba545c814bc19c3b1ec9ce88a7f13575e0e4ce1d DIFF: https://github.com/llvm/llvm-project/commit/ba545c814bc19c3b1ec9ce88a7f13575e0e4ce1d.diff

[PATCH] D73346: [clangd] Rename DiagnosticsConsumer -> ClangdServer::Callbacks, and make it optional

hokein accepted this revision. hokein added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/ClangdServer.h:76 +/// May be called concurrently for separate files, not for a single file. +virtual void

[clang] 0a002f6 - [Sema] Try 2: Sanity-check alignment requested via `__attribute__((assume_aligned(imm)))`

Author: Roman Lebedev Date: 2020-01-24T16:56:39+03:00 New Revision: 0a002f679be4a9d363e6f21b707493c18158930b URL: https://github.com/llvm/llvm-project/commit/0a002f679be4a9d363e6f21b707493c18158930b DIFF: https://github.com/llvm/llvm-project/commit/0a002f679be4a9d363e6f21b707493c18158930b.diff

[PATCH] D72811: [WIP][OPENMP5.0] allow lvalue for motion clause

ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7711-7713 +isa(Next->getAssociatedExpression()) || +isa(Next->getAssociatedExpression()) || +isa(Next->getAssociatedExpression())) &&

[clang] 1d0972f - [Sema] Introduce MaximumAlignment value, to be used instead of magical constants

Author: Roman Lebedev Date: 2020-01-24T17:49:17+03:00 New Revision: 1d0972ff5eab544579c400ee674d5c9c60391ee1 URL: https://github.com/llvm/llvm-project/commit/1d0972ff5eab544579c400ee674d5c9c60391ee1 DIFF: https://github.com/llvm/llvm-project/commit/1d0972ff5eab544579c400ee674d5c9c60391ee1.diff

[PATCH] D73346: [clangd] Rename DiagnosticsConsumer -> ClangdServer::Callbacks, and make it optional

This revision was automatically updated to reflect the committed changes. sammccall marked an inline comment as done. Closed by commit rG6ef1ccecf7ae: [clangd] Rename DiagnosticsConsumer - ClangdServer::Callbacks, and make it… (authored by sammccall). Changed prior to commit:

[clang-tools-extra] 6ef1cce - [clangd] Rename DiagnosticsConsumer -> ClangdServer::Callbacks, and make it optional

Author: Sam McCall Date: 2020-01-24T15:59:32+01:00 New Revision: 6ef1ccecf7aea94fb38fc8d891f0097829b9228e URL: https://github.com/llvm/llvm-project/commit/6ef1ccecf7aea94fb38fc8d891f0097829b9228e DIFF: https://github.com/llvm/llvm-project/commit/6ef1ccecf7aea94fb38fc8d891f0097829b9228e.diff

[PATCH] D73261: [dwarf5] Support DebugInfo for constexpr for C++ variables and functions

probinson added a comment. In D73261#1838552 , @djtodoro wrote: > > Is it necessary to use DIFlags? I am willing to do that but generally, it > > is not welcomed because we have a limited number of DIFlags and most of > > them are currently in use. > >

[PATCH] D73346: [clangd] Rename DiagnosticsConsumer -> ClangdServer::Callbacks, and make it optional

sammccall marked 4 inline comments as done. sammccall added inline comments. Comment at: clang-tools-extra/clangd/ClangdServer.h:76 +/// May be called concurrently for separate files, not for a single file. +virtual void onFileUpdated(PathRef File, const TUStatus ){}; +

[PATCH] D73261: [dwarf5] Support DebugInfo for constexpr for C++ variables and functions

djtodoro added a comment. I agree, we must pay attention and address the bitcode compatibility. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73261/new/ https://reviews.llvm.org/D73261 ___ cfe-commits mailing list

[PATCH] D73336: [clangd][Hover] Change arrow in return type back to →

This revision was automatically updated to reflect the committed changes. Closed by commit rG1b996faa1575: [clangd][Hover] Change arrow in return type back to → (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73336/new/

[PATCH] D73344: [clangd][Hover] Handle uninstantiated templates

kadircet created this revision. kadircet added reviewers: hokein, sammccall. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Fixes https://github.com/clangd/clangd/issues/263 Repository: rG LLVM Github Monorepo

[PATCH] D73346: [clangd] Rename DiagnosticsConsumer -> ClangdServer::Callbacks, and make it optional

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62158 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D72467: Remove "mask" operand from shufflevector.

spatel accepted this revision. spatel added a comment. This revision is now accepted and ready to land. LGTM, but should get a 2nd opinion since I'm not familiar with some of the parts. Also, please update the related LangRef text for shufflevector in or alongside this patch. Could add some

[PATCH] D73350: Small StreamChecker refactoring (NFC).

balazske created this revision. Herald added subscribers: cfe-commits, gamesh411, Szelethus, dkrupp. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73350 Files: clang/lib/StaticAnalyzer/Checkers/StreamChecker.cpp Index:

[PATCH] D72467: Remove "mask" operand from shufflevector.

sdesmalen added a comment. Thanks @efriedma for working on this. The overall approach seems good to me! Mostly added some nits, with the exception of one question on what to do with a shufflevector with fixed-width mask and scalable source vectors. Comment at:

[PATCH] D73320: [WebAssembly] Add reference types target feature

aheejin updated this revision to Diff 240130. aheejin added a comment. Herald added a subscriber: jfb. Fix test comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73320/new/ https://reviews.llvm.org/D73320 Files:

[PATCH] D73336: [clangd][Hover] Change arrow in return type back to →

kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Currently  is used in hover response to represent return types, but it is not widely available.

[PATCH] D73007: [Sema] Avoid Wrange-loop-analysis false positives

aaron.ballman added a comment. In D73007#1837621 , @rtrieu wrote: > I'm in favor of splitting the warning into subgroups, then deciding which > ones should be in -Wall. We've done this with other warnings such as the > conversion warnings and

[PATCH] D73344: [clangd][Hover] Handle uninstantiated templates

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts :

[PATCH] D73335: clang-format: [JS] options for arrow functions.

mprobst created this revision. mprobst added a reviewer: krasimir. Herald added a project: clang. clang-format currently always wraps the body of non-empty arrow functions: const x = () => { z(); }; This change implements support for the `AllowShortLambdasOnASingleLine` style options,

[PATCH] D73320: [WebAssembly] Add reference types target feature

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62156 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D73336: [clangd][Hover] Change arrow in return type back to →

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62133 tests passed, 5 failed and 807 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D72810: [LifetimeAnalysis] Add support for lifetime annotations on functions

gribozavr2 marked an inline comment as done. gribozavr2 added inline comments. Comment at: clang/include/clang/AST/LifetimeAttr.h:22 + +/// This represents an abstract memory location that is used in the lifetime +/// contract representation. gribozavr2 wrote: >

[PATCH] D71823: Support Swift calling convention for WebAssembly targets

kateinoigakukun added a comment. Could you merge this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71823/new/ https://reviews.llvm.org/D71823 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D72820: [FPEnv] Add pragma FP_CONTRACT support under strict FP.

kpn added inline comments. Comment at: clang/lib/CodeGen/CGExprScalar.cpp:3381 + "constrained mode"); +FMulAdd = Builder.CreateCall( +CGF.CGM.getIntrinsic(llvm::Intrinsic::experimental_constrained_fmuladd, craig.topper wrote: > Doesn't this

[clang-tools-extra] 7d20e80 - [clangd] Show background index status using LSP 3.15 work-done progress notifications

Author: Sam McCall Date: 2020-01-24T12:21:08+01:00 New Revision: 7d20e80225b3e4f3f5a8c57929061bef55f3d855 URL: https://github.com/llvm/llvm-project/commit/7d20e80225b3e4f3f5a8c57929061bef55f3d855 DIFF: https://github.com/llvm/llvm-project/commit/7d20e80225b3e4f3f5a8c57929061bef55f3d855.diff

[PATCH] D73218: [clangd] Show background index status using LSP 3.15 work-done progress notifications

This revision was automatically updated to reflect the committed changes. Closed by commit rG7d20e80225b3: [clangd] Show background index status using LSP 3.15 work-done progress… (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73099: [Alignment][NFC] Deprecate Align::None()

gchatelet updated this revision to Diff 240148. gchatelet added a comment. - rebasing Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73099/new/ https://reviews.llvm.org/D73099 Files: clang/lib/CodeGen/CGBuiltin.cpp

[PATCH] D72996: [Sema] Attempt to perform call-size-specific `__attribute__((alloc_align(param_idx)))` validation

This revision was automatically updated to reflect the committed changes. Closed by commit rGba545c814bc1: [Sema] Try 2: Attempt to perform call-size-specific `__attribute__((alloc_align… (authored by lebedev.ri). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73261: [dwarf5] Support DebugInfo for constexpr for C++ variables and functions

djtodoro added a comment. > Is it necessary to use DIFlags? I am willing to do that but generally, it is > not welcomed because we have a limited number of DIFlags and most of them are > currently in use. Are there any flags within `DIFlags` that could be applied only to functions? If so, we

[PATCH] D73344: [clangd][Hover] Handle uninstantiated templates

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Can you file a new bug or modify the upstream one to audit existing uses of getTemplateInstantiationPattern() (and maybe related functions)? getSpecializedTemplate() is used with no

[PATCH] D73203: [clang-tidy] Prevent a remove only fixit causing a conflict when enclosed by another fixit

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62126 tests passed, 5 failed and 807 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D73347: [AArch64][SVE] Add SVE2 intrinsics for pairwise arithmetic

kmclaughlin created this revision. kmclaughlin added reviewers: dancgr, efriedma, sdesmalen, c-rhodes. Herald added subscribers: psnobl, rkruppe, hiraditya, kristof.beyls, tschuett. Herald added a project: LLVM. Implements the following intrinsics: - addp - smaxp, sminp, umaxp & uminp - sadalp &

[PATCH] D73346: [clangd] Rename DiagnosticsConsumer -> ClangdServer::Callbacks, and make it optional

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62158 tests passed, 5 failed and 811 were skipped. failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp

[PATCH] D73335: clang-format: [JS] options for arrow functions.

mprobst added a comment. FYI I've started a conversation on whether `SLS_All` should be the default Google style. But the option I think is useful in either case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73335/new/

[PATCH] D73097: [AArch64][SVE] Add intrinsics for FFR manipulation

This revision was automatically updated to reflect the committed changes. Closed by commit rG4c4861b577cb: [AArch64][SVE] Add intrinsics for FFR manipulation (authored by kmclaughlin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73097/new/

[PATCH] D72906: [X86] Improve X86 cmpps/cmppd/cmpss/cmpsd intrinsics with strictfp

uweigand added a comment. In D72906#1837905 , @craig.topper wrote: > In D72906#1826849 , @uweigand wrote: > > > In D72906#1826122 , @craig.topper > > wrote: > > > > > In

[PATCH] D73346: [clangd] Rename DiagnosticsConsumer -> ClangdServer::Callbacks, and make it optional

sammccall updated this revision to Diff 240165. sammccall added a comment. Document new interface and threading model of existing functions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73346/new/ https://reviews.llvm.org/D73346 Files:

[PATCH] D72994: [Sema] Sanity-check alignment requested via `__attribute__((assume_aligned(imm)))`

This revision was automatically updated to reflect the committed changes. Closed by commit rG0a002f679be4: [Sema] Try 2: Sanity-check alignment requested via `__attribute__… (authored by lebedev.ri). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D72998: [IR] Attribute/AttrBuilder: use Value::MaximumAlignment magic constant

lebedev.ri added a comment. In D72998#1838811 , @gchatelet wrote: > Actually there's an issue with the code. It doesn't compile in shared_library > mode. > >ld.lld: error: undefined symbol: clang::Sema::MaximumAlignment > >>> referenced by

[PATCH] D72998: [IR] Attribute/AttrBuilder: use Value::MaximumAlignment magic constant

gchatelet added a comment. In D72998#1838834 , @lebedev.ri wrote: > Is that still an issue after a8c3608a27a82cf1c66f33b96a06423fe0e708fc > ? >

[PATCH] D73354: clang-format: insert trailing commas into containers.

krasimir added a comment. General comment: I think this is useful and could also be useful for other languages too (some folks have asked for this for some C++ code). As this is a non-whitespace only change, it's a bit more risky than whitespace-only changes. I think we need to establish a

[PATCH] D73360: [OpenCL] Restrict address space conversions in nested pointers

Anastasia created this revision. Anastasia added reviewers: rjmccall, jeroen.dobbelaere, bevinh. Herald added subscribers: ebevhan, yaxunl. This patch is fixing the issue reported in: http://lists.llvm.org/pipermail/cfe-dev/2020-January/064273.html and in the bug:

[PATCH] D73354: clang-format: insert trailing commas into containers.

sammccall added a comment. For rolling this out, I think the best path is: - check in the option, but don't turn it on with any styles yet - test it by taking a large codebase, formatting it (normal options), format it (with comma insertion), look at the diffs (internally google3diff can do

[PATCH] D73369: [clangd] Simplify "preferred" vs "definition" logic a bit in XRefs AST code.

sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Now Preferred is always the canonical (first) decl, Definition is always the def if available. In

[PATCH] D72630: [clang-tidy] Ignore implicit casts in modernize-use-default-member-init

hans added a comment. Ah, you're right, it's there. The reason for my comment was http://lists.llvm.org/pipermail/llvm-branch-commits/2020-January/013555.html but it seems it was also successfully pushed to master. Sorry for the noise. Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D71092: [VFS] More consistent support for Windows

amccarth added a comment. cc: +thakis, who expressed interest in seeing a fix for the text exempted on Windows. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71092/new/ https://reviews.llvm.org/D71092 ___ cfe-commits mailing list

[PATCH] D73300: [clang-tidy] Add library for clang-tidy main function

DmitryPolukhin updated this revision to Diff 240241. DmitryPolukhin added a comment. Added #include "ClangTidyMain.h" in ClangTidyMain.cpp Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73300/new/ https://reviews.llvm.org/D73300 Files:

[PATCH] D73029: Extend ExprTraversal class with acending traversal

aaron.ballman added a comment. Test cases? Comment at: clang/lib/AST/ASTContext.cpp:1066 +if (Node.getNodeKind().hasPointerIdentity()) { + auto ParentList = + getDynNodeFromMap(Node.getMemoizationData(), PointerParents); Please spell the type

[PATCH] D72811: [WIP][OPENMP5.0] allow lvalue for motion clause

cchen marked an inline comment as done. cchen added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:15235 + bool VisitUnaryOperator(UnaryOperator *UO) { +DerefCnt++; +CurComponents.emplace_back(UO, nullptr); cchen wrote: > ABataev wrote: > >

[PATCH] D72811: [WIP][OPENMP5.0] allow lvalue for motion clause

cchen marked an inline comment as done. cchen added a comment. In D72811#1837392 , @jdoerfert wrote: > Thanks for working on this! While you are at it, `*this` is probably one of > the most important ones to test and support. Can anyone tell me how to

[PATCH] D73300: [clang-tidy] Add library for clang-tidy main function

This revision was automatically updated to reflect the committed changes. Closed by commit rG3f8b100e94b5: [clang-tidy] Add library for clang-tidy main function (authored by DmitryPolukhin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73300/new/

[PATCH] D73029: Extend ExprTraversal class with acending traversal

rsmith added a comment. We're working on moving the parent map out of `ASTContext` and into something specific to tooling; please don't add more dependencies onto it in the AST library. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73029/new/

[PATCH] D73028: Extract ExprTraversal class from Expr

rsmith added a comment. In D73028#1839494 , @steveire wrote: > In D73028#1839383 , @rsmith wrote: > > > > > > The follow-up is here: https://reviews.llvm.org/D73029 . > > I have the need to change the ascending

[PATCH] D72811: [WIP][OPENMP5.0] allow lvalue for motion clause

ABataev added a comment. In D72811#1839538 , @cchen wrote: > In D72811#1837392 , @jdoerfert wrote: > > > Thanks for working on this! While you are at it, `*this` is probably one of > > the most important ones to

[PATCH] D72811: [WIP][OPENMP5.0] allow lvalue for motion clause

cchen marked an inline comment as done. cchen added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:15235 + bool VisitUnaryOperator(UnaryOperator *UO) { +DerefCnt++; +CurComponents.emplace_back(UO, nullptr); ABataev wrote: > cchen wrote: > >

[PATCH] D72675: [Clang][Driver] Fix -ffast-math/-ffp-contract interaction

andrew.w.kaylor added a comment. In D72675#1839492 , @wristow wrote: > 1. Should we enable FMA "by default" at (for example) '-O2'? We recently introduced a new option "-ffp-model=[precise|fast|strict], which is supposed to serve as an umbrella option

[clang] 764f408 - [WebAssembly] Add reference types target feature

Author: Heejin Ahn Date: 2020-01-24T14:26:27-08:00 New Revision: 764f4089e89e4693b7bb8f1ee18080703ce760dd URL: https://github.com/llvm/llvm-project/commit/764f4089e89e4693b7bb8f1ee18080703ce760dd DIFF: https://github.com/llvm/llvm-project/commit/764f4089e89e4693b7bb8f1ee18080703ce760dd.diff

[PATCH] D73320: [WebAssembly] Add reference types target feature

This revision was automatically updated to reflect the committed changes. Closed by commit rG764f4089e89e: [WebAssembly] Add reference types target feature (authored by aheejin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73320/new/

[PATCH] D70926: [clang-format] Add option for not breaking line before ObjC params

ghvg1313 updated this revision to Diff 240260. ghvg1313 marked an inline comment as done. ghvg1313 added a comment. - rebase master Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70926/new/ https://reviews.llvm.org/D70926 Files:

[PATCH] D73028: Extract ExprTraversal class from Expr

rsmith added a comment. > An addition to the API will be concerned with ascending through the AST in > different traversal modes. Ascending through the AST is not possibly by design. (For example, we share AST nodes between template instantiations, and statement nodes don't contain parent

[PATCH] D71600: PowerPC 32-bit - forces 8 byte lock/lock_free decisions at compiled time

dim added inline comments. Comment at: clang/lib/AST/ExprConstant.cpp:11180 +// Avoid emiting call for runtime decision on PowerPC 32-bit +// The lock free possibilities on this platform are covered by the lines `s/emiting/emitting/` Repository:

[PATCH] D73380: [clang] Annotating C++'s `operator new` with more attributes

lebedev.ri created this revision. lebedev.ri added reviewers: rsandifo, erichkeane, rjmccall, jdoerfert, eugenis. lebedev.ri added a project: LLVM. Herald added subscribers: atanasyan, jrtc27. Herald added a project: clang. lebedev.ri added a parent revision: D73020: [Sema] Perform call checking

  1   2   3   >