[clang] [clang] Refactor: Move CTAD code from SemaTemplate.cpp to a dedicated file, NFC (PR #98524)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
Endilll wrote: LGTM https://github.com/llvm/llvm-project/pull/98524 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Emit constraint intrinsics for arc and hyperbolic trig clang builtins (PR #98949)

2024-07-15 Thread Farzon Lotfi via cfe-commits
https://github.com/farzonl updated https://github.com/llvm/llvm-project/pull/98949 >From 3c31554214d642c8b3ce8b17739d68445adc81d5 Mon Sep 17 00:00:00 2001 From: Farzon Lotfi Date: Mon, 15 Jul 2024 15:39:14 -0400 Subject: [PATCH 1/2] [clang] Emit constraint intrinsics for arc and hyperbolic tri

[clang] [PAC][clang] Enhance preprocessor ptrauth tests (PR #98862)

2024-07-15 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux` running on `sanitizer-buildbot2` while building `clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/66/builds/1535 Here is the relevant piece of the bu

[clang] [clang-tools-extra] [clang-tidy] Fix false in unnecessary-value-param inside templates (PR #98488)

2024-07-15 Thread Dmitry Polukhin via cfe-commits
https://github.com/dmpolukhin updated https://github.com/llvm/llvm-project/pull/98488 >From a05c4ca2c2e61653e7bd8d3a2f5faa6b86daa615 Mon Sep 17 00:00:00 2001 From: Dmitry Polukhin Date: Thu, 11 Jul 2024 07:01:49 -0700 Subject: [PATCH 1/3] [clang-tidy] Fix false in unnecessary-value-param inside

[clang] [llvm] [DirectX] Start documenting DXIL Resource handling (PR #90553)

2024-07-15 Thread Justin Bogner via cfe-commits
bogner wrote: Just noticed that this was missing the change I had locally for updating the load and store operations to reflect the two different buffer types. I've pushed that in the latest update. https://github.com/llvm/llvm-project/pull/90553 ___

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Joseph Huber via cfe-commits
jhuber6 wrote: Ping https://github.com/llvm/llvm-project/pull/96561 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Bounds Safety][NFC] Add `SemaBoundsSafety` class and move existing Sema checks there (PR #98954)

2024-07-15 Thread Henrik G. Olsson via cfe-commits
https://github.com/hnrklssn approved this pull request. LGTM, might want to give the rest some time to have a look though https://github.com/llvm/llvm-project/pull/98954 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Justin Lebar via cfe-commits
jlebar wrote: @Artem-B https://github.com/llvm/llvm-project/pull/96561 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] clang: Relax LangOpts checks when lexing quoted numbers during preprocessing (PR #95798)

2024-07-15 Thread Jan Svoboda via cfe-commits
@@ -2068,7 +2068,8 @@ bool Lexer::LexNumericConstant(Token &Result, const char *CurPtr) { } // If we have a digit separator, continue. - if (C == '\'' && (LangOpts.CPlusPlus14 || LangOpts.C23)) { + if (C == '\'' && + (LangOpts.CPlusPlus14 || LangOpts.C23 || Parsing

[clang] [llvm] [Clang][Coroutines] Introducing the `[[clang::coro_inplace_task]]` attribute (PR #94693)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 updated https://github.com/llvm/llvm-project/pull/94693 >From bcd6e4ffa3941c9151647f03e559f1fe48f8941d Mon Sep 17 00:00:00 2001 From: Yuxuan Chen Date: Tue, 4 Jun 2024 23:22:00 -0700 Subject: [PATCH] [Clang] Introduce [[clang::coro_inplace_task]] Implement noa

[clang] [Clang] Refactor uses of in SemaOverload.cpp (PR #98965)

2024-07-15 Thread via cfe-commits
https://github.com/vortex73 created https://github.com/llvm/llvm-project/pull/98965 None >From 60a51d9150fa3f31c3eab89e1f1da3b30d190b48 Mon Sep 17 00:00:00 2001 From: Vortex Date: Tue, 16 Jul 2024 03:01:14 +0530 Subject: [PATCH] [Clang] Refactor uses of in SemaOverload.cpp --- clang/lib/Sem

[clang] [Clang] Refactor uses of in SemaOverload.cpp (PR #98965)

2024-07-15 Thread via cfe-commits
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

[clang] [Bounds Safety][NFC] Add `SemaBoundsSafety` class and move existing Sema checks there (PR #98954)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
@@ -0,0 +1,42 @@ +//=== SemaBoundsSafety.h - Bounds Safety specific routines-*- 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: Apa

[clang] [Bounds Safety][NFC] Add `SemaBoundsSafety` class and move existing Sema checks there (PR #98954)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
https://github.com/Endilll requested changes to this pull request. Unfortunately, I think this should be held off until we have a bigger picture for the future of `Sema`. I'll elaborate below. If you take a close look at the existing set of `Sema` parts, it's almost entirely comprised of other

[clang] [Bounds Safety][NFC] Add `SemaBoundsSafety` class and move existing Sema checks there (PR #98954)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
https://github.com/Endilll edited https://github.com/llvm/llvm-project/pull/98954 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Bounds Safety][NFC] Add `SemaBoundsSafety` class and move existing Sema checks there (PR #98954)

2024-07-15 Thread Henrik G. Olsson via cfe-commits
hnrklssn wrote: > If you take a close look at the existing set of `Sema` parts, it's almost > entirely comprised of other languages or language extensions (from C and C++ > standpoint) and backends. They were considered natural enough to be split off > `Sema`, which, among other things, means

[clang] [clang-tools-extra] [clang-tidy] Fix false in unnecessary-value-param inside templates (PR #98488)

2024-07-15 Thread Piotr Zegar via cfe-commits
https://github.com/PiotrZSL commented: Release notes entry is missing. https://github.com/llvm/llvm-project/pull/98488 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Refactor uses of in SemaOverload.cpp (PR #98965)

2024-07-15 Thread via cfe-commits
https://github.com/vortex73 updated https://github.com/llvm/llvm-project/pull/98965 >From 60a51d9150fa3f31c3eab89e1f1da3b30d190b48 Mon Sep 17 00:00:00 2001 From: Vortex Date: Tue, 16 Jul 2024 03:01:14 +0530 Subject: [PATCH 1/3] [Clang] Refactor uses of in SemaOverload.cpp --- clang/lib/Sema/

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/97679 >From b5e0335199541225a1b0eb5eaf661cd4df55dbd7 Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 4 Jul 2024 01:14:11 -0400 Subject: [PATCH 1/6] [clang-doc] add enum test --- clang-tools-extra/test/clang-d

[clang] [Clang] Refactor uses of in SemaOverload.cpp (PR #98965)

2024-07-15 Thread via cfe-commits
https://github.com/vortex73 edited https://github.com/llvm/llvm-project/pull/98965 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Refactor uses of in SemaOverload.cpp (PR #98965)

2024-07-15 Thread via cfe-commits
https://github.com/vortex73 ready_for_review https://github.com/llvm/llvm-project/pull/98965 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/97679 >From b5e0335199541225a1b0eb5eaf661cd4df55dbd7 Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 4 Jul 2024 01:14:11 -0400 Subject: [PATCH 1/6] [clang-doc] add enum test --- clang-tools-extra/test/clang-d

[clang] [Clang] Refactor uses of in SemaOverload.cpp (PR #98965)

2024-07-15 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Narayan (vortex73) Changes - [ ] adds checks to called functions containing `Cand->Function` as an argument. - [ ] Assigned `Cand->Function` as a `FunctionDecl*` to enhance readablity. Solves: #98769 and #98942 --- Full diff: https://gith

[clang] [llvm] [Clang][Coroutines] Introducing the `[[clang::coro_inplace_task]]` attribute (PR #94693)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 updated https://github.com/llvm/llvm-project/pull/94693 >From 4f9596fa73a1ffd56ca34e4c36f7f048e1a0 Mon Sep 17 00:00:00 2001 From: Yuxuan Chen Date: Tue, 4 Jun 2024 23:22:00 -0700 Subject: [PATCH] [Clang] Introduce [[clang::coro_inplace_task]] Implement noa

[clang] [Clang] Refactor uses of `Cand->Function` in SemaOverload.cpp (PR #98965)

2024-07-15 Thread via cfe-commits
https://github.com/vortex73 edited https://github.com/llvm/llvm-project/pull/98965 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Bounds Safety][NFC] Add `SemaBoundsSafety` class and move existing Sema checks there (PR #98954)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
Endilll wrote: > > If you take a close look at the existing set of `Sema` parts, it's almost > > entirely comprised of other languages or language extensions (from C and > > C++ standpoint) and backends. They were considered natural enough to be > > split off `Sema`, which, among other things,

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/97679 >From b5e0335199541225a1b0eb5eaf661cd4df55dbd7 Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 4 Jul 2024 01:14:11 -0400 Subject: [PATCH 1/6] [clang-doc] add enum test --- clang-tools-extra/test/clang-d

[clang] [llvm] [Clang][Coroutines] Introducing the `[[clang::coro_inplace_task]]` attribute (PR #94693)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 updated https://github.com/llvm/llvm-project/pull/94693 >From 23051f31dc3f6f2e0ba29a69cbba0668c6248f48 Mon Sep 17 00:00:00 2001 From: Yuxuan Chen Date: Tue, 4 Jun 2024 23:22:00 -0700 Subject: [PATCH] [Clang] Introduce [[clang::coro_inplace_task]] Implement noa

[clang] [clang-tools-extra] [clang-tidy] Fix false in unnecessary-value-param inside templates (PR #98488)

2024-07-15 Thread Dmitry Polukhin via cfe-commits
https://github.com/dmpolukhin updated https://github.com/llvm/llvm-project/pull/98488 >From a05c4ca2c2e61653e7bd8d3a2f5faa6b86daa615 Mon Sep 17 00:00:00 2001 From: Dmitry Polukhin Date: Thu, 11 Jul 2024 07:01:49 -0700 Subject: [PATCH 1/4] [clang-tidy] Fix false in unnecessary-value-param inside

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread via cfe-commits
@@ -0,0 +1,121 @@ +// RUN: rm -rf %t && mkdir -p %t +// RUN: clang-doc --format=html --doxygen --output=%t --executor=standalone %s +// RUN: clang-doc --format=md --doxygen --output=%t --executor=standalone %s +// RUN: FileCheck %s < %t/GlobalNamespace/index.html -check-prefix=HTM

[clang] [Clang] Introduce Frontend Attribute [[clang::coro_inplace_task]] (PR #98971)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 created https://github.com/llvm/llvm-project/pull/98971 This patch is the frontend implementation of the coroutine elide improvement project detailed in this discourse post: https://discourse.llvm.org/t/language-extension-for-better-more-deterministic-halo-for

[clang] [Clang] Introduce Frontend Attribute [[clang::coro_inplace_task]] (PR #98971)

2024-07-15 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Yuxuan Chen (yuxuanchen1997) Changes This patch is the frontend implementation of the coroutine elide improvement project detailed in this discourse post: https://discourse.llvm.org/t/language-extension-for-better-more-deterministic-halo-

[clang] [llvm] Finish deleting the le32/le64 targets (PR #98497)

2024-07-15 Thread Steven Johnson via cfe-commits
steven-johnson wrote: Unfortunately, this utterly breaks Halide, which was in fact relying on both of these targets. https://github.com/llvm/llvm-project/pull/98497 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin

[clang] [llvm] Finish deleting the le32/le64 targets (PR #98497)

2024-07-15 Thread Steven Johnson via cfe-commits
steven-johnson wrote: > LGTM! That partial Le32/Leb64 restore was to give Halide some time. I understand that you want these gone, but the fact of the matter is that Halide isn't ready for this change, and so the next integrate of LLVM into google3 will break the world. We urgently need these

[clang] [llvm] [LLVM] Perform HALO on "coro_must_elide" coroutines (PR #98974)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 created https://github.com/llvm/llvm-project/pull/98974 This patch is the middle end implementation for the coroutine HALO improvement project published on discourse: https://discourse.llvm.org/t/language-extension-for-better-more-deterministic-halo-for-c-coro

[clang] [llvm] [LLVM] Perform HALO on "coro_must_elide" coroutines (PR #98974)

2024-07-15 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-modules Author: Yuxuan Chen (yuxuanchen1997) Changes This patch is the middle end implementation for the coroutine HALO improvement project published on discourse: https://discourse.llvm.org/t/language-extension-for-better-more-deterministic-halo

[clang] [Clang] Introduce Frontend Attribute [[clang::coro_inplace_task]] (PR #98971)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 edited https://github.com/llvm/llvm-project/pull/98971 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Introduce Frontend Attribute [[clang::coro_inplace_task]] (PR #98971)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 edited https://github.com/llvm/llvm-project/pull/98971 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [Clang][Coroutines] Introducing the `[[clang::coro_inplace_task]]` attribute (PR #94693)

2024-07-15 Thread Yuxuan Chen via cfe-commits
yuxuanchen1997 wrote: Closing this PR as we are discussing on https://discourse.llvm.org/t/language-extension-for-better-more-deterministic-halo-for-c-coroutines/ and as suggested this PR is split into a FE PR https://github.com/llvm/llvm-project/pull/98971 and a ME PR https://github.com/llvm

[clang] [llvm] [Clang][Coroutines] Introducing the `[[clang::coro_inplace_task]]` attribute (PR #94693)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 closed https://github.com/llvm/llvm-project/pull/94693 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] C++20 Coroutines: Introduce Frontend Attribute [[clang::coro_inplace_task]] (PR #98971)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 edited https://github.com/llvm/llvm-project/pull/98971 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [LLVM][Coroutines] Perform HALO on "coro_must_elide" coroutines (PR #98974)

2024-07-15 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 edited https://github.com/llvm/llvm-project/pull/98974 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Artem Belevich via cfe-commits
https://github.com/Artem-B edited https://github.com/llvm/llvm-project/pull/96561 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Artem Belevich via cfe-commits
@@ -0,0 +1,64 @@ + +Clang nvlink Wrapper + + +.. contents:: + :local: + +.. _clang-nvlink-wrapper: + +Introduction + + +This tools works as a wrapper around the NVIDIA ``nvlink`` linker. The purpose +of this wrapper is to prov

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Artem Belevich via cfe-commits
@@ -0,0 +1,776 @@ +//===-- clang-nvlink-wrapper/ClangNVLinkWrapper.cpp - NVIDIA linker util --===// +// +// 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] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Artem Belevich via cfe-commits
@@ -0,0 +1,776 @@ +//===-- clang-nvlink-wrapper/ClangNVLinkWrapper.cpp - NVIDIA linker util --===// +// +// 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] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Artem Belevich via cfe-commits
https://github.com/Artem-B commented: First batch of comments on the patch -- I only got till about the middle of ClangNVLinkWrapper.cpp. Will continue reviewing tomorrow. https://github.com/llvm/llvm-project/pull/96561 ___ cfe-commits mailing list cf

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Artem Belevich via cfe-commits
@@ -0,0 +1,776 @@ +//===-- clang-nvlink-wrapper/ClangNVLinkWrapper.cpp - NVIDIA linker util --===// +// +// 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] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Artem Belevich via cfe-commits
@@ -0,0 +1,64 @@ + +Clang nvlink Wrapper + + +.. contents:: + :local: + +.. _clang-nvlink-wrapper: + +Introduction + + +This tools works as a wrapper around the NVIDIA ``nvlink`` linker. The purpose +of this wrapper is to prov

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Joseph Huber via cfe-commits
@@ -0,0 +1,64 @@ + +Clang nvlink Wrapper + + +.. contents:: + :local: + +.. _clang-nvlink-wrapper: + +Introduction + + +This tools works as a wrapper around the NVIDIA ``nvlink`` linker. The purpose +of this wrapper is to prov

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Joseph Huber via cfe-commits
@@ -0,0 +1,64 @@ + +Clang nvlink Wrapper + + +.. contents:: + :local: + +.. _clang-nvlink-wrapper: + +Introduction + + +This tools works as a wrapper around the NVIDIA ``nvlink`` linker. The purpose +of this wrapper is to prov

[clang] [Clang] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Joseph Huber via cfe-commits
@@ -0,0 +1,776 @@ +//===-- clang-nvlink-wrapper/ClangNVLinkWrapper.cpp - NVIDIA linker util --===// +// +// 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] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Joseph Huber via cfe-commits
@@ -0,0 +1,776 @@ +//===-- clang-nvlink-wrapper/ClangNVLinkWrapper.cpp - NVIDIA linker util --===// +// +// 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] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Joseph Huber via cfe-commits
@@ -0,0 +1,776 @@ +//===-- clang-nvlink-wrapper/ClangNVLinkWrapper.cpp - NVIDIA linker util --===// +// +// 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] Introduce 'clang-nvlink-wrapper' to work around 'nvlink' (PR #96561)

2024-07-15 Thread Joseph Huber via cfe-commits
jhuber6 wrote: > First batch of comments on the patch -- I only got till about the middle of > ClangNVLinkWrapper.cpp. Will continue reviewing tomorrow. Appreciate it. Sorry about dropping a huge patch like this on you. https://github.com/llvm/llvm-project/pull/96561 __

[clang] Implement resource binding type prefix mismatch diagnostic infrastructure (PR #97103)

2024-07-15 Thread Joshua Batista via cfe-commits
@@ -1,9 +1,10 @@ // RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library -x hlsl -o - -fsyntax-only %s -verify -// expected-error@+1 {{invalid resource class specifier 'c' used; expected 'b', 's', 't', or 'u'}} -float a : register(c0, space1); +// valid, The register keywor

[clang] [llvm] Finish deleting the le32/le64 targets (PR #98497)

2024-07-15 Thread Eli Friedman via cfe-commits
efriedma-quic wrote: It looks like the removal missed a few bits: PNaClABIBuiltinVaList and clang/lib/CodeGen/Targets/PNaCl.cpp weren't removed. > Unfortunately, this utterly breaks Halide, which was in fact relying on both > of these targets. We can temporarily revert this, but we aren't goi

[clang] [llvm] Finish deleting the le32/le64 targets (PR #98497)

2024-07-15 Thread Fangrui Song via cfe-commits
MaskRay wrote: It seems that Halide (https://github.com/halide/Halide/pull/5934), an important user, still needs generic 32-bit 64-bit Clang targets. The removed le32/le64 targets fulfilled this need. (This change broke my internal users. Sorry that I did not realize that Halide#5934 was closed

[clang] [llvm] Finish deleting the le32/le64 targets (PR #98497)

2024-07-15 Thread Andrew Adams via cfe-commits
abadams wrote: Our actual need is for a stable and user-friendly way to write fragments of mostly-target-agnostic llvm IR. We concat these with other fragments and give it an actual target triple later. This lets us assemble a target-specific Halide runtime dynamically by mixing and matching

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
https://github.com/Endilll updated https://github.com/llvm/llvm-project/pull/98736 >From 44b4a682f2b3d7e140f7b1bd124e7aabdbf439ad Mon Sep 17 00:00:00 2001 From: Vlad Serebrennikov Date: Sat, 13 Jul 2024 13:10:25 +0300 Subject: [PATCH 1/2] [clang] Add deprecation warning for `-Ofast` driver opt

[clang] [llvm] Finish deleting the le32/le64 targets (PR #98497)

2024-07-15 Thread Eli Friedman via cfe-commits
efriedma-quic wrote: There is no such thing as "generic" LLVM IR, in general: so many target-specific aspects of C have to be directly encoded into the IR, so there's no way to actually make it generic. In this respect, the "le32/le64" targets aren't really any more generic than any other tar

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
@@ -442,6 +442,9 @@ def warn_drv_deprecated_arg : Warning< def warn_drv_deprecated_arg_no_relaxed_template_template_args : Warning< "argument '-fno-relaxed-template-template-args' is deprecated">, InGroup; +def warn_drv_deprecated_arg_ofast : Warning< + "argument '-Ofast'

[clang] [clang][ExtractAPI][NFC] Remove some nullptr dereference problems (PR #98914)

2024-07-15 Thread via cfe-commits
https://github.com/QuietMisdreavus approved this pull request. https://github.com/llvm/llvm-project/pull/98914 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
@@ -470,6 +470,9 @@ New Compiler Flags Deprecated Compiler Flags - +- ``-Ofast`` is deprecated in favor of ``-O3``, possibly combined with ``-ffast-math``. Endilll wrote: Fixed. Thank you! https://github.com/llvm/llvm-project/pull/98

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
@@ -10,6 +10,7 @@ // RUN: %clang -Ofast -fno-strict-aliasing -### %s 2>&1 | FileCheck -check-prefix=CHECK-OFAST-NO-STRICT-ALIASING %s // RUN: %clang -Ofast -fno-vectorize -### %s 2>&1 | FileCheck -check-prefix=CHECK-OFAST-NO-VECTORIZE %s +// CHECK-OFAST: warning: argument '-

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
@@ -931,7 +931,8 @@ def O : Joined<["-"], "O">, Group, def O_flag : Flag<["-"], "O">, Visibility<[ClangOption, CC1Option, FC1Option]>, Alias, AliasArgs<["1"]>; def Ofast : Joined<["-"], "Ofast">, Group, - Visibility<[ClangOption, CC1Option, FlangOption]>; + Visibility<[Clan

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
@@ -931,7 +931,8 @@ def O : Joined<["-"], "O">, Group, def O_flag : Flag<["-"], "O">, Visibility<[ClangOption, CC1Option, FC1Option]>, Alias, AliasArgs<["1"]>; def Ofast : Joined<["-"], "Ofast">, Group, - Visibility<[ClangOption, CC1Option, FlangOption]>; + Visibility<[Clan

[clang] [clang] Emit constraint intrinsics for arc and hyperbolic trig clang builtins (PR #98949)

2024-07-15 Thread Eli Friedman via cfe-commits
https://github.com/efriedma-quic approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/98949 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add nested namespace test case (PR #97681)

2024-07-15 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/97681 >From f9d5cc074e4b65a92703bd09e62696a29fed1237 Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 4 Jul 2024 01:31:13 -0400 Subject: [PATCH 1/6] [clang-doc] add nested naemspace --- .../test/clang-doc/name

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Fangrui Song via cfe-commits
@@ -442,6 +442,9 @@ def warn_drv_deprecated_arg : Warning< def warn_drv_deprecated_arg_no_relaxed_template_template_args : Warning< "argument '-fno-relaxed-template-template-args' is deprecated">, InGroup; +def warn_drv_deprecated_arg_ofast : Warning< + "argument '-Ofast'

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/97679 >From b5e0335199541225a1b0eb5eaf661cd4df55dbd7 Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 4 Jul 2024 01:14:11 -0400 Subject: [PATCH 1/5] [clang-doc] add enum test --- clang-tools-extra/test/clang-d

[clang] [llvm] Finish deleting the le32/le64 targets (PR #98497)

2024-07-15 Thread Andrew Adams via cfe-commits
abadams wrote: We were advised to use the pnacl targets at the time the code was written. We're trying wasm now. The IR can be generic if you use a subset of C that excludes all the things that vary across the targets you care about. For cases we really can't avoid it (e.g. win32 calling con

[clang-tools-extra] [clang-doc] add nested namespace test case (PR #97681)

2024-07-15 Thread via cfe-commits
PeterChou1 wrote: > To clarify my earlier comment, FileCheck is just matching text on the input > based on the check lines. So in this cases the LINE directive should just > affect what text will be accepted when doin matching. It won’t try to match > the lines to the position in the file bein

[clang] [clang] Fix self-capturing `__block` variables (PR #89475)

2024-07-15 Thread via cfe-commits
@@ -2764,6 +2794,9 @@ void CodeGenFunction::emitByrefStructureInit(const AutoVarEmission &emission) { auto layoutInfo = CGM.getObjCRuntime().BuildByrefLayout(CGM, type); storeHeaderField(layoutInfo, getPointerSize(), "byref.layout"); } + + if (emission.NeedsInitOnHe

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Fangrui Song via cfe-commits
@@ -10,31 +10,36 @@ // RUN: %clang -Ofast -fno-strict-aliasing -### %s 2>&1 | FileCheck -check-prefix=CHECK-OFAST-NO-STRICT-ALIASING %s // RUN: %clang -Ofast -fno-vectorize -### %s 2>&1 | FileCheck -check-prefix=CHECK-OFAST-NO-VECTORIZE %s +// CHECK-OFAST: argument '-Ofast'

[clang] [llvm] Finish deleting the le32/le64 targets (PR #98497)

2024-07-15 Thread Steven Johnson via cfe-commits
steven-johnson wrote: > We're trying wasm now. This looked promising in our GitHub tests, but for reasons that aren't clear (yet), ~everything compiled this way inside google3 resulted in linker errors (missing symbols). https://github.com/llvm/llvm-project/pull/98497

[clang] 864478c - [Clang] Disallow explicit object parameters in more contexts (#89078)

2024-07-15 Thread via cfe-commits
Author: Sirraide Date: 2024-07-16T01:55:11+02:00 New Revision: 864478cc74f5e258f86014886df16aa8d393bcc6 URL: https://github.com/llvm/llvm-project/commit/864478cc74f5e258f86014886df16aa8d393bcc6 DIFF: https://github.com/llvm/llvm-project/commit/864478cc74f5e258f86014886df16aa8d393bcc6.diff LOG:

[clang] [Clang] Disallow explicit object parameters in more contexts (PR #89078)

2024-07-15 Thread via cfe-commits
https://github.com/Sirraide closed https://github.com/llvm/llvm-project/pull/89078 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [Coverage][MC/DC] Show uncoverable and unreachable conditions (PR #94137)

2024-07-15 Thread via cfe-commits
https://github.com/Lambdaris updated https://github.com/llvm/llvm-project/pull/94137 >From 23669f721d056d6e4961e2572e438e5c1f91cab1 Mon Sep 17 00:00:00 2001 From: Lambdaris Date: Sun, 2 Jun 2024 10:19:31 +0800 Subject: [PATCH 1/3] [coverage] Mark branches with either counter is zero as folded

[clang] [llvm] [Coverage][MC/DC] Show uncoverable and unreachable conditions (PR #94137)

2024-07-15 Thread via cfe-commits
https://github.com/Lambdaris edited https://github.com/llvm/llvm-project/pull/94137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] IR: Store the default subtarget info in named metadata instead of the context. (PR #98673)

2024-07-15 Thread Jessica Clarke via cfe-commits
@@ -1240,11 +1240,22 @@ void IRLinker::linkNamedMDNodes() { if (IsPerformingImport && NMD.getName() == "llvm.stats") continue; +// Default subtarget info is only intended to be used before LTO and +// shouldn't be present after merging because the default sub

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,121 @@ +// RUN: rm -rf %t && mkdir -p %t +// RUN: clang-doc --format=html --doxygen --output=%t --executor=standalone %s +// RUN: clang-doc --format=md --doxygen --output=%t --executor=standalone %s +// RUN: FileCheck %s < %t/GlobalNamespace/index.html -check-prefix=HTM

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-15 Thread Vlad Serebrennikov via cfe-commits
https://github.com/Endilll updated https://github.com/llvm/llvm-project/pull/98736 >From 44b4a682f2b3d7e140f7b1bd124e7aabdbf439ad Mon Sep 17 00:00:00 2001 From: Vlad Serebrennikov Date: Sat, 13 Jul 2024 13:10:25 +0300 Subject: [PATCH 1/3] [clang] Add deprecation warning for `-Ofast` driver opt

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi approved this pull request. LGTM, modulo some nits. I do think you may want to have a separate prefix for the lines, but if you change `--check-prefix=` to `--check-prefixes=HTML-ANIMAL,HTML-ANIMAL-LINE`, you should be able to check them w/o having to rewrite everyth

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,128 @@ +// RUN: rm -rf %t && mkdir -p %t +// RUN: clang-doc --format=html --doxygen --output=%t --executor=standalone %s +// RUN: clang-doc --format=md --doxygen --output=%t --executor=standalone %s +// RUN: FileCheck %s < %t/GlobalNamespace/index.html -check-prefix=HTM

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/97679 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add enum test (PR #97679)

2024-07-15 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,128 @@ +// RUN: rm -rf %t && mkdir -p %t +// RUN: clang-doc --format=html --doxygen --output=%t --executor=standalone %s +// RUN: clang-doc --format=md --doxygen --output=%t --executor=standalone %s +// RUN: FileCheck %s < %t/GlobalNamespace/index.html -check-prefix=HTM

[clang] [llvm] IR: Store the default subtarget info in named metadata instead of the context. (PR #98673)

2024-07-15 Thread via cfe-commits
@@ -1240,11 +1240,22 @@ void IRLinker::linkNamedMDNodes() { if (IsPerformingImport && NMD.getName() == "llvm.stats") continue; +// Default subtarget info is only intended to be used before LTO and +// shouldn't be present after merging because the default sub

[clang-tools-extra] [clang-doc] add nested namespace test case (PR #97681)

2024-07-15 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/97681 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add nested namespace test case (PR #97681)

2024-07-15 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,274 @@ +// RUN: rm -rf %t && mkdir -p %t +// RUN: clang-doc --format=html --output=%t --executor=standalone %s +// RUN: clang-doc --format=md --output=%t --executor=standalone %s +// RUN: FileCheck %s < %t/index_json.js -check-prefix=JSON-INDEX +// RUN: FileCheck %s < %

[clang-tools-extra] [clang-doc] add nested namespace test case (PR #97681)

2024-07-15 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,274 @@ +// RUN: rm -rf %t && mkdir -p %t +// RUN: clang-doc --format=html --output=%t --executor=standalone %s +// RUN: clang-doc --format=md --output=%t --executor=standalone %s +// RUN: FileCheck %s < %t/index_json.js -check-prefix=JSON-INDEX +// RUN: FileCheck %s < %

[clang-tools-extra] [clang-doc] add nested namespace test case (PR #97681)

2024-07-15 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi approved this pull request. LGTM, once the LINE checks are in a more stable position w/ their own prefixes, and a couple of nits on whitespace. https://github.com/llvm/llvm-project/pull/97681 ___ cfe-commits mailing list cfe

[clang] bddab51 - [X86][MC,Driver] Support -msse2avx to encode SSE instruction with VEX prefix (#96860)

2024-07-15 Thread via cfe-commits
Author: JaydeepChauhan14 Date: 2024-07-16T09:40:45+08:00 New Revision: bddab518db003f7c5ce9014e9ebd932fb7a85d2f URL: https://github.com/llvm/llvm-project/commit/bddab518db003f7c5ce9014e9ebd932fb7a85d2f DIFF: https://github.com/llvm/llvm-project/commit/bddab518db003f7c5ce9014e9ebd932fb7a85d2f.di

[clang] [llvm] [X86][MC,Driver] Support -msse2avx to encode SSE instruction with VEX prefix (PR #96860)

2024-07-15 Thread Shengchen Kan via cfe-commits
https://github.com/KanRobert closed https://github.com/llvm/llvm-project/pull/96860 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [X86][MC,Driver] Support -msse2avx to encode SSE instruction with VEX prefix (PR #96860)

2024-07-15 Thread via cfe-commits
github-actions[bot] wrote: @JaydeepChauhan14 Congratulations on having your first Pull Request (PR) merged into the LLVM Project! Your changes will be combined with recent changes from other authors, then tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with

[clang-tools-extra] [clangd] [C++20] [Modules] Introduce initial support for C++20 Modules (PR #66462)

2024-07-15 Thread Chuanqi Xu via cfe-commits
ChuanqiXu9 wrote: @Arthapz @guijiyang thanks for reporting. But could you try to give feedbacks to that repo instead of this page. I feel this page it too long now. (My browser needs seconds to load it). And this page is indeed not a good place to track that. --- @guijiyang it looks like th

[clang] [C++20] [Modules] [Itanium ABI] Generate the vtable in the module unit of dynamic classes (PR #75912)

2024-07-15 Thread Chuanqi Xu via cfe-commits
ChuanqiXu9 wrote: > > > (FWIW, check some of the recent modules changes @ChuanqiXu9 has been > > working on to see that reviewer bandwidth here is pretty thin (& my > > experience in LLVM in general, including clang, is that reviewer bandwidth > > is pretty thin - though it is something we sh

[clang] 578cf72 - [RISCV] Add support for getHostCPUFeatures using hwprobe (#94352)

2024-07-15 Thread via cfe-commits
Author: Yingwei Zheng Date: 2024-07-16T10:11:39+08:00 New Revision: 578cf724de56e6c6768859bb1b6a597ee5d14ea0 URL: https://github.com/llvm/llvm-project/commit/578cf724de56e6c6768859bb1b6a597ee5d14ea0 DIFF: https://github.com/llvm/llvm-project/commit/578cf724de56e6c6768859bb1b6a597ee5d14ea0.diff

[clang] [llvm] [RISCV] Add support for getHostCPUFeatures using hwprobe (PR #94352)

2024-07-15 Thread Yingwei Zheng via cfe-commits
https://github.com/dtcxzyw closed https://github.com/llvm/llvm-project/pull/94352 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [AIX] Add -msave-reg-params to save arguments to stack (PR #97524)

2024-07-15 Thread Chen Zheng via cfe-commits
https://github.com/chenzheng1030 approved this pull request. LGTM. Thanks for adding this support. https://github.com/llvm/llvm-project/pull/97524 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/c

  1   2   3   4   5   >