https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/144368
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/144368
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/144368
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
woruyu wrote:
Additionally, I noticed the same issue occurs with `-fprofile-list=` (e.g.
`clang test.c -Xclang -fprofile-list=1 -c`). I plan to address it in a
follow-up patch.
https://github.com/llvm/llvm-project/pull/147959
___
cfe-commits mail
@@ -1697,6 +1698,15 @@ ASTContext::getRelocationInfoForCXXRecord(const
CXXRecordDecl *RD) const {
return std::nullopt;
}
+void ASTContext::initSanitizers(const LangOptions &LangOpts,
+SourceManager &SM) {
+ std::string Error;
+ if (!NoSanit
rofirrim wrote:
> Need a rebase
Done.
I still owe you to split that new `enum`.
@Meinersbur if you have thoughts about this comment here
https://github.com/llvm/llvm-project/pull/139293#issuecomment-3074305797 , I'd
be very thankful.
https://github.com/llvm/llvm-project/pull/139293
https://github.com/woruyu updated
https://github.com/llvm/llvm-project/pull/147959
>From e18ed1ac787a0571318b8cfe1d4eee023ba11fc2 Mon Sep 17 00:00:00 2001
From: woruyu <1214539...@qq.com>
Date: Thu, 10 Jul 2025 21:08:24 +0800
Subject: [PATCH 1/6] fix: replace report_fatal_error with Diags and ex
github-actions[bot] wrote:
@shashforge 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 a bui
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/147400
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Shashi Shankar
Date: 2025-07-24T08:48:27+02:00
New Revision: 31db0f0a7ae43981fdfadc693662c239f921a05b
URL:
https://github.com/llvm/llvm-project/commit/31db0f0a7ae43981fdfadc693662c239f921a05b
DIFF:
https://github.com/llvm/llvm-project/commit/31db0f0a7ae43981fdfadc693662c239f921a05b.diff
https://github.com/negativ updated
https://github.com/llvm/llvm-project/pull/146355
>From fc3b77d8c4b5dd264bd746ed997bcea6cddaf389 Mon Sep 17 00:00:00 2001
From: Andrey Karlov
Date: Mon, 30 Jun 2025 17:05:41 +0300
Subject: [PATCH 01/19] Initial implementation
---
.../bugprone/unchecked-option
https://github.com/pskrgag approved this pull request.
LGTM! Not sure why windows CI is unhappy, but the change makes total sense to me
https://github.com/llvm/llvm-project/pull/150240
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lis
https://github.com/naoNao89 updated
https://github.com/llvm/llvm-project/pull/150225
From 2d9ae771247af025d6319c044e9f8727e9bebfd8 Mon Sep 17 00:00:00 2001
From: naoNao89 <90588855+naona...@users.noreply.github.com>
Date: Wed, 23 Jul 2025 20:54:02 +0700
Subject: [PATCH 1/2] [clang][StaticAnalyze
@@ -7771,6 +7771,37 @@ TEST_F(FormatTest, ConstructorInitializers) {
"Constructor() :\n"
"// Comment forcing unwanted break.\n"
"() {}");
+
+ // Braced initializers with trailing commas.
+ verifyFormat("MyClass::MyC
@@ -7771,6 +7771,37 @@ TEST_F(FormatTest, ConstructorInitializers) {
"Constructor() :\n"
"// Comment forcing unwanted break.\n"
"() {}");
+
+ // Braced initializers with trailing commas.
+ verifyFormat("MyClass::MyC
-explorer/clang-trunk-20250723/bin/../include/c++/v1/variant:1186:37:
fatal error: recursive template instantiation exceeded maximum depth of 1024
1186 | enable_if_t, variant>,
int>= 0,
| ^
/opt/compiler-explorer/clang-trunk-202
@@ -14,35 +14,66 @@
#define LLVM_CLANG_LIB_CODEGEN_SANITIZER_HANDLER_H
#define LIST_SANITIZER_CHECKS
\
- SANITIZER_CHECK(AddOverflow, add_overflow, 0)
\
- SANITIZER_CHECK(BuiltinUnreachable, bu
https://github.com/owenca closed
https://github.com/llvm/llvm-project/pull/150166
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
It's better to add a value to `SpaceBeforeParensOptions` instead. See #150367.
https://github.com/llvm/llvm-project/pull/150166
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Owen Pan (owenca)
Changes
Closes #149971
---
Full diff: https://github.com/llvm/llvm-project/pull/150367.diff
6 Files Affected:
- (modified) clang/docs/ClangFormatStyleOptions.rst (+8)
- (modified) clang/include/clang/Format/Format.h (
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Owen Pan (owenca)
Changes
Closes #149971
---
Full diff: https://github.com/llvm/llvm-project/pull/150367.diff
6 Files Affected:
- (modified) clang/docs/ClangFormatStyleOptions.rst (+8)
- (modified) clang/include/clang/Format/For
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/150367
Closes #149971
>From bbe06c902d24df2ae9163ba0b43eca45e82c6506 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Wed, 23 Jul 2025 15:51:34 -0700
Subject: [PATCH] [clang-format] Add AfterNot to SpaceBeforeParensOption
phoebewang wrote:
> This requires a release note, especially because it fixes a bug.
We don't change the intrinsic behavior and bug fixes usually don't need a
release note. It's only needed when backporting to release branch.
https://github.com/llvm/llvm-project/pull/149745
___
@@ -76,27 +76,45 @@ namespace test1 {
~A();
};
- // CHECK-LABEL: define{{.*}} void @_ZN5test14testEPA10_A20_NS_1AE(
- void test(A (*arr)[10][20]) {
+ // CHECK-LABEL: define{{.*}} void @_ZN5test11fEPA10_A20_NS_1AE(
+ void f(A (*arr)[10][20]) {
delete [] arr;
@@ -17762,14 +17762,19 @@ TEST_F(FormatTest, ConfigurableSpaceBeforeParens) {
}
TEST_F(FormatTest, SpaceAfterLogicalNot) {
- FormatStyle Spaces = getLLVMStyle();
- Spaces.SpaceAfterLogicalNot = true;
+ auto Spaces = getLLVMStyle();
+ EXPECT_EQ(Spaces.SpaceAfterLogicalNot,
@@ -4483,13 +4483,32 @@ struct FormatStyle {
/// \version 3.5
bool SpaceAfterCStyleCast;
- /// If ``true``, a space is inserted after the logical not operator (``!``).
- /// \code
- ///true: false:
- ///! someExpression();
@@ -4483,13 +4483,32 @@ struct FormatStyle {
/// \version 3.5
bool SpaceAfterCStyleCast;
- /// If ``true``, a space is inserted after the logical not operator (``!``).
- /// \code
- ///true: false:
- ///! someExpression();
https://github.com/naoNao89 updated
https://github.com/llvm/llvm-project/pull/150365
From 2d9ae771247af025d6319c044e9f8727e9bebfd8 Mon Sep 17 00:00:00 2001
From: naoNao89 <90588855+naona...@users.noreply.github.com>
Date: Wed, 23 Jul 2025 20:54:02 +0700
Subject: [PATCH 1/2] [clang][StaticAnalyze
hoodmane wrote:
If this change looks good can we merge it?
https://github.com/llvm/llvm-project/pull/150201
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tJener edited
https://github.com/llvm/llvm-project/pull/150361
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tJener edited
https://github.com/llvm/llvm-project/pull/150361
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
qinkunbao wrote:
Hi @marcogmaia and @kadircet ,
I believe the this PR broke a few buildbots. Would you mind taking a look?
```
[ RUN ] OverridePureVirtualsTests.MultiAccessSpecifiersOverride
Built preamble of size 211928 for file /clangd-test/TestTU.cpp version null in
0.00 seconds
#0 0x
https://github.com/ilovepi approved this pull request.
https://github.com/llvm/llvm-project/pull/145967
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aadanen wrote:
I thought about "Undoing Input" as the section header but I feel like %undo is
pretty self explanatory. As far as "%help" and clang-repl --help go, I think
those sound like nice ideas. Both would be useful for new people using
clang-repl.
As far as "%foobar" throwing an error
https://github.com/cyndyishida edited
https://github.com/llvm/llvm-project/pull/150349
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cyndyishida wrote:
This PR would also resolve: https://github.com/llvm/llvm-project/issues/57432 ,
but I'm not sure if its still relevant for the affected client code
https://github.com/llvm/llvm-project/pull/150349
___
cfe-commits mailing list
cfe-c
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Cyndy Ishida (cyndyishida)
Changes
The mentioned flag is already both a cc1 & driver flag; however, whether it
is respected was tied to either:
1. Whether it was passed as a cc1 option (`Xclang`)
2. or `-fmodules` accompanying it
This pos
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/150349
The mentioned flag is already both a cc1 & driver flag; however, whether it is
respected was tied to either:
1. Whether it was passed as a cc1 option (`Xclang`)
2. or `-fmodules` accompanying it
This poses
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Aaron Danen (aadanen)
Changes
so far just added undo to the documentation. We can see about implementing
%help or something similar. This will attempt to solve issue #143666
---
Full diff: https://github.com/llvm/llvm-project/pull/150348
https://github.com/aadanen created
https://github.com/llvm/llvm-project/pull/150348
so far just added undo to the documentation. We can see about implementing
%help or something similar. This will attempt to solve issue #143666
>From 4e24cb20efa22e364d41026ae40aec470ff786aa Mon Sep 17 00:00:0
https://github.com/alexfh closed
https://github.com/llvm/llvm-project/pull/150003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Samarth Narang
Date: 2025-07-24T03:04:05+02:00
New Revision: 22fef005225b129d73ade4ed995fc0ec0c7be044
URL:
https://github.com/llvm/llvm-project/commit/22fef005225b129d73ade4ed995fc0ec0c7be044
DIFF:
https://github.com/llvm/llvm-project/commit/22fef005225b129d73ade4ed995fc0ec0c7be044.diff
https://github.com/alexfh approved this pull request.
The fix looks good now.
We might still want to revisit the original patch though
(https://github.com/llvm/llvm-project/pull/145166#issuecomment-3100202999).
https://github.com/llvm/llvm-project/pull/150003
__
mizvekov wrote:
> I added comment `// FIXME (GH147000): duplicate diagnostics` to
> `clang/test/SemaCXX/nested-name-spec.cpp` in #147003. Please remove the
> comment if that issue is fixed by this patch.
This is actually fixed by changes I left out of this patch, I updated the
commit descript
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/147835
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alexfh wrote:
> I'm trying to unwrap this more, so far I got to this point:
> https://gcc.godbolt.org/z/q76YxfKzP
>
> Hopefully I can leave only libc++ includes soon.
It turned out to be trickier than I thought, but the test is being
automatically reduced. I'll post the result when it's ready
Halalaluyafail3 wrote:
> So we error by default, but still allow users to opt out of the error if they
> find they need to support an `_Atomic` underlying type.
Does clang normally reject some valid programs by default with a non-gnu
standard selected, intentionally (i.e. not a bug)? If so, is
e-kud wrote:
> > TBH I'm not sure if we need to add `avx512f,evex512` to intrinsics
> > attributes and/or to `.td` as well.
>
> Yes, we need them for now. Otherwise, we cannot allocate ZMM registers.
What about `amx-tile`? It looks like it should be a predicate for all AMX
instructions. But c
@@ -16,7 +16,15 @@
#define __DEFAULT_FN_ATTRS_AVX512
\
__attribute__((__always_inline__, __nodebug__,
\
- __target__("amx-avx512,avx10.2-512")))
+ __target__("amx-avx
https://github.com/e-kud updated
https://github.com/llvm/llvm-project/pull/148633
>From 97044a895e5df9ec591775589495c7f9f7e855a7 Mon Sep 17 00:00:00 2001
From: Evgenii Kudriashov
Date: Mon, 14 Jul 2025 06:21:27 -0700
Subject: [PATCH 1/3] [X86][AVX10.2] Decouple AMX-AVX512 from AVX10.2
MIME-Vers
https://github.com/shafik commented:
Thank you for the fix, please add more details in the summary next time.
Something along the lines of "added diagnostic which now triggers when
which avoids the crash "
https://github.com/llvm/llvm-project/pull/149982
___
@@ -0,0 +1,22 @@
+// FIXME: Currently, a nullptr check is in place before emitting a UBSan trap
reason inside
+// CodeGenFunction::EmitTrapCheck. This is disadvantageous because we may not
emit a trap
+// message in all cases where a trap message should be emitted. The check w
https://github.com/shafik commented:
Thank you for the fix, please add a release note.
https://github.com/llvm/llvm-project/pull/150025
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dschuff wrote:
This change looks fine.
Regarding nontrapping call, In general I think we probably shouldn't have clang
builtins for functionality that can be implemented in code on top of existing
builtins. Builtins are basically language extensions (but often without the
extensive vetting tha
Prabhuk wrote:
Thanks @vvereschaka for reporting. And thanks @zeroomega for the revert.
https://github.com/llvm/llvm-project/pull/87574
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hoodmane wrote:
Other builtins with dynamic return type:
`__builtin_call_with_static_chain`
> This builtin is only available for C. This builtin can be used to call Go
> closures from C.
`__builtin_choose_expr`
> This built-in function is analogous to the ? : operator in C, except t
https://github.com/shashforge updated
https://github.com/llvm/llvm-project/pull/147400
>From 29c3c9c6d25cc7c47d01a9d819ea37e92361aea2 Mon Sep 17 00:00:00 2001
From: Shashi Shankar
Date: Mon, 7 Jul 2025 22:54:22 +0200
Subject: [PATCH 1/5] Sema: suppress deprecated field warnings in implicit
spe
@@ -0,0 +1,190 @@
+// RUN: %clang_cc1 -triple x86_64-unknown-linux-gnu -emit-llvm -fsanitize=kcfi
-o - %s | FileCheck %s
+// RUN: %clang_cc1 -triple x86_64-unknown-linux-gnu -emit-llvm -fsanitize=kcfi
-x c++ -o - %s | FileCheck %s --check-prefixes=CHECK,MEMBER
+// RUN: %clang_cc
hoodmane wrote:
Well it would also be possible to make success the return value, and make the
first argument a `void* result`. This would be a more normal API, but it has
the disadvantage that it would never be able to work with functions that return
reference types. Maybe that isn't that big
@@ -12251,6 +12264,9 @@ bool Sema::CheckFunctionDeclaration(Scope *S,
FunctionDecl *NewFD,
if (NewFD->hasAttr())
SYCL().CheckSYCLEntryPointFunctionDecl(NewFD);
+ if (NewFD->hasAttr())
+SYCL().CheckSYCLExternalFunctionDecl(NewFD);
tahonermann wrote:
https://github.com/ddpagan closed
https://github.com/llvm/llvm-project/pull/150305
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: David Pagan
Date: 2025-07-23T14:04:49-07:00
New Revision: f443f561331dc54aaed6897f51d7632d62a5ea95
URL:
https://github.com/llvm/llvm-project/commit/f443f561331dc54aaed6897f51d7632d62a5ea95
DIFF:
https://github.com/llvm/llvm-project/commit/f443f561331dc54aaed6897f51d7632d62a5ea95.diff
L
Author: Joseph Huber
Date: 2025-07-23T16:04:25-05:00
New Revision: 6bc54a4874eeaddf2a1f7c75aed53d9d38ed313c
URL:
https://github.com/llvm/llvm-project/commit/6bc54a4874eeaddf2a1f7c75aed53d9d38ed313c
DIFF:
https://github.com/llvm/llvm-project/commit/6bc54a4874eeaddf2a1f7c75aed53d9d38ed313c.diff
Sirraide wrote:
> I don't think an opt-in flag justifies the maintenance burden; I wouldn't
> expect users to enable that flag by default (honestly, the folks who benefit
> the most from the canonicalization are likely the same folks who would not
> think to add a new flag to the command line
@@ -12166,6 +12174,15 @@ static void DiagnoseBadDeduction(Sema &S, NamedDecl
*Found, Decl *Templated,
diag::note_ovl_candidate_explicit_arg_mismatch_unnamed)
<< (index + 1);
}
+
+if (PartialDiagnosticAt *PDiag = DeductionFailure.getSFINAEDiagnost
https://github.com/jthackray updated
https://github.com/llvm/llvm-project/pull/150054
>From b74d772aa18f0994d30379606f826e69c76c85bc Mon Sep 17 00:00:00 2001
From: Jonathan Thackray
Date: Tue, 22 Jul 2025 17:16:09 +0100
Subject: [PATCH 1/2] [clang] Rename files that MacOS libtool warns about (N
hoodmane wrote:
It seems that builtins can do that. In `SemaWasm.cpp` we call
`TheCall->setType(FuncTy->getReturnType());` and that implies that the return
type of the call to `__builtin_wasm_nontrapping_call()` is the same as the
return type of `func`.
https://github.com/llvm/llvm-project/pu
jamieschmeiser wrote:
> I think it makes sense for Clang to handle the C Standard Library headers
> which can introduce `NULL`, but I'm surprised to see us adding new headers
> like `dbm.h`, the `sys` directory, and `unistd,h`.
These other headers define NULL to 0 on AIX and are included throu
@@ -0,0 +1,21 @@
+.. title:: clang-tidy - llvm-mlir-op-builder
+
+llvm-mlir-op-builder
+
+
+Checks for uses of MLIR's old/to be deprecated `OpBuilder::create` form
+and suggests using `T::create` instead.
EugeneZelenko wrote:
```suggestion
Che
@@ -1,4 +1,4 @@
-//===--- AMDCPU.cpp - Emit LLVM Code for builtins
-===//
+//===--- BuiltinAMDCPU.cpp - Emit LLVM Code for builtins
--===//
farzonl wrote:
```suggestion
//===--- BuiltinAMDGPU.cpp - Emit LLVM Code for
https://github.com/rupprecht closed
https://github.com/llvm/llvm-project/pull/150301
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Jordan Rupprecht
Date: 2025-07-23T15:46:49-05:00
New Revision: 34447efd4614cfc8f17aa5cb7305f76ebaf6eaf5
URL:
https://github.com/llvm/llvm-project/commit/34447efd4614cfc8f17aa5cb7305f76ebaf6eaf5
DIFF:
https://github.com/llvm/llvm-project/commit/34447efd4614cfc8f17aa5cb7305f76ebaf6eaf5.di
https://github.com/rupprecht edited
https://github.com/llvm/llvm-project/pull/150301
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sbc100 wrote:
> My thought is that it returns the return value. If one of the signatures
> matches it sets `*success = 1` otherwise it sets `*success = 0`. It probably
> can be written in terms of `__builtin_wasm_test_function_pointer_signature`
> with C++ templates but lot of projects I contr
https://github.com/vbvictor updated
https://github.com/llvm/llvm-project/pull/149739
>From cb439cc703699279f690d6a4d0497a60ebfa4c08 Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Mon, 21 Jul 2025 00:06:26 +0300
Subject: [PATCH 1/2] [clang-tidy] Add parallel execution by default in
'run-cl
@@ -12251,6 +12264,9 @@ bool Sema::CheckFunctionDeclaration(Scope *S,
FunctionDecl *NewFD,
if (NewFD->hasAttr())
SYCL().CheckSYCLEntryPointFunctionDecl(NewFD);
+ if (NewFD->hasAttr())
+SYCL().CheckSYCLExternalFunctionDecl(NewFD);
erichkeane wrote:
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/149589
>From 7ecaad5127246bce4a836d2bbf419f062d170b85 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Fri, 18 Jul 2025 13:59:44 -0700
Subject: [PATCH] [clang-doc] integrate JSON as the source for Mustache
templates
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jordan Rupprecht (rupprecht)
Changes
`libomp` is the default value when unconfigured in cmake, but llvm can be
configured to have `libgomp` be the default instead. Explicitly specify this
value so the test does not fail when it assumes li
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Jordan Rupprecht (rupprecht)
Changes
`libomp` is the default value when unconfigured in cmake, but llvm can be
configured to have `libgomp` be the default instead. Explicitly specify this
value so the test does not fail when it ass
https://github.com/rupprecht created
https://github.com/llvm/llvm-project/pull/150301
`libomp` is the default value when unconfigured in cmake, but llvm can be
configured to have `libgomp` be the default instead. Explicitly specify this
value so the test does not fail when it assumes libomp is
@@ -12251,6 +12264,9 @@ bool Sema::CheckFunctionDeclaration(Scope *S,
FunctionDecl *NewFD,
if (NewFD->hasAttr())
SYCL().CheckSYCLEntryPointFunctionDecl(NewFD);
+ if (NewFD->hasAttr())
+SYCL().CheckSYCLExternalFunctionDecl(NewFD);
tahonermann wrote:
https://github.com/evelez7 edited
https://github.com/llvm/llvm-project/pull/149589
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/149588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Erick Velez
Date: 2025-07-23T12:53:07-07:00
New Revision: 4db2f3ac89b1e62af4893b647d77f3ab1f390066
URL:
https://github.com/llvm/llvm-project/commit/4db2f3ac89b1e62af4893b647d77f3ab1f390066
DIFF:
https://github.com/llvm/llvm-project/commit/4db2f3ac89b1e62af4893b647d77f3ab1f390066.diff
L
https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/150290
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Piotr Zegar
Date: 2025-07-23T21:51:58+02:00
New Revision: 478130545bc41a8bb80304e5d931559a9d2b6171
URL:
https://github.com/llvm/llvm-project/commit/478130545bc41a8bb80304e5d931559a9d2b6171
DIFF:
https://github.com/llvm/llvm-project/commit/478130545bc41a8bb80304e5d931559a9d2b6171.diff
L
llvmbot wrote:
@llvm/pr-subscribers-clangir
Author: Amr Hesham (AmrDeveloper)
Changes
This change adds support for CK_LValueToRValueBitCast for ComplexType
https://github.com/llvm/llvm-project/issues/141365
---
Full diff: https://github.com/llvm/llvm-project/pull/150296.diff
2 Files Af
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Amr Hesham (AmrDeveloper)
Changes
This change adds support for CK_LValueToRValueBitCast for ComplexType
https://github.com/llvm/llvm-project/issues/141365
---
Full diff: https://github.com/llvm/llvm-project/pull/150296.diff
2 Files Affe
https://github.com/PiotrZSL updated
https://github.com/llvm/llvm-project/pull/150290
>From 76f481c6471f71151855082360416078c14675ea Mon Sep 17 00:00:00 2001
From: Piotr Zegar
Date: Wed, 23 Jul 2025 19:11:28 +
Subject: [PATCH 1/2] [clang-tidy] Ignore pure-virtual in
portability-template...
vbvictor wrote:
> Also I think `clang-tidy-diff.py` should refer to Clang-Tidy. Several places
> are affected.
Do you mean renaming `clang-tidy` -> `Clang-Tidy` in scripts output/docs?
https://github.com/llvm/llvm-project/pull/149739
___
cfe-commits
evelez7 wrote:
### Merge activity
* **Jul 23, 7:51 PM UTC**: A user started a stack merge that includes this pull
request via
[Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/149588).
https://github.com/llvm/llvm-project/pull/149588
https://github.com/AmrDeveloper created
https://github.com/llvm/llvm-project/pull/150296
This change adds support for CK_LValueToRValueBitCast for ComplexType
https://github.com/llvm/llvm-project/issues/141365
>From 670aae7394804492de6c86b6068a650391de7004 Mon Sep 17 00:00:00 2001
From: AmrDev
@@ -12937,6 +12937,10 @@ bool ASTContext::DeclMustBeEmitted(const Decl *D) {
if (D->hasAttr())
return false;
+ if (LangOpts.SYCLIsDevice && !D->hasAttr() &&
+ !D->hasAttr())
erichkeane wrote:
An additional thought after clicking 'start-review': P
https://github.com/5chmidti approved this pull request.
https://github.com/llvm/llvm-project/pull/150290
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/150290
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vbvictor approved this pull request.
LGTM with nit
https://github.com/llvm/llvm-project/pull/150290
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/150290
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -171,3 +171,20 @@ struct NoInstantiation{
};
};
} // namespace PartialSpecializationNoInstantiation
+
+namespace PR139031 {
vbvictor wrote:
nit: IMO `PR-` or `GH-XXX` namespaces just bloat code without any benefits,
`git blame` can do the job.
And
@@ -12937,6 +12937,10 @@ bool ASTContext::DeclMustBeEmitted(const Decl *D) {
if (D->hasAttr())
return false;
+ if (LangOpts.SYCLIsDevice && !D->hasAttr() &&
+ !D->hasAttr())
tahonermann wrote:
Your suggested change would not be correct because it
https://github.com/qinkunbao edited
https://github.com/llvm/llvm-project/pull/150294
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 452 matches
Mail list logo
