llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Chris B (llvm-beanz)
Changes
This updates the DXV and Metal Converter actions to properly use temporary
files created by the driver. I've abstracted away a check to determine if an
action is the last in the sequence because we may have be
https://github.com/MacDue approved this pull request.
https://github.com/llvm/llvm-project/pull/116761
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ojhunt wrote:
I'm planning on removing the docs, but I've left them in for now just so that
reviewers can read them if they want context to understand the PR.
@cor3ntin @AaronBallman @erichkeane I'm ok removing the flag, I'll look into
how to diagnose the "you're using an extension/future lang
https://github.com/spall converted_to_draft
https://github.com/llvm/llvm-project/pull/129939
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/momo5502 updated
https://github.com/llvm/llvm-project/pull/128866
>From bb4091d2f9b7062aa83e5bee2ba525478a7dbd0a Mon Sep 17 00:00:00 2001
From: Maurice Heumann
Date: Wed, 26 Feb 2025 14:31:47 +0100
Subject: [PATCH 1/7] Instantiate destructors from initialized anonymous union
Xazax-hun wrote:
> But at this time point, there are no other sub project use it, which means it
> is not a common part.
Using this logic we would almost never move anything to reusable libraries. In
LLVM, we tend to introduce reusable componantes/functionality a lot. As far as
I remember, wh
Xazax-hun wrote:
Also, I already know about downstream users who are also interested in using
this option in their tools.
https://github.com/llvm/llvm-project/pull/128150
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/
@@ -5037,6 +5037,66 @@ def ArithmeticFence : LangBuiltin<"ALL_LANGUAGES"> {
let Prototype = "void(...)";
}
+class SNUWTemplate :
+ Template<["int", "long int", "long long int"],
+ ["_nuw", "l_nuw", "ll_nuw"]>;
philnik777 wrote:
I'm
https://github.com/steakhal approved this pull request.
https://github.com/llvm/llvm-project/pull/129697
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2269,10 +2269,11 @@ StmtResult Sema::ActOnForStmt(SourceLocation ForLoc,
SourceLocation LParenLoc,
for (auto *DI : DS->decls()) {
if (VarDecl *VD = dyn_cast(DI)) {
VarDeclSeen = true;
- if (VD->isLocalVarDecl() && !VD->hasLocalStorage()) {
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-aarch64-linux-bootstrap-msan` running on `sanitizer-buildbot9` while
building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/94/builds/5061
Here is the releva
HerrCai0907 wrote:
* **#130417** https://app.graphite.dev/github/pr/llvm/llvm-project/130417?utm_source=stack-comment-icon";
target="_blank">https://static.graphite.dev/graphite-32x32-black.png"; alt="Graphite"
width="10px" height="10px"/>
* **#130416** https://app.graphite.dev/github/pr/llvm/l
https://github.com/HerrCai0907 ready_for_review
https://github.com/llvm/llvm-project/pull/130415
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Congcong Cai (HerrCai0907)
Changes
The old `constructFrom` has hidden requirement which TypeMatcher must be used
before ArgumentMatcher because there are bind inside.
Inlining this function to make it more intuitive.
---
Full diff: h
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Leslie (wsehjk)
Changes
This pr is to fix #92342.
---
Full diff: https://github.com/llvm/llvm-project/pull/130400.diff
2 Files Affected:
- (modified) clang/lib/Sema/SemaChecking.cpp (+6-18)
- (modified) clang/test/Sema/constant_builti
HerrCai0907 wrote:
Hello @DeNiCoN, Are you still working on this patch? If not, I hope I can take
over this patch to finally finish it.
https://github.com/llvm/llvm-project/pull/123734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/130415
The old `constructFrom` has hidden requirement which TypeMatcher must be used
before ArgumentMatcher because there are bind inside.
Inlining this function to make it more intuitive.
>From ca39210f6a28569116
steakhal wrote:
Hi, I'm not sure where to move your addition in the ReleaseNotes, but I'm
pretty sure it shouldn't be in the Static Analyzer section.
https://github.com/llvm/llvm-project/pull/125384
___
cfe-commits mailing list
cfe-commits@lists.llvm.
https://github.com/HerrCai0907 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/129406
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
uweigand wrote:
> * However, if I removed the check for COMPILER_RT_HAS_FLOAT16 in the source
> files, it builds. Not sure why/if that is needed there (there is no check for
> it in e.g. extendhfdf2.c), or how it could be set.
This is set if the host compiler used to build compiler-rt supports
uweigand wrote:
> For the missing libfunctions, I see these files in compiler-rt/lib/builtins:
>
> ```
> compiler-rt/lib/builtins/extendhfsf2.c
> compiler-rt/lib/builtins/extendhftf2.c
> compiler-rt/lib/builtins/extendhfxf2.c
> ```
>
> For double, I get undefined reference to `__extendhfdf2' fr
https://github.com/steakhal created
https://github.com/llvm/llvm-project/pull/130418
This PR splits the existing modeling of builtin assume from the
BuiltinFunctionChecker.
We just sink the execution path if we are about to leave the assume expression
with a false assumption.
Assumptions with
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balazs Benics (steakhal)
Changes
This PR splits the existing modeling of builtin assume from the
BuiltinFunctionChecker.
We just sink the execution path if we are about to leave the assume expression
with a false assump
steakhal wrote:
This patch should conclude the assume attribute modeling. After this, we would
have some basic support for them.
https://github.com/llvm/llvm-project/pull/130418
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-x86_64-linux-bootstrap-asan` running on `sanitizer-buildbot1` while
building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/52/builds/6640
Here is the relevan
@@ -5037,6 +5037,66 @@ def ArithmeticFence : LangBuiltin<"ALL_LANGUAGES"> {
let Prototype = "void(...)";
}
+class SNUWTemplate :
+ Template<["int", "long int", "long long int"],
+ ["_nuw", "l_nuw", "ll_nuw"]>;
+
+class SNSWTemplate :
+ Template<["in
@@ -5037,6 +5037,66 @@ def ArithmeticFence : LangBuiltin<"ALL_LANGUAGES"> {
let Prototype = "void(...)";
}
+class SNUWTemplate :
philnik777 wrote:
Can we just make these builtins generic instead? There are like a dozen builtin
families that first only got
Carlos =?utf-8?q?Gálvez?= ,
Carlos =?utf-8?q?Gálvez?=
Message-ID:
In-Reply-To:
HerrCai0907 wrote:
> > The current patch is still unstable in my opinion.
>
> If that is the case that is an even stronger argument to not land it but let
> it cook a bit more.
>
I hope it can be land and then m
Xazax-hun wrote:
My proposal for this change is in `MatchASTConsumer` behind an off by default
option and make clang tidy turn this on by default. Open a ticket about not
considering the indirect field declarations top level. Properly update the
documentation and the changelog. I think this ap
AmrDeveloper wrote:
> I have mixed feelings about this change. It mostly overlaps with what I am
> working on, which is class `LexicalScope`, which has a bunch of the code for
> handling the return value. I think I would prefer that this PR be dropped, as
> the changes will be part of the PR I
Endilll wrote:
I think the failure in question is
```
==
FAIL: test_file_eq (tests.cindex.test_file.TestFile.test_file_eq)
--
Traceback (most recent call last):
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
llvm has recently started to use `__builitn_memchr` at compile time, so
implement this. Still needs some work but the basics are done.
---
Full diff: https://github.com/llvm/llvm-project/pull/130420.diff
2
https://github.com/kammce updated
https://github.com/llvm/llvm-project/pull/129459
>From 103a6843690182640d661e71517faf693c9e1c7d Mon Sep 17 00:00:00 2001
From: Khalil Estell
Date: Sat, 1 Mar 2025 14:29:42 -0800
Subject: [PATCH 1/3] [clangd] Add `BuiltinHeaders` flag to Config file
Usage:
```
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
101 - 134 of 134 matches
Mail list logo
