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

mprobst closed this revision. mprobst marked an inline comment as done. mprobst added a comment. This has landed as https://reviews.llvm.org/rGa324fcf1ae6 (not sure why this hasn't closed automatically). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D69868: Allow "callbr" to return non-void values

void added a comment. Friendly ping. :-) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69868/new/ https://reviews.llvm.org/D69868 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.l

[PATCH] D73462: [dwarf-5] Support DebugInfo for Defaulted parameters for C++ templates

awpandey updated this revision to Diff 241984. awpandey marked 2 inline comments as done. awpandey added a comment. Thanks, @aprantl, and @dblaikie. I have added two new test cases as per your suggestions. I have to update // dityperefs-3.8.ll// and //dityperefs-3.8.ll.bc// because they are no l

[PATCH] D73543: [clang] Add support for __builtin_memcpy_inline

gchatelet updated this revision to Diff 241988. gchatelet marked 3 inline comments as done. gchatelet added a comment. - Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73543/new/ https://reviews.llvm.org/D73543 Files: clang/docs/

[PATCH] D73543: [clang] Add support for __builtin_memcpy_inline

gchatelet added inline comments. Comment at: clang/docs/LanguageExtensions.rst:2252 + + * ``__builtin_memcpy_inline`` + efriedma wrote: > This is in the wrong section of the documentation. We could > constant-evaluate __builtin_memcpy_inline, I guess, but that

[PATCH] D72746: [clangd] Add a flag for implicit references in the Index

kbobyrev added inline comments. Comment at: clang-tools-extra/clangd/index/Ref.h:32 + // + // class Foo {}; + // ^ Foo declaration hokein wrote: > could you confirm with it? this looks like a `Foo` class definition. Good point, I thought it should be bo

[PATCH] D72746: [clangd] Add a flag for implicit references in the Index

kbobyrev updated this revision to Diff 241994. kbobyrev added a comment. Remove unnecessary include. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72746/new/ https://reviews.llvm.org/D72746 Files: clang-tools-extra/clangd/index/Ref.h clang-too

[PATCH] D72746: [clangd] Add a flag for implicit references in the Index

kbobyrev updated this revision to Diff 241993. kbobyrev marked 14 inline comments as done. kbobyrev added a comment. Address a number of comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72746/new/ https://reviews.llvm.org/D72746 Files: cl

[clang-tools-extra] b79cb54 - [clangd] Refactor TUScheduler options into a struct. NFC

Author: Sam McCall Date: 2020-02-03T11:20:48+01:00 New Revision: b79cb547121dbd9f4bd4eaaf05354829ab74ac8e URL: https://github.com/llvm/llvm-project/commit/b79cb547121dbd9f4bd4eaaf05354829ab74ac8e DIFF: https://github.com/llvm/llvm-project/commit/b79cb547121dbd9f4bd4eaaf05354829ab74ac8e.diff LO

[PATCH] D73775: [clang-tidy] Cover cases like (b && c && b) in the redundant expression check

alexeyr updated this revision to Diff 242000. alexeyr added a comment. Followed Eugene Zelenko's comments and fixed a false positive. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73775/new/ https://reviews.llvm.org/D73775 Files: clang-tools-ext

[clang-tools-extra] 6b15a3d - [clangd] TUScheduler::run() (i.e. workspace/symbol) counts towards concurrent threads

Author: Sam McCall Date: 2020-02-03T11:32:33+01:00 New Revision: 6b15a3d722a98c8c4f99ac45cae118e4af9244af URL: https://github.com/llvm/llvm-project/commit/6b15a3d722a98c8c4f99ac45cae118e4af9244af DIFF: https://github.com/llvm/llvm-project/commit/6b15a3d722a98c8c4f99ac45cae118e4af9244af.diff LO

[PATCH] D73775: [clang-tidy] Cover cases like (b && c && b) in the redundant expression check

