[clang] [libclang/python] Bump minimum compatibility to Python 3.6 (PR #77228)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Craig Hesling (linux4life798) Changes Remove Python 2 support and clean up code that conditions based on version. Issue #76664. --- Full diff: https://github.com/llvm/llvm-project/pull/77228.diff 2 Files Affected: - (modified) clang/bi

[clang] [libclang/python] Bump minimum compatibility to Python 3.6 (PR #77228)

https://github.com/linux4life798 created https://github.com/llvm/llvm-project/pull/77228 Remove Python 2 support and clean up code that conditions based on version. Issue #76664. >From 1ce6508ee1052c46c44a0f8f5dfd424df83ce209 Mon Sep 17 00:00:00 2001 From: Craig Hesling Date: Sat, 2 Dec 2023

[clang] [llvm] [SpecialCaseList] Use glob by default (PR #74809)

MaskRay wrote: > ah it's because we something like > > ``` > [cfi-unrelated-cast|cfi-derived-cast] > > src:*third_party/vulkan_memory_allocator/include/vk_mem_alloc.h > ``` > > it seems like the new system doesn't match > `[cfi-unrelated-cast|cfi-derived-cast]` The glob mode can use the sect

[clang-tools-extra] Fix #75686: add iter_swap and iter_move to the matched name (PR #76117)

https://github.com/Da-Viper updated https://github.com/llvm/llvm-project/pull/76117 >From 97eeda4684804229d9faad19ea7b7888dcd91786 Mon Sep 17 00:00:00 2001 From: Ezike Ebuka Date: Thu, 21 Dec 2023 02:30:34 + Subject: [PATCH 1/6] Fix #75686: add iter_swap and iter_move to the matched name

[mlir] [llvm] [clang-tools-extra] [clang] [mlir][spirv] Support alias/restrict function argument decorations (PR #76353)

antiagainst wrote: > @kuhar @antiagainst Thanks for your comment and support to land this PR! If > you have any issues or TODO tasks in SPIR-V dialect, I would be happy if you > could share it with me. :) Thanks for your further interest! Please feel free to take a look at issues with the `ml

[llvm] [clang] [libcxx] Reapply "[libc++][streams] P1759R6: Native handles and file streams" (PR #77190)

https://github.com/zeroomega commented: I test run it with C++26 on Windows, it clears the test failures happened in the original PR. https://github.com/llvm/llvm-project/pull/77190 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists

[mlir] [clang-tools-extra] [llvm] [clang] [mlir][spirv] Support alias/restrict function argument decorations (PR #76353)

sott0n wrote: @kuhar @antiagainst Thanks for your comment and support to land this PR! If you have any issues or TODO tasks in SPIR-V dialect, I would be happy if you could share it with me. https://github.com/llvm/llvm-project/pull/76353 ___ cfe-com

[clang] [lldb] [clang-tools-extra] [compiler-rt] [mlir] [llvm] Fix Multiple Build Errors on different platforms (PR #77216)

https://github.com/JDevlieghere requested changes to this pull request. https://github.com/llvm/llvm-project/pull/77216 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[lldb] [compiler-rt] [llvm] [mlir] [clang-tools-extra] [clang] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -108,13 +108,9 @@ else() linux/HostInfoLinux.cpp linux/LibcGlue.cpp linux/Support.cpp + android/HostInfoAndroid.cpp + android/LibcGlue.cpp ) -if (CMAKE_SYSTEM_NAME MATCHES "Android") JDevlieghere wrote: While Android m

[clang] [mlir] [llvm] [compiler-rt] [lldb] [clang-tools-extra] Fix Multiple Build Errors on different platforms (PR #77216)

https://github.com/JDevlieghere edited https://github.com/llvm/llvm-project/pull/77216 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [mlir] [llvm] [compiler-rt] [lldb] [clang-tools-extra] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -108,13 +108,9 @@ else() linux/HostInfoLinux.cpp linux/LibcGlue.cpp linux/Support.cpp + android/HostInfoAndroid.cpp + android/LibcGlue.cpp ) -if (CMAKE_SYSTEM_NAME MATCHES "Android") trcrsired wrote: I will try to find

[clang] [mlir] [llvm] [compiler-rt] [lldb] [clang-tools-extra] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -108,13 +108,9 @@ else() linux/HostInfoLinux.cpp linux/LibcGlue.cpp linux/Support.cpp + android/HostInfoAndroid.cpp + android/LibcGlue.cpp ) -if (CMAKE_SYSTEM_NAME MATCHES "Android") trcrsired wrote: That is probably b

[clang] [mlir] [libc] [flang] [libcxx] [llvm] [compiler-rt] [clang-tools-extra] [mlir][Linalg] Support dynamic tiles in `lower_pack` transform (PR #76003)

srcarroll wrote: I made a PR to extend UB cases in verifier https://github.com/llvm/llvm-project/pull/77217 https://github.com/llvm/llvm-project/pull/76003 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -108,13 +108,9 @@ else() linux/HostInfoLinux.cpp linux/LibcGlue.cpp linux/Support.cpp + android/HostInfoAndroid.cpp + android/LibcGlue.cpp ) -if (CMAKE_SYSTEM_NAME MATCHES "Android") asl wrote: Not quite. It's distant

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -108,13 +108,9 @@ else() linux/HostInfoLinux.cpp linux/LibcGlue.cpp linux/Support.cpp + android/HostInfoAndroid.cpp + android/LibcGlue.cpp ) -if (CMAKE_SYSTEM_NAME MATCHES "Android") trcrsired wrote: > Well, so this is

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -46,64 +46,64 @@ class Triple { enum ArchType { UnknownArch, -arm,// ARM (little endian): arm, armv.*, xscale trcrsired wrote: i will fix it later. but first pass CI with other things first to ensure it does not break anything. http

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -108,13 +108,9 @@ else() linux/HostInfoLinux.cpp linux/LibcGlue.cpp linux/Support.cpp + android/HostInfoAndroid.cpp + android/LibcGlue.cpp ) -if (CMAKE_SYSTEM_NAME MATCHES "Android") asl wrote: Well, so this is not a c

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

https://github.com/asl edited https://github.com/llvm/llvm-project/pull/77216 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -46,64 +46,64 @@ class Triple { enum ArchType { UnknownArch, -arm,// ARM (little endian): arm, armv.*, xscale asl wrote: You can ignore these. https://github.com/llvm/llvm-project/pull/77216 __

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -46,64 +46,64 @@ class Triple { enum ArchType { UnknownArch, -arm,// ARM (little endian): arm, armv.*, xscale trcrsired wrote: without doing code formatting the CI won't pass. https://github.com/llvm/llvm-project/pull/77216 _

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -108,13 +108,9 @@ else() linux/HostInfoLinux.cpp linux/LibcGlue.cpp linux/Support.cpp + android/HostInfoAndroid.cpp + android/LibcGlue.cpp ) -if (CMAKE_SYSTEM_NAME MATCHES "Android") trcrsired wrote: I use "Linux" to d

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -108,13 +108,9 @@ else() linux/HostInfoLinux.cpp linux/LibcGlue.cpp linux/Support.cpp + android/HostInfoAndroid.cpp + android/LibcGlue.cpp ) -if (CMAKE_SYSTEM_NAME MATCHES "Android") asl wrote: What is wrong with andro

[llvm] [clang] [mlir] [compiler-rt] [clang-tools-extra] [lldb] Fix Multiple Build Errors on different platforms (PR #77216)

@@ -46,64 +46,64 @@ class Triple { enum ArchType { UnknownArch, -arm,// ARM (little endian): arm, armv.*, xscale asl wrote: Please do not do such large code reformatting. It makes impossible to review them https://github.com/llvm/llvm-

[clang] [lldb] [llvm] [mlir] [compiler-rt] [clang-tools-extra] Fix Multiple Build Errors on different platforms (PR #77216)

https://github.com/trcrsired updated https://github.com/llvm/llvm-project/pull/77216 >From a1156a564c798192cb7608e28ce9b9ed846a8b1c Mon Sep 17 00:00:00 2001 From: trcrsired Date: Sat, 6 Jan 2024 20:21:06 -0500 Subject: [PATCH 1/2] Fix Multiple Build Errors on different platforms Particularly f

[clang] [lldb] [llvm] [mlir] [compiler-rt] [clang-tools-extra] Fix Multiple Build Errors on different platforms (PR #77216)

https://github.com/trcrsired updated https://github.com/llvm/llvm-project/pull/77216 >From a1156a564c798192cb7608e28ce9b9ed846a8b1c Mon Sep 17 00:00:00 2001 From: trcrsired Date: Sat, 6 Jan 2024 20:21:06 -0500 Subject: [PATCH 1/2] Fix Multiple Build Errors on different platforms Particularly f

[clang] [lldb] [llvm] [mlir] [compiler-rt] [clang-tools-extra] Fix Multiple Build Errors on different platforms (PR #77216)

llvmbot wrote: @llvm/pr-subscribers-mlir-execution-engine Author: cqwrteur (trcrsired) Changes Particularly for canadian compilation --- Patch is 23.86 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/77216.diff 8 Files Affected: - (modified)

[mlir] [clang] [llvm] [clang-tools-extra] [lldb] [compiler-rt] Fix Multiple Build Errors on different platforms (PR #77216)

llvmbot wrote: @llvm/pr-subscribers-clang Author: cqwrteur (trcrsired) Changes Particularly for canadian compilation --- Patch is 23.86 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/77216.diff 8 Files Affected: - (modified) clang-tools-ext

[clang] [llvm] [lldb] [clang-tools-extra] [mlir] [compiler-rt] Fix Multiple Build Errors on different platforms (PR #77216)

llvmbot wrote: @llvm/pr-subscribers-lldb Author: cqwrteur (trcrsired) Changes Particularly for canadian compilation --- Patch is 23.86 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/77216.diff 8 Files Affected: - (modified) clang-tools-extr

[mlir] [clang] [llvm] [clang-tools-extra] [lldb] [compiler-rt] Fix Multiple Build Errors on different platforms (PR #77216)

github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it i

[mlir] [clang] [llvm] [clang-tools-extra] [lldb] [compiler-rt] Fix Multiple Build Errors on different platforms (PR #77216)

https://github.com/trcrsired created https://github.com/llvm/llvm-project/pull/77216 Particularly for canadian compilation >From a1156a564c798192cb7608e28ce9b9ed846a8b1c Mon Sep 17 00:00:00 2001 From: trcrsired Date: Sat, 6 Jan 2024 20:21:06 -0500 Subject: [PATCH] Fix Multiple Build Errors on

[clang] [clang][coverage] fixing "if constexpr" and "if consteval" coverage report (PR #77214)

https://github.com/hanickadot edited https://github.com/llvm/llvm-project/pull/77214 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][coverage] fixing "if constexpr" and "if consteval" coverage report (PR #77214)

https://github.com/hanickadot updated https://github.com/llvm/llvm-project/pull/77214 From 8f1370aae4db2048c35516a85fb72c742557942b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hana=20Dusi=CC=81kova=CC=81?= Date: Sun, 7 Jan 2024 00:13:08 +0100 Subject: [PATCH] [coverage] fix incorrect coverage repo

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

https://github.com/XDeme updated https://github.com/llvm/llvm-project/pull/77045 >From d9cbbe48b96d27bff3fc926b60d039ed05f00489 Mon Sep 17 00:00:00 2001 From: XDeme Date: Fri, 5 Jan 2024 01:23:16 -0300 Subject: [PATCH 1/5] [clang-format] Fix crash involving array designators and dangling comma

[clang] [clang][coverage] fixing "if constexpr" and "if consteval" coverage report (PR #77214)

hanickadot wrote: https://github.com/llvm/llvm-project/assets/6557263/47ff77c0-9101-44cf-b2d5-ffea514bfc0c";> (notice wrong coverage "if constexpr" for positive one, and completely missing for negative one, also notice "if consteval" marking always the same branch as uncovered) https://gith

[clang] [clang][coverage] fixing "if constexpr" and "if consteval" coverage report (PR #77214)

https://github.com/hanickadot edited https://github.com/llvm/llvm-project/pull/77214 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][coverage] fixing "if constexpr" and "if consteval" coverage report (PR #77214)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Hana Dusíková (hanickadot) Changes It was a while since I noticed coverage report is broken for "if constexpr" and "if consteval" (as shown on first picture). [clang] [clang][coverage] fixing "if constexpr" and "if consteval" coverage report (PR #77214)

2024-01-06 Thread Hana Dusíková via cfe-commits

https://github.com/hanickadot created https://github.com/llvm/llvm-project/pull/77214 It was a while since I noticed coverage report is broken for "if constexpr" and "if consteval" (as shown on first picture). https://github.com/llvm/llvm-project/assets/6557263/dbdbc8a6-ad16-44da-882d-8e229ee69

[clang] [llvm] [clang-tools-extra] [compiler-rt] fix wrong links in documentation (PR #76502)

2024-01-06 Thread Hana Dusíková via cfe-commits

https://github.com/hanickadot updated https://github.com/llvm/llvm-project/pull/76502 From 93e8cc7c4310f8b91be4d72f4862f5ed7c06c33b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hana=20Dusi=CC=81kova=CC=81?= Date: Thu, 28 Dec 2023 14:30:35 +0100 Subject: [PATCH] [llvm] fix wrong links in documentati

[clang] [clang-format] Handle templated elaborated type specifier in function… (PR #77013)

2024-01-06 Thread Björn Schäpers via cfe-commits

https://github.com/HazardyKnusperkeks approved this pull request. https://github.com/llvm/llvm-project/pull/77013 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread Björn Schäpers via cfe-commits

https://github.com/HazardyKnusperkeks approved this pull request. https://github.com/llvm/llvm-project/pull/77045 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread Björn Schäpers via cfe-commits

@@ -1444,16 +1444,26 @@ WhitespaceManager::CellDescriptions WhitespaceManager::getCells(unsigned Start, } else if (C.Tok->is(tok::comma)) { if (!Cells.empty()) Cells.back().EndIndex = i; -if (C.Tok->getNextNonComment()->isNot(tok::r_brace)) // d

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread Björn Schäpers via cfe-commits

@@ -1444,16 +1444,26 @@ WhitespaceManager::CellDescriptions WhitespaceManager::getCells(unsigned Start, } else if (C.Tok->is(tok::comma)) { if (!Cells.empty()) Cells.back().EndIndex = i; -if (C.Tok->getNextNonComment()->isNot(tok::r_brace)) // d

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread Björn Schäpers via cfe-commits

https://github.com/HazardyKnusperkeks edited https://github.com/llvm/llvm-project/pull/77045 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] fix wrong links in documentation (PR #76502)

2024-01-06 Thread via cfe-commits

llvmbot wrote: @llvm/pr-subscribers-clang-codegen Author: Hana Dusíková (hanickadot) Changes Hi, I noticed some links in documentation is pointing to bad url (there were two wrong paths) I also noticed file `llvm/docs/tutorial/BuildingAJIT5.htm` is missing and it's pointed from `Buildin

[clang] fix wrong links in documentation (PR #76502)

2024-01-06 Thread via cfe-commits

llvmbot wrote: @llvm/pr-subscribers-clang Author: Hana Dusíková (hanickadot) Changes Hi, I noticed some links in documentation is pointing to bad url (there were two wrong paths) I also noticed file `llvm/docs/tutorial/BuildingAJIT5.htm` is missing and it's pointed from `BuildingAJIT4.h

[clang] fix wrong links in documentation (PR #76502)

2024-01-06 Thread Hana Dusíková via cfe-commits

https://github.com/hanickadot updated https://github.com/llvm/llvm-project/pull/76502 From b72e9299c957bd73744605e12d52b229fc289d5b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hana=20Dusi=CC=81kova=CC=81?= Date: Sun, 7 Jan 2024 00:13:08 +0100 Subject: [PATCH] [coverage] fix incorrect coverage repo

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread via cfe-commits

https://github.com/XDeme edited https://github.com/llvm/llvm-project/pull/77045 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][NFC] Refactor Builtins.def to be a tablegen file (PR #68324)

2024-01-06 Thread Nikolas Klauser via cfe-commits

@@ -0,0 +1,333 @@ +//=- ClangDiagnosticsEmitter.cpp - Generate Clang diagnostics tables -*- C++ -*- +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Ap

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread via cfe-commits

https://github.com/XDeme edited https://github.com/llvm/llvm-project/pull/77045 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Handle templated elaborated type specifier in function… (PR #77013)

2024-01-06 Thread via cfe-commits

https://github.com/XDeme updated https://github.com/llvm/llvm-project/pull/77013 >From 45d01cbc7ec958518b1739daa9e9b0dc35c2d194 Mon Sep 17 00:00:00 2001 From: XDeme Date: Thu, 4 Jan 2024 19:04:21 -0300 Subject: [PATCH 1/4] [clang-format] Handle templated elaborated type specifier in function re

[clang] [libc] [lldb] [mlir] [llvm] [NFC][ObjectSizeOffset] Use classes instead of std::pair (PR #76882)

2024-01-06 Thread via cfe-commits

dyung wrote: Hi @bwendling, your change to MemoryBuiltins.h is hitting an error in the version of Visual Studio 2019 that we use internally to build: ``` C:\j\w\779ddbee\p\llvm\include\llvm/Analysis/MemoryBuiltins.h(217): error C2990: 'llvm::SizeOffsetType': non-class template has already been

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread via cfe-commits

@@ -1444,16 +1444,22 @@ WhitespaceManager::CellDescriptions WhitespaceManager::getCells(unsigned Start, } else if (C.Tok->is(tok::comma)) { if (!Cells.empty()) Cells.back().EndIndex = i; -if (C.Tok->getNextNonComment()->isNot(tok::r_brace)) // d

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread via cfe-commits

https://github.com/XDeme updated https://github.com/llvm/llvm-project/pull/77045 >From d9cbbe48b96d27bff3fc926b60d039ed05f00489 Mon Sep 17 00:00:00 2001 From: XDeme Date: Fri, 5 Jan 2024 01:23:16 -0300 Subject: [PATCH 1/4] [clang-format] Fix crash involving array designators and dangling comma

[llvm] [clang] [SpecialCaseList] Use glob by default (PR #74809)

2024-01-06 Thread Arthur Eubanks via cfe-commits

aeubanks wrote: ah it's because we something like ``` [cfi-unrelated-cast|cfi-derived-cast] src:*third_party/vulkan_memory_allocator/include/vk_mem_alloc.h ``` it seems like the new system doesn't match `[cfi-unrelated-cast|cfi-derived-cast]` https://github.com/llvm/llvm-project/pull/74809 _

[clang] [clang-format] Break after string literals with trailing line breaks (PR #76795)

2024-01-06 Thread Owen Pan via cfe-commits

@@ -2499,6 +2499,15 @@ TEST_F(TokenAnnotatorTest, BraceKind) { EXPECT_BRACE_KIND(Tokens[6], BK_Block); } +TEST_F(TokenAnnotatorTest, StreamOperator) { + auto Tokens = annotate("\"foo\\n\" << aux << \"foo\\n\" << \"foo\";"); + ASSERT_EQ(Tokens.size(), 9u) << Tokens; + EXPE

[llvm] [clang] [SpecialCaseList] Use glob by default (PR #74809)

2024-01-06 Thread Arthur Eubanks via cfe-commits

aeubanks wrote: the file name is `vk_mem_alloc.h` so that shouldn't be the issue https://github.com/llvm/llvm-project/pull/74809 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libunwind] [libc] [clang-tools-extra] [libcxx] [llvm] [lldb] [compiler-rt] [clang] [flang] [libc++] Implement ranges::iota (PR #68494)

2024-01-06 Thread James E T Smith via cfe-commits

jamesETsmith wrote: @cjdb, thanks for taking a look! I've addressed some of your comments already and will try to wrap up the rest early next week. I've got plenty of time to work on this over the next couple of weeks so I shouldn't have a problem wrapping this up before the end of January ht

[mlir] [clang-tools-extra] [clang] [llvm] [mlir][spirv] Support alias/restrict function argument decorations (PR #76353)

2024-01-06 Thread Lei Zhang via cfe-commits

https://github.com/antiagainst closed https://github.com/llvm/llvm-project/pull/76353 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[mlir] [clang-tools-extra] [clang] [llvm] [mlir][spirv] Support alias/restrict function argument decorations (PR #76353)

2024-01-06 Thread Lei Zhang via cfe-commits

https://github.com/antiagainst edited https://github.com/llvm/llvm-project/pull/76353 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [flang] [libcxx] [libc] [clang-tools-extra] [compiler-rt] [lldb] [libunwind] [libc++] Implement ranges::iota (PR #68494)

2024-01-06 Thread James E T Smith via cfe-commits

https://github.com/jamesETsmith edited https://github.com/llvm/llvm-project/pull/68494 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [flang] [libcxx] [libc] [clang-tools-extra] [compiler-rt] [lldb] [libunwind] [libc++] Implement ranges::iota (PR #68494)

2024-01-06 Thread James E T Smith via cfe-commits

@@ -13,7 +13,7 @@ // Range algorithms should return `std::ranges::dangling` when given a dangling range. jamesETsmith wrote: This should be resolved with [be7faa6](https://github.com/llvm/llvm-project/pull/68494/commits/be7faa6fb11c09822a9a855c7cf1aecab4f4c7

[clang] [Clang][Sema] Print more static_assert exprs (PR #74852)

2024-01-06 Thread Craig Topper via cfe-commits

https://github.com/topperc updated https://github.com/llvm/llvm-project/pull/74852 >From f281d34a51f662c934f158e4770774b0dc3588a2 Mon Sep 17 00:00:00 2001 From: Seth Pellegrino Date: Thu, 7 Dec 2023 08:45:51 -0800 Subject: [PATCH 1/4] [Clang][Sema] Print more static_assert exprs This change in

[mlir] [clang-tools-extra] [clang] [llvm] [mlir][spirv] Support function argument decorations for ptr in the PhysicalStorageBuffer (PR #76353)

2024-01-06 Thread Lei Zhang via cfe-commits

antiagainst wrote: Thanks a lot for the contribution and bearing with me for the nitpicking. :) To avoid burden you further, I rebased and revised some comment/error message slightly. :) I'll land once bots are happy. https://github.com/llvm/llvm-project/pull/76353

[mlir] [clang-tools-extra] [clang] [llvm] [mlir][spirv] Support function argument decorations for ptr in the PhysicalStorageBuffer (PR #76353)

2024-01-06 Thread Lei Zhang via cfe-commits

https://github.com/antiagainst updated https://github.com/llvm/llvm-project/pull/76353 >From ebd9634057e9417905d7fcd27bac829c5d0889e0 Mon Sep 17 00:00:00 2001 From: Kohei Yamaguchi Date: Fri, 22 Dec 2023 17:22:25 + Subject: [PATCH 01/10] [mlir][spirv] Support function argument decorations f

[clang] [clang] Reword apologetic Clang diagnostic messages (PR #76310)

2024-01-06 Thread Pavel Gueorguiev via cfe-commits

https://github.com/pav-code updated https://github.com/llvm/llvm-project/pull/76310 >From 82fe20f1ccc2e9129282c71bf5bdfd6cfd4fadf3 Mon Sep 17 00:00:00 2001 From: Pavel Gueorguiev Date: Sat, 23 Dec 2023 14:19:50 -0500 Subject: [PATCH 1/2] [clang] Reword apologetic Clang diagnostic messages Fixe

[compiler-rt] [clang-tools-extra] [flang] [lldb] [llvm] [libcxx] [libunwind] [libc] [clang] [libc++] Implement ranges::iota (PR #68494)

2024-01-06 Thread James E T Smith via cfe-commits

@@ -0,0 +1,71 @@ +// -*- C++ -*- +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-

[compiler-rt] [clang-tools-extra] [flang] [lldb] [llvm] [libcxx] [libunwind] [libc] [clang] [libc++] Implement ranges::iota (PR #68494)

2024-01-06 Thread James E T Smith via cfe-commits

@@ -0,0 +1,71 @@ +// -*- C++ -*- +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-

[compiler-rt] [clang-tools-extra] [flang] [lldb] [llvm] [libcxx] [libunwind] [libc] [clang] [libc++] Implement ranges::iota (PR #68494)

2024-01-06 Thread James E T Smith via cfe-commits

@@ -0,0 +1,71 @@ +// -*- C++ -*- +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-

[libcxx] [clang] [llvm] Reapply "[libc++][streams] P1759R6: Native handles and file streams" (PR #77190)

2024-01-06 Thread via cfe-commits

llvmbot wrote: @llvm/pr-subscribers-libcxx Author: Hristo Hristov (H-G-Hristov) Changes Fixes build on Windows in C++26 mode. Reverted in: https://github.com/llvm/llvm-project/commit/40c07b559aa6ab4bac074c943967d3207bc07ae0 Original PR: https://github.com/llvm/llvm-project/pull/76632 --

[libcxx] [clang] [llvm] Reapply "[libc++][streams] P1759R6: Native handles and file streams" (PR #77190)

2024-01-06 Thread Hristo Hristov via cfe-commits

https://github.com/Zingam ready_for_review https://github.com/llvm/llvm-project/pull/77190 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[compiler-rt] [mlir] [clang-tools-extra] [flang] [lldb] [llvm] [libcxx] [libunwind] [lld] [libc] [clang] [libcxxabi] [clang] static operators should evaluate object argument (PR #68485)

2024-01-06 Thread Tianlan Zhou via cfe-commits

https://github.com/SuperSodaSea updated https://github.com/llvm/llvm-project/pull/68485 >From 03276260c48d9cafb2a0d80825156e77cdf02eba Mon Sep 17 00:00:00 2001 From: SuperSodaSea Date: Sat, 7 Oct 2023 21:05:17 +0800 Subject: [PATCH 01/11] [clang] static operators should evaluate object argumen

[flang] [clang] [libunwind] [compiler-rt] [libc] [libcxx] [clang-tools-extra] [lldb] [llvm] [libc++] Implement ranges::iota (PR #68494)

2024-01-06 Thread James E T Smith via cfe-commits

@@ -1149,9 +1171,11 @@ struct Proxy { // Calling swap(Proxy{}, Proxy{}) would fail (pass prvalues) // Compare operators are defined for the convenience of the tests - friend constexpr bool operator==(const Proxy&, const Proxy&) -requires (std::equality_comparable && !

[clang-tools-extra] [clang-tidy] Add support for in-class initializers in readability-redundant-member-init (PR #77206)

2024-01-06 Thread via cfe-commits

llvmbot wrote: @llvm/pr-subscribers-clang-tidy Author: Piotr Zegar (PiotrZSL) Changes Support detecting redundant in-class initializers. Moved from https://reviews.llvm.org/D157262 Fixes: #62525 --- Full diff: https://github.com/llvm/llvm-project/pull/77206.diff 4 Files Affected: - (

[clang-tools-extra] [clang-tidy] Add support for in-class initializers in readability-redundant-member-init (PR #77206)

2024-01-06 Thread Piotr Zegar via cfe-commits

https://github.com/PiotrZSL edited https://github.com/llvm/llvm-project/pull/77206 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Add support for in-class initializers in readability-redundant-member-init (PR #77206)

2024-01-06 Thread Piotr Zegar via cfe-commits

https://github.com/PiotrZSL created https://github.com/llvm/llvm-project/pull/77206 Support detecting redundant in-class initializers. Moved from https://reviews.llvm.org/D157262 Fixes: #62525 >From 66e4026ff568fbd805ddd777596102381e4e0066 Mon Sep 17 00:00:00 2001 From: Piotr Zegar Date: Sat

[clang-tools-extra] [clang-tidy] Fix false-positives in misc-static-assert caused by non-constexpr variables (PR #77203)

2024-01-06 Thread via cfe-commits

llvmbot wrote: @llvm/pr-subscribers-clang-tidy Author: Piotr Zegar (PiotrZSL) Changes Ignore false-positives when referring to non-constexpr variables in non-unevaluated context (like decltype, sizeof, ...). Moved from https://reviews.llvm.org/D158657 Fixes: #24066 --- Full diff: https

[clang-tools-extra] [clang-tidy] Fix false-positives in misc-static-assert caused by non-constexpr variables (PR #77203)

2024-01-06 Thread Piotr Zegar via cfe-commits

https://github.com/PiotrZSL created https://github.com/llvm/llvm-project/pull/77203 Ignore false-positives when referring to non-constexpr variables in non-unevaluated context (like decltype, sizeof, ...). Moved from https://reviews.llvm.org/D158657 Fixes: #24066 >From 66900e4518b439cb9190e2

[clang] [clang] move -Wcast-function-type under -Wextra (PR #77178)

2024-01-06 Thread Vlad Serebrennikov via cfe-commits

https://github.com/Endilll edited https://github.com/llvm/llvm-project/pull/77178 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] move -Wcast-function-type under -Wextra (PR #77178)

2024-01-06 Thread Vlad Serebrennikov via cfe-commits

@@ -0,0 +1,42 @@ +// RUN: %clang_cc1 %s -fsyntax-only -Wextra -verify Endilll wrote: As written, tests checks only default language mode for C (which is `gnu99` AFAIK). Maybe we want to check other modes as well? https://github.com/llvm/llvm-project/pull/77178

[clang] [clang] move -Wcast-function-type under -Wextra (PR #77178)

2024-01-06 Thread Vlad Serebrennikov via cfe-commits

@@ -0,0 +1,42 @@ +// RUN: %clang_cc1 %s -fsyntax-only -Wextra -verify + + +int t(int array[static 12]); +int u(int i); +const int v(int i); /* expected-warning {{'const' type qualifier on return type has no effec}} */ +int x(long); + +typedef int (f1)(long); +typedef int (f2)(voi

[clang] [clang] move -Wcast-function-type under -Wextra (PR #77178)

2024-01-06 Thread Vlad Serebrennikov via cfe-commits

https://github.com/Endilll commented: I think this PR moves things in the right direction, but I'm not well-versed in C enough to approve. > This should have a release note and I think this is a potentially breaking > change since folks using Wextra may get this diagnostic now. I agree that r

[clang] [clang] move -Wcast-function-type under -Wextra (PR #77178)

2024-01-06 Thread Vlad Serebrennikov via cfe-commits

@@ -0,0 +1,42 @@ +// RUN: %clang_cc1 %s -fsyntax-only -Wextra -verify + + +int t(int array[static 12]); +int u(int i); +const int v(int i); /* expected-warning {{'const' type qualifier on return type has no effec}} */ Endilll wrote: ```suggestion const int v(int

[clang] [clang] move -Wcast-function-type under -Wextra (PR #77178)

2024-01-06 Thread Vlad Serebrennikov via cfe-commits

https://github.com/Endilll edited https://github.com/llvm/llvm-project/pull/77178 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [compiler-rt] [flang] [libc] [libunwind] [llvm] [lld] [libcxxabi] [mlir] [libcxx] [lldb] [clang-tools-extra] [clang] static operators should evaluate object argument (PR #68485)

2024-01-06 Thread Tianlan Zhou via cfe-commits

https://github.com/SuperSodaSea updated https://github.com/llvm/llvm-project/pull/68485 >From 03276260c48d9cafb2a0d80825156e77cdf02eba Mon Sep 17 00:00:00 2001 From: SuperSodaSea Date: Sat, 7 Oct 2023 21:05:17 +0800 Subject: [PATCH 01/10] [clang] static operators should evaluate object argumen

[clang] [Clang][Sema] Print more static_assert exprs (PR #74852)

2024-01-06 Thread via cfe-commits

@@ -0,0 +1,117 @@ +// RUN: %clang_cc1 -std=c++2a -verify %s + +struct A { + int a, b[3], c; + bool operator==(const A&) const = default; +}; + +constexpr auto a0 = A{0, 0, 3, 4, 5}; + +// expected-note@+1 {{evaluates to '{0, {0, 3, 4}, 5} == {1, {2, 3, 4}, 5}'}} +static_assert(a

[clang] [clang-format] Break after string literals with trailing line breaks (PR #76795)

2024-01-06 Thread Björn Schäpers via cfe-commits

@@ -2499,6 +2499,15 @@ TEST_F(TokenAnnotatorTest, BraceKind) { EXPECT_BRACE_KIND(Tokens[6], BK_Block); } +TEST_F(TokenAnnotatorTest, StreamOperator) { + auto Tokens = annotate("\"foo\\n\" << aux << \"foo\\n\" << \"foo\";"); + ASSERT_EQ(Tokens.size(), 9u) << Tokens; + EXPE

[clang] [clang-format] Fix crash involving array designators and dangling comma (PR #77045)

2024-01-06 Thread Björn Schäpers via cfe-commits

@@ -1444,16 +1444,22 @@ WhitespaceManager::CellDescriptions WhitespaceManager::getCells(unsigned Start, } else if (C.Tok->is(tok::comma)) { if (!Cells.empty()) Cells.back().EndIndex = i; -if (C.Tok->getNextNonComment()->isNot(tok::r_brace)) // d

[clang] [clang-format] Handle templated elaborated type specifier in function… (PR #77013)

2024-01-06 Thread Björn Schäpers via cfe-commits

https://github.com/HazardyKnusperkeks commented: Ok, one last thing. From what I figured it was wrongly detected as class definition. So you could add a token annotator test, which verifies that the `{` is not an `ClassLBrace`? https://github.com/llvm/llvm-project/pull/77013 __

[flang] [clang] [llvm] [compiler-rt] [libc] [clang-tools-extra] [lldb] [GlobalIsel] Combine select of binops (PR #76763)

2024-01-06 Thread Thorsten Schütt via cfe-commits

tschuett wrote: I reverted the commit. https://github.com/llvm/llvm-project/pull/76763 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[llvm] [libcxx] [clang] Reapply "[libc++][streams] P1759R6: Native handles and file streams" (PR #77190)

2024-01-06 Thread Hristo Hristov via cfe-commits

https://github.com/Zingam updated https://github.com/llvm/llvm-project/pull/77190 >From 1d2c365efe493b4d62cb84a517d00a7d9dbd58b7 Mon Sep 17 00:00:00 2001 From: Zingam Date: Sat, 6 Jan 2024 12:47:23 +0200 Subject: [PATCH 1/2] Revert "Revert "[libc++][streams] P1759R6: Native handles and file st

[clang] [ARM] arm_acle.h add Coprocessor Instrinsics (PR #75440)

2024-01-06 Thread via cfe-commits

@@ -836,6 +837,70 @@ void ARMTargetInfo::getTargetDefines(const LangOptions &Opts, if (Opts.RWPI) Builder.defineMacro("__ARM_RWPI", "1"); + // Macros for enabling co-proc intrinsics + uint64_t FeatureCoprocBF = 0; + switch (ArchKind) { + default: +break; + case

[clang-tools-extra] [compiler-rt] [lldb] [llvm] [libc] [flang] [clang] [GlobalIsel] Combine select of binops (PR #76763)

2024-01-06 Thread Kai Luo via cfe-commits

Thorsten =?utf-8?q?Schütt?= , Thorsten =?utf-8?q?Schütt?= , Thorsten =?utf-8?q?Schütt?= Message-ID: In-Reply-To: bzEq wrote: It's also exhausting memory when `llvm-lit llvm-project/llvm/test/CodeGen/AMDGPU/llvm.exp2.ll`. See https://lab.llvm.org/buildbot/#/builders/249/builds/13786. https://

[clang] [ARM] arm_acle.h add Coprocessor Instrinsics (PR #75440)

2024-01-06 Thread via cfe-commits

@@ -836,6 +837,70 @@ void ARMTargetInfo::getTargetDefines(const LangOptions &Opts, if (Opts.RWPI) Builder.defineMacro("__ARM_RWPI", "1"); + // Macros for enabling co-proc intrinsics + uint64_t FeatureCoprocBF = 0; + switch (ArchKind) { + default: +break; + case

[clang-tools-extra] [compiler-rt] [lldb] [llvm] [libc] [flang] [clang] [GlobalIsel] Combine select of binops (PR #76763)

2024-01-06 Thread Thorsten Schütt via cfe-commits

tschuett wrote: The file was not changed by this PR and there is no `select` in the file. https://github.com/llvm/llvm-project/pull/76763 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[clang] [ARM] arm_acle.h add Coprocessor Instrinsics (PR #75440)

2024-01-06 Thread via cfe-commits

https://github.com/hstk30-hw updated https://github.com/llvm/llvm-project/pull/75440 >From 18af0ae248707b7c33b24065cdbab5399337f8bc Mon Sep 17 00:00:00 2001 From: hstk30-hw Date: Thu, 14 Dec 2023 15:40:03 +0800 Subject: [PATCH] feat: arm_acle.h add Coprocessor Instrinsics --- clang/lib/Basic/

[clang-tools-extra] [compiler-rt] [lldb] [llvm] [libc] [flang] [clang] [GlobalIsel] Combine select of binops (PR #76763)

2024-01-06 Thread Thorsten Schütt via cfe-commits

tschuett wrote: I had also issues with this test. There is also a GH issue: https://github.com/llvm/llvm-project/issues/76821. https://github.com/llvm/llvm-project/pull/76763 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.o

[clang-tools-extra] [compiler-rt] [lldb] [llvm] [libc] [flang] [clang] [GlobalIsel] Combine select of binops (PR #76763)

2024-01-06 Thread via cfe-commits

Thorsten =?utf-8?q?Schütt?= , Thorsten =?utf-8?q?Schütt?= , Thorsten =?utf-8?q?Schütt?= Message-ID: In-Reply-To: ronlieb wrote: @jhuber6 @jplehrthis may also be failing in staging libc bot https://lab.llvm.org/staging/#/builders/134 https://github.com/llvm/llvm-project/pull/76763 ___

[clang-tools-extra] [compiler-rt] [lldb] [llvm] [libc] [flang] [clang] [GlobalIsel] Combine select of binops (PR #76763)

2024-01-06 Thread via cfe-commits

Thorsten =?utf-8?q?Sch=C3=BCtt?= , Thorsten =?utf-8?q?Sch=C3=BCtt?= , Thorsten =?utf-8?q?Sch=C3=BCtt?= Message-ID: In-Reply-To: ronlieb wrote: this lit test is now taking a very long time after this patch landed ./bin/llvm-lit ../llvm/test/CodeGen/AMDGPU/llvm.log2.ll relevant cmake settings

[llvm] [clang] [RFC] Introducing `__builtin_consistent` to generate AArch64 BC.cond … (PR #72175)

2024-01-06 Thread Nikita Popov via cfe-commits

nikic wrote: @davidbolvansky The semantics of this builtin are slightly different, see the discussion at https://discourse.llvm.org/t/rfc-consistent-branches-support-in-llvm/74889. But yes, the consensus is that adding this builtin is not worthwhile. https://github.com/llvm/llvm-project/pull/

[clang] 0c7d46a - [Clang] Correctly construct template arguments for template template parameters (#76811)

2024-01-06 Thread via cfe-commits

Author: Younan Zhang Date: 2024-01-06T22:26:34+08:00 New Revision: 0c7d46a7fd5b7956e285d385a6945153d6a06eb0 URL: https://github.com/llvm/llvm-project/commit/0c7d46a7fd5b7956e285d385a6945153d6a06eb0 DIFF: https://github.com/llvm/llvm-project/commit/0c7d46a7fd5b7956e285d385a6945153d6a06eb0.diff

  1   2   >