alexeyr marked 11 inline comments as done. alexeyr added inline comments. Comment at: clang-tools-extra/clang-tidy/misc/RedundantExpressionCheck.cpp:363 + ASTContext &Context) { + const auto OpKind = getOp(TheExpr); + // if there are no nested operators of

[PATCH] D73869: [clang][AST] Add an AST matcher for deducedTemplateSpeializationType.

hokein created this revision. hokein added a reviewer: gribozavr2. Herald added a project: clang. misc-unused-using clang-tidy check needs this matcher to fix a false positive of C++17 deduced class template types. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73869 Files:

[PATCH] D73413: [clang-tidy] Add check to detect external definitions with no header declaration

tonyelewis added inline comments. Comment at: clang-tools-extra/clang-tidy/misc/MissingHeaderFileDeclarationCheck.cpp:75 + continue; +if (AnyHeader || File->NamePart.equals_lower(ThisFile->NamePart)) + return; // Found a good candidate for matching decl ---

[PATCH] D73869: [clang][AST] Add an AST matcher for deducedTemplateSpeializationType.

gribozavr2 accepted this revision. gribozavr2 added inline comments. This revision is now accepted and ready to land. Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:6043 +/// \endcode +/// \c deducedTemplateSpecializationType() matches the type in \c c. +extern const A

[PATCH] D72746: [clangd] Add a flag for implicit references in the Index

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 61851 tests passed, 6 failed and 780 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 failed:

[clang] bdbdf74 - [clang][AST] Add an AST matcher for deducedTemplateSpeializationType.

Author: Haojian Wu Date: 2020-02-03T12:11:44+01:00 New Revision: bdbdf748225525bb18cd0ffeb51c3e713f7b8e1b URL: https://github.com/llvm/llvm-project/commit/bdbdf748225525bb18cd0ffeb51c3e713f7b8e1b DIFF: https://github.com/llvm/llvm-project/commit/bdbdf748225525bb18cd0ffeb51c3e713f7b8e1b.diff LO

[PATCH] D72746: [clangd] Add a flag for implicit references in the Index

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 61851 tests passed, 6 failed and 780 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 failed:

[PATCH] D73869: [clang][AST] Add an AST matcher for deducedTemplateSpeializationType.

hokein updated this revision to Diff 242007. hokein marked an inline comment as done. hokein added a comment. address review comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73869/new/ https://reviews.llvm.org/D73869 Files: clang/docs/LibA

[PATCH] D73869: [clang][AST] Add an AST matcher for deducedTemplateSpeializationType.

This revision was automatically updated to reflect the committed changes. Closed by commit rGbdbdf7482255: [clang][AST] Add an AST matcher for deducedTemplateSpeializationType. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73869

[clang] cf7e98e - [ARM,MVE] Add intrinsics for vdupq.

Author: Simon Tatham Date: 2020-02-03T11:20:06Z New Revision: cf7e98e6f7805f4e2693a6dbbd12c10fe06fde70 URL: https://github.com/llvm/llvm-project/commit/cf7e98e6f7805f4e2693a6dbbd12c10fe06fde70 DIFF: https://github.com/llvm/llvm-project/commit/cf7e98e6f7805f4e2693a6dbbd12c10fe06fde70.diff LOG:

[PATCH] D73356: [ARM,MVE] Add intrinsics for vdupq.

This revision was automatically updated to reflect the committed changes. Closed by commit rGcf7e98e6f780: [ARM,MVE] Add intrinsics for vdupq. (authored by simon_tatham). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73356/new/ https://reviews.llvm.

[PATCH] D73357: [ARM,MVE] Add intrinsics for v[id]dupq and v[id]wdupq.

This revision was automatically updated to reflect the committed changes. Closed by commit rGf8d4afc49ad2: [ARM,MVE] Add intrinsics for v[id]dupq and v[id]wdupq. (authored by simon_tatham). Changed prior to commit: https://reviews.llvm.org/D73357?vs=240207&id=242011#toc Repository: rG LLVM G

[PATCH] D73786: [ARM,MVE] Fix vreinterpretq in big-endian mode.

This revision was automatically updated to reflect the committed changes. Closed by commit rG961530fdc9f7: [ARM,MVE] Fix vreinterpretq in big-endian mode. (authored by simon_tatham). Changed prior to commit: https://reviews.llvm.org/D73786?vs=241727&id=242012#toc Repository: rG LLVM Github M

[PATCH] D73869: [clang][AST] Add an AST matcher for deducedTemplateSpeializationType.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62408 tests passed, 0 failed and 839 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by running `gi

[PATCH] D73536: [analyser][taint] Remove taint from symbolic expressions if used in comparisons

Szelethus added a comment. In D73536#1845031 , @NoQ wrote: > > Describing value constraints in the taint config file is unfeasible. > > This is the only correct way to go, because, as you yourself point out, every > sink function (or other use of tainted

[PATCH] D73834: Update for Clang 10 release notes

hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. Looks good to me, go a head and commit it to the branch. > Can you cherry-pick c03349e40f21 > for > D73434

[PATCH] D73869: [clang][AST] Add an AST matcher for deducedTemplateSpeializationType.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62408 tests passed, 0 failed and 839 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by running `gi

[PATCH] D73873: [clangd] Mechanism to make update debounce responsive to rebuild speed.

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. Currently we delay AST rebuilds by 500ms after each edit, to wait for further edi

[PATCH] D73873: [clangd] Mechanism to make update debounce responsive to rebuild speed.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62406 tests passed, 0 failed and 839 were skipped. {icon times-circle color=red} clang-tidy: fail. clang-tidy found 0 errors and 1 warnings

[PATCH] D72304: [OpenMP][OMPIRBuilder] Add Directives (master and critical) to OMPBuilder.

fghanim added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72304/new/ https://reviews.llvm.org/D72304 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi

[PATCH] D73865: [CodeGenModule] Assume dso_local for -fpic -fno-semantic-interposition

MaskRay created this revision. MaskRay added reviewers: peter.smith, rnk, serge-sans-paille, sfertile. Herald added subscribers: cfe-commits, luismarques, apazos, sameer.abuasal, pzheng, s.egerton, lenary, Jim, jsji, jocewei, PkmX, arphaman, the_o, brucehoult, MartinMosbeck, rogfer01, atanasyan,

[PATCH] D73865: [CodeGenModule] Assume dso_local for -fpic -fno-semantic-interposition

peter.smith added a comment. If I've understood correctly this would make LLVM more aggressive for PIC relocation models, but perhaps more honest in an inter procedural optimisation context? The code changes look fine to me, I'm wondering if we've discussed this widely enough with the community

[PATCH] D73865: [CodeGenModule] Assume dso_local for -fpic -fno-semantic-interposition

merge_guards_bot added a comment. Herald added subscribers: wuzish, dexonsmith. {icon times-circle color=red} Unit tests: fail. 62284 tests passed, 121 failed and 839 were skipped. failed: Clang.CXX/modules-ts/basic/basic_link/p3.cppm failed: Clang.CXX/modules-ts/codegen-basics.cppm failed

[clang] 0ce5773 - [clang] Add a missing change of bdbdf748225525bb18cd0ffeb51c3e713f7b8e1b, fix the buildbot.

Author: Haojian Wu Date: 2020-02-03T13:43:26+01:00 New Revision: 0ce57731da9bf5967d0332b0b1d35790b65007d7 URL: https://github.com/llvm/llvm-project/commit/0ce57731da9bf5967d0332b0b1d35790b65007d7 DIFF: https://github.com/llvm/llvm-project/commit/0ce57731da9bf5967d0332b0b1d35790b65007d7.diff LO

Re: [clang] 1f3f8c3 - PR44721: Don't consider overloaded operators for built-in comparisons

Yes, go ahead. On Fri, Jan 31, 2020 at 2:19 AM Richard Smith wrote: > > Hi Hans, > > This is a pretty safe bugfix for a new feature in Clang 10. OK for branch? > > On Thu, 30 Jan 2020 at 17:17, Richard Smith via cfe-commits > wrote: >> >> >> Author: Richard Smith >> Date: 2020-01-30T17:16:50-08

[clang-tools-extra] 6423ae4 - Allow modernize-use-using to apply to enumerations as well.

Author: Karasev Nikita Date: 2020-02-03T07:54:38-05:00 New Revision: 6423ae417e17611c4ee529f5848e839e6d9cb795 URL: https://github.com/llvm/llvm-project/commit/6423ae417e17611c4ee529f5848e839e6d9cb795 DIFF: https://github.com/llvm/llvm-project/commit/6423ae417e17611c4ee529f5848e839e6d9cb795.diff

[PATCH] D73090: [clang-tidy] Fix PR#44528 'modernize-use-using and enums'

aaron.ballman closed this revision. aaron.ballman added a comment. In D73090#1854031 , @f00kat wrote: > Sorry for my english. Sometimes it's more difficult than bug fixing It's not a problem at all, that's why we help one another out spotting those thin

[PATCH] D73876: [clang-tidy] Fix a false positive about C++17 deduced class template types in unused-using-decl check.

hokein created this revision. hokein added a reviewer: gribozavr2. Herald added a subscriber: xazax.hun. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73876 Files: clang-tools-extra/clang-tidy/misc/UnusedUsingDeclsCheck.cpp clang-tools-extra/t

[PATCH] D73775: [clang-tidy] Cover cases like (b && c && b) in the redundant expression check

aaron.ballman added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/misc-redundant-expression.cpp:114 if (P.a[X++] != P.a[X++]) return 1; + if (X && X++ && X) return 1; What do you think about the following? ``` bool foo(int&); bool

[PATCH] D73856: [docu] Improve docu of misc-misplaced-const

aaron.ballman added a comment. > East const makes the problem more obvious. With west const people were > telling me the const is on the wrong side. I don't see how this makes the problem more obvious and it seems odd to me that we'd mix styles in code and prose. I prefer leaving this as a west

[PATCH] D72867: [clangd] Support renaming designated initializers

kbobyrev added inline comments. Comment at: clang-tools-extra/clangd/FindTarget.cpp:639 +void VisitDesignatedInitExpr(const DesignatedInitExpr *DIE) { + for (const DesignatedInitExpr::Designator &D : DIE->designators()) { +if (!D.isFieldDesignator())

[PATCH] D72867: [clangd] Support renaming designated initializers

kbobyrev updated this revision to Diff 242040. kbobyrev marked an inline comment as done. kbobyrev added a comment. Add tests for nested designated initializers. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72867/new/ https://reviews.llvm.org/D728

[PATCH] D72742: Don't assume promotable integers are zero/sign-extended already in x86-64 ABI.

emilio added a comment. Could anyone update me with how do they want me to proceed here? Is fixing the coercions enough to allow this to land? Do I need to make it target-specific? If so, which targets should keep the current behavior? Another slightly more backwards-compatible alternative (tho

[PATCH] D73876: [clang-tidy] Fix a false positive about C++17 deduced class template types in unused-using-decl check.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62415 tests passed, 0 failed and 839 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by running `gi

[PATCH] D72867: [clangd] Support renaming designated initializers

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62405 tests passed, 0 failed and 839 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[clang] eacca48 - [Concepts] Instantiate invented template type parameter type-constraint along with function parameters

Author: Saar Raz Date: 2020-02-03T15:47:32+02:00 New Revision: eacca4824463d8b96e2e1c9f8bbf886055218a16 URL: https://github.com/llvm/llvm-project/commit/eacca4824463d8b96e2e1c9f8bbf886055218a16 DIFF: https://github.com/llvm/llvm-project/commit/eacca4824463d8b96e2e1c9f8bbf886055218a16.diff LOG:

[PATCH] D73873: [clangd] Mechanism to make update debounce responsive to rebuild speed.

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. looks good, a few nits. Comment at: clang-tools-extra/clangd/TUScheduler.cpp:249 + llvm::SmallVector + RebuildTimes; /* GUARDED_BY(Mutex) */ /// File that ASTWorker

[clang] 8c16d8b - Revert "[Concepts] Instantiate invented template type parameter type-constraint along with function parameters"

Author: Saar Raz Date: 2020-02-03T16:04:48+02:00 New Revision: 8c16d8b235b997389a26d0ac7efe7840e208bb4d URL: https://github.com/llvm/llvm-project/commit/8c16d8b235b997389a26d0ac7efe7840e208bb4d DIFF: https://github.com/llvm/llvm-project/commit/8c16d8b235b997389a26d0ac7efe7840e208bb4d.diff LOG:

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

baloghadamsoftware added a comment. Please add tests for all the functions in the list. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72705/new/ https://reviews.llvm.org/D72705 ___ cfe-commits mailing

[clang] 1ca7403 - [OpenMP][OMPIRBuilder] Add Directives (master and critical) to OMPBuilder.

Author: Fady Ghanim Date: 2020-02-03T08:44:23-06:00 New Revision: 1ca740387b9bbdc142ac81c8bdd6370a8813e328 URL: https://github.com/llvm/llvm-project/commit/1ca740387b9bbdc142ac81c8bdd6370a8813e328 DIFF: https://github.com/llvm/llvm-project/commit/1ca740387b9bbdc142ac81c8bdd6370a8813e328.diff L

[PATCH] D73891: [RISCV] Support experimental/unratified extensions

simoncook created this revision. simoncook added reviewers: asb, lenary, PaoloS, s.egerton. Herald added subscribers: cfe-commits, luismarques, apazos, sameer.abuasal, pzheng, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, MaskRay, jr

[PATCH] D72304: [OpenMP][OMPIRBuilder] Add Directives (master and critical) to OMPBuilder.

This revision was automatically updated to reflect the committed changes. Closed by commit rG1ca740387b9b: [OpenMP][OMPIRBuilder] Add Directives (master and critical) to OMPBuilder. (authored by fghanim, committed by jdoerfert). Changed prior to commit: https://reviews.llvm.org/D72304?vs=238858

[clang] 84959ae - [Concepts] Instantiate invented template type parameter type-constraint along with function parameters

Author: Saar Raz Date: 2020-02-03T16:51:49+02:00 New Revision: 84959ae47f447fca9d56a9c61e8c46e993d0387a URL: https://github.com/llvm/llvm-project/commit/84959ae47f447fca9d56a9c61e8c46e993d0387a DIFF: https://github.com/llvm/llvm-project/commit/84959ae47f447fca9d56a9c61e8c46e993d0387a.diff LOG:

[PATCH] D73742: [Clang][Driver] After default -fintegrated-cc1, fix report_fatal_error no longer generates preprocessed source + reproducer.sh

hans added a comment. > Evidently I can cut the patch in smaller pieces, let me know. Yes, I think this would be good. There's a lot going on in this patch, and it would be good to separate the simple stuff from the tricky parts. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 9dcfc7c - Revert "[OpenMP][OMPIRBuilder] Add Directives (master and critical) to OMPBuilder."

Author: Johannes Doerfert Date: 2020-02-03T08:59:14-06:00 New Revision: 9dcfc7cd64abb301124cafaa95661b76a1fc5032 URL: https://github.com/llvm/llvm-project/commit/9dcfc7cd64abb301124cafaa95661b76a1fc5032 DIFF: https://github.com/llvm/llvm-project/commit/9dcfc7cd64abb301124cafaa95661b76a1fc5032.d

[PATCH] D73876: [clang-tidy] Fix a false positive about C++17 deduced class template types in unused-using-decl check.

gribozavr2 accepted this revision. gribozavr2 added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/test/clang-tidy/checkers/misc-unused-using-decls-cxx17.cpp:10 +}; +// Deduction hint (CTAD) +template Foo(T t) -> Foo; ---

[PATCH] D73742: [Clang][Driver] After default -fintegrated-cc1, fix report_fatal_error no longer generates preprocessed source + reproducer.sh

aganea added a comment. In D73742#1854773 , @hans wrote: > > Evidently I can cut the patch in smaller pieces, let me know. > > Yes, I think this would be good. There's a lot going on in this patch, and it > would be good to separate the simple stuff from

[PATCH] D72304: [OpenMP][OMPIRBuilder] Add Directives (master and critical) to OMPBuilder.

jdoerfert reopened this revision. jdoerfert added a comment. This revision is now accepted and ready to land. Reverted in 9dcfc7cd64abb301124cafaa95661b76a1fc5032 because of breakages and warnings: http://lab.llvm.org:8011/bui

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

balazske updated this revision to Diff 242064. balazske added a comment. - Bug fixes in the generic code. - Added new check types. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72705/new/ https://reviews.llvm.org/D72705 Files: clang/include/clan

[PATCH] D73775: [clang-tidy] Cover cases like (b && c && b) in the redundant expression check

alexeyr marked an inline comment as done. alexeyr added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/misc-redundant-expression.cpp:114 if (P.a[X++] != P.a[X++]) return 1; + if (X && X++ && X) return 1; aaron.ballman wrote: > What

[PATCH] D73742: [Clang][Driver] After default -fintegrated-cc1, fix report_fatal_error no longer generates preprocessed source + reproducer.sh

hans added a comment. I think so, but I need to look closer at the CrashRecoveryContext changes, and it would help to do that separately. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73742/new/ https://reviews.llvm.org/D73742 _

[PATCH] D73876: [clang-tidy] Fix a false positive about C++17 deduced class template types in unused-using-decl check.

hokein updated this revision to Diff 242065. hokein marked an inline comment as done. hokein added a comment. address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73876/new/ https://reviews.llvm.org/D73876 Files: clang-tools-extra/cla

[PATCH] D72857: [SYCL] Driver option to enable SYCL mode and select SYCL version

ABataev added inline comments. Comment at: clang/include/clang/Driver/Options.td:3412-3414 +def fsycl : Flag<["-"], "fsycl">, Group, + HelpText<"Enable SYCL kernels compilation for device">; +def sycl_std_EQ : Joined<["-"], "sycl-std=">, Group, Flags<[CC1Option]>, -

[PATCH] D72954: [clang-doc] Improving Markdown Output

Clayton reopened this revision. Clayton added a comment. This revision is now accepted and ready to land. Fixing build error CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72954/new/ https://reviews.llvm.org/D72954 ___ cfe-commits mailing li

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

balazske added a comment. In D72705#1854641 , @baloghadamsoftware wrote: > Please add tests for all the functions in the list. How to handle the combinatoric explosion in test code? It is too much code to add all functions to `system-header-simulator.h`

[PATCH] D72954: [clang-doc] Improving Markdown Output

Clayton updated this revision to Diff 242066. Clayton added a comment. build error fixed. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72954/new/ https://reviews.llvm.org/D72954 Files: clang-tools-extra/clang-doc/HTMLGenerator.cpp clang-tools-extra/clang-doc/MDGenerator.cpp clan

[clang-tools-extra] 7d4c23b - [clang-tidy] Fix a false positive about C++17 deduced class template types in unused-using-decl check.

Author: Haojian Wu Date: 2020-02-03T16:25:10+01:00 New Revision: 7d4c23b349f8d0575304182650850f2594935d7c URL: https://github.com/llvm/llvm-project/commit/7d4c23b349f8d0575304182650850f2594935d7c DIFF: https://github.com/llvm/llvm-project/commit/7d4c23b349f8d0575304182650850f2594935d7c.diff LO

[PATCH] D73876: [clang-tidy] Fix a false positive about C++17 deduced class template types in unused-using-decl check.

This revision was automatically updated to reflect the committed changes. Closed by commit rG7d4c23b349f8: [clang-tidy] Fix a false positive about C++17 deduced class template types in… (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D73876: [clang-tidy] Fix a false positive about C++17 deduced class template types in unused-using-decl check.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62415 tests passed, 0 failed and 839 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by running `gi

[PATCH] D71155: [analyzer] CERT: STR30-C

NoQ added a comment. Let's separate `CStringChecker` improvements into a separate patch and have a separate set of tests for it. Comment at: clang/lib/StaticAnalyzer/Checkers/CStringChecker.cpp:2097-2098 +void CStringChecker::evalCharPtrCommon(CheckerContext &C, +

[PATCH] D73891: [RISCV] Support experimental/unratified extensions

simoncook updated this revision to Diff 242080. simoncook added a comment. Don't put option in m_riscv_Features_Group, we don't want it being handled like a feature. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73891/new/ https://reviews.llvm.org/D73891 Files: clang/include/clang/D

[PATCH] D71433: [analyzer] CERT: POS34-C

NoQ added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/cert/PutenvWithAutoChecker.cpp:57-58 +bugreporter::trackExpressionValue(Report->getErrorNode(), ArgExpr, *Report); + } else if (const SymbolRef Sym = + ArgV.getAsSymbol()) { // It is a

[PATCH] D71553: [RISCV] Add Clang frontend support for Bitmanip extension

simoncook updated this revision to Diff 242082. simoncook added a comment. Rebase changes on top of experimental feature support (D73891 ) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71553/new/ https://reviews.ll

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

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

[PATCH] D73520: [analyzer] BugReporterVisitors: Refactor and documentation

NoQ added inline comments. Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitors.h:195 + + static const char *getTag() { return "FindLastStore"; } + Charusso wrote: > I have made every tag a small-string. This way it's harder to be

[PATCH] D73897: [analyzer] StdLibraryFunctionsChecker refactor: remove macros

martong created this revision. martong added a reviewer: NoQ. Herald added subscribers: cfe-commits, Charusso, gamesh411, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, xazax.hun, whisperity. Herald added a project: clang. Repository: rG LLVM G

[PATCH] D73898: [analyzer] StdLibraryFunctionsChecker: Add argument constraints

martong created this revision. martong added a reviewer: NoQ. Herald added subscribers: cfe-commits, Charusso, gamesh411, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, xazax.hun, whisperity. Herald added a project: clang. Repository: rG LLVM G

[PATCH] D70818: [Analyzer] Model STL Algoirthms to improve the iterator checkers

NoQ added a comment. Other than the packaging debate, i think this check looks good! Comment at: clang/include/clang/StaticAnalyzer/Checkers/Checkers.td:641 -} // end: "optin.cplusplus" - -let ParentPackage = CplusplusAlpha in { - -def ContainerModeling : Checker<"ContainerMo

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

ABataev added inline comments. Comment at: clang/docs/OpenMPSupport.rst:194 +--+--+--+---+ -| devi

[PATCH] D73720: [Analyzer] Use note tags to track container begin and and changes

NoQ added a comment. Also, yay, thanks for using the new API! Comment at: clang/lib/StaticAnalyzer/Checkers/ContainerModeling.cpp:41 const SVal &OldCont = UndefinedVal()) const; - void handleAssign(CheckerContext &C, const SVal &Cont) const; - void ha

[PATCH] D73720: [Analyzer] Use note tags to track container begin and and changes

NoQ added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/ContainerModeling.cpp:731 + } + return C.getNoteTag([Text, Name](BugReport &BR) -> std::string { + SmallString<256> Msg; You'll need to check whether the container is actually of inte

[PATCH] D71830: [OpenMP][Part 2] Use reusable OpenMP context/traits handling

kiranchandramohan added a comment. Had a quick look today. Will spend some more time tomorrow. Please see a few comments inline. Comment at: clang/include/clang/AST/OpenMPClause.h:6429 +/// and an ordered collection of properties. +struct OpenMPTraitInfo { + struct OpenMPTrait

[PATCH] D69781: [analyzer] Add test directory for scan-build

NoQ reopened this revision. NoQ added a comment. This revision is now accepted and ready to land. I'll try to re-land this. I heard there were a lot more issues than just honoring `CLANG_INSTALL_SCANBUILD=NO` (which i'll do my best to test before pushing), so i'll monitor the buildbots carefully

[PATCH] D71830: [OpenMP][Part 2] Use reusable OpenMP context/traits handling

jdoerfert marked 4 inline comments as done. jdoerfert added a comment. In D71830#1854989 , @kiranchandramohan wrote: > Had a quick look today. Will spend some more time tomorrow. > Please see a few comments inline. Thanks! I'll rebase asap to address y

[PATCH] D73900: [BPF] use base lvalue type for preserve_{struct,union}_access_index metadata

yonghong-song created this revision. yonghong-song added reviewers: ast, anakryiko. Herald added subscribers: cfe-commits, arphaman. Herald added a project: clang. Linux commit https://github.com/torvalds/linux/commit/1cf5b23988ea0086a252a5c8b005b075f1e9b030#diff-289313b9fec99c6f0acfea19d9cfd9

[PATCH] D72857: [SYCL] Driver option to enable SYCL mode and select SYCL version

bader marked an inline comment as done. bader added inline comments. Comment at: clang/include/clang/Driver/Options.td:3412-3414 +def fsycl : Flag<["-"], "fsycl">, Group, + HelpText<"Enable SYCL kernels compilation for device">; +def sycl_std_EQ : Joined<["-"], "sycl-std=">, Gro

[PATCH] D73903: [AArch64][SVE] Add remaining SVE2 intrinsics for widening DSP operations

kmclaughlin created this revision. kmclaughlin added reviewers: sdesmalen, dancgr, efriedma, c-rhodes. Herald added subscribers: psnobl, rkruppe, hiraditya, kristof.beyls, tschuett. Herald added a reviewer: rengolin. Herald added a project: LLVM. kmclaughlin added a parent revision: D73719: [AArch6

[PATCH] D73904: [clang] stop baremetal driver to append .a to lib

christof created this revision. christof added reviewers: peter.smith, jroelofs. Herald added a project: clang. Herald added a subscriber: cfe-commits. When the clang baremetal driver selects the rt.builtins static library it prefix with "-l" and appends ".a". The result is a nonsense option whic

[PATCH] D69591: Devirtualize a call on alloca without waiting for post inline cleanup and next DevirtSCCRepeatedPass iteration.

yamauchi updated this revision to Diff 242103. yamauchi added a comment. Fix typo. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69591/new/ https://reviews.llvm.org/D69591 Files: clang/test/CodeGenCXX/member-function-pointer-calls.cpp llvm/lib

[PATCH] D73898: [analyzer] StdLibraryFunctionsChecker: Add argument constraints

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62418 tests passed, 0 failed and 845 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D72857: [SYCL] Driver option to enable SYCL mode and select SYCL version

ABataev added inline comments. Comment at: clang/include/clang/Driver/Options.td:3412-3414 +def fsycl : Flag<["-"], "fsycl">, Group, + HelpText<"Enable SYCL kernels compilation for device">; +def sycl_std_EQ : Joined<["-"], "sycl-std=">, Group, Flags<[CC1Option]>, -

[PATCH] D73897: [analyzer] StdLibraryFunctionsChecker refactor: remove macros

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62099 tests passed, 5 failed and 784 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 failed:

[clang] a781521 - [OPENMP50]Codegen support for order(concurrent) clause.

Author: Alexey Bataev Date: 2020-02-03T12:27:33-05:00 New Revision: a781521867e9952e8d5856e10bf900b37f8ec4e8 URL: https://github.com/llvm/llvm-project/commit/a781521867e9952e8d5856e10bf900b37f8ec4e8 DIFF: https://github.com/llvm/llvm-project/commit/a781521867e9952e8d5856e10bf900b37f8ec4e8.diff

[clang] 6c3252e - [OPENMP][DOCS]Update status of conditional lastprivate, NFC.

Author: Alexey Bataev Date: 2020-02-03T12:43:40-05:00 New Revision: 6c3252e5211b0c6ce6eb4a1123d60bb74faec0be URL: https://github.com/llvm/llvm-project/commit/6c3252e5211b0c6ce6eb4a1123d60bb74faec0be DIFF: https://github.com/llvm/llvm-project/commit/6c3252e5211b0c6ce6eb4a1123d60bb74faec0be.diff

[PATCH] D73897: [analyzer] StdLibraryFunctionsChecker refactor: remove macros

NoQ added a comment. This looks fantastic, i didn't think of this originally. All hail type safety! Nits below. Comment at: clang/lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp:182 + +FunctionSummaryTy &Specification(ValueRangeSet VRS) { + Ranges.push_back(

[PATCH] D73906: [ARM] Make ARM::ArchExtKind use 64-bit underlying type, NFCI

miyuki created this revision. miyuki added reviewers: simon_tatham, eli.friedman, ostannard, dmgreen. Herald added subscribers: llvm-commits, cfe-commits, hiraditya, kristof.beyls. Herald added projects: clang, LLVM. This patch changes the underlying type of the ARM::ArchExtKind enumeration to uin

[PATCH] D73898: [analyzer] StdLibraryFunctionsChecker: Add argument constraints

NoQ added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp:411 + if (ExplodedNode *N = C.generateErrorNode(State)) { +// FIXME Add detailed diagnostic. +std::string Msg = "Function argument constraint is not satisf

[PATCH] D73900: [BPF] use base lvalue type for preserve_{struct,union}_access_index metadata

anakryiko accepted this revision. anakryiko added a comment. This revision is now accepted and ready to land. lgtm Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73900/new/ https://reviews.llvm.org/D73900 _

[PATCH] D73904: [clang] stop baremetal driver to append .a to lib

peter.smith added a comment. I think this is the right thing to do. According to https://sourceware.org/binutils/docs/ld/Options.html#Options Add the archive or object file specified by namespec to the list of files to link. This option may be used any number of times. If namespec is of the f

  1   2   >