@@ -1078,22 +1091,65 @@ void
ModuleSanitizerCoverage::InjectCoverageAtBlock(Function &F, BasicBlock &BB,
Store->setNoSanitizeMetadata();
}
if (Options.StackDepth && IsEntryBB && !IsLeafFunc) {
-// Check stack depth. If it's the deepest so far, record it.
Modu
@@ -1078,22 +1091,65 @@ void
ModuleSanitizerCoverage::InjectCoverageAtBlock(Function &F, BasicBlock &BB,
Store->setNoSanitizeMetadata();
}
if (Options.StackDepth && IsEntryBB && !IsLeafFunc) {
-// Check stack depth. If it's the deepest so far, record it.
Modu
@@ -391,6 +391,7 @@ Improvements to Clang's diagnostics
- An error is now emitted when a ``musttail`` call is made to a function
marked with the ``not_tail_called`` attribute. (#GH133509).
+- The ``-mcmodel=tiny`` option will now be diagnosed on all targets other than
ARM o
owenca wrote:
> > 3. File has been formatted with clang-format, except for the grammar,
> > because clang-format butchers it.
>
> Yeah, this problem comes up rather frequently, I wonder if clang-format can
> get some additional smarts to make that less of a challenge? CC
> @mydeveloperday @ow
https://github.com/ShashwathiNavada updated
https://github.com/llvm/llvm-project/pull/125643
>From 0aebcd7119fbcd51154c5d9706752e8ff3f041bc Mon Sep 17 00:00:00 2001
From: ShashwathiNavada
Date: Tue, 4 Feb 2025 00:16:09 -0600
Subject: [PATCH 01/13] Adding diagnostics for unsupported option
---
https://github.com/zmodem updated
https://github.com/llvm/llvm-project/pull/138562
>From e221ba3b0f7b08bcfc56bf75f7505265c332637d Mon Sep 17 00:00:00 2001
From: Hans Wennborg
Date: Mon, 5 May 2025 20:24:15 +0200
Subject: [PATCH 1/2] [Sema] Warn about omitting deprecated enumerator in
switch
T
https://github.com/ShashwathiNavada edited
https://github.com/llvm/llvm-project/pull/125643
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ShashwathiNavada edited
https://github.com/llvm/llvm-project/pull/125643
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Raul Tambre
Date: 2025-05-06T11:19:57+03:00
New Revision: d66dbd6931a4358c0e4fd7c749179aa229fb36a4
URL:
https://github.com/llvm/llvm-project/commit/d66dbd6931a4358c0e4fd7c749179aa229fb36a4
DIFF:
https://github.com/llvm/llvm-project/commit/d66dbd6931a4358c0e4fd7c749179aa229fb36a4.diff
L
https://github.com/tambry closed
https://github.com/llvm/llvm-project/pull/138521
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tambry wrote:
/cherry-pick 58e6883c8b6e571d6bd774645ee2b6348cfed6ba
https://github.com/llvm/llvm-project/pull/138460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
/pull-request llvm/llvm-project#138652
https://github.com/llvm/llvm-project/pull/138460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tambry wrote:
Seems like the cherry-pick only works if the branch of the PR still exists.
:thinking:
https://github.com/llvm/llvm-project/pull/138460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
https://github.com/saksham-joshi updated
https://github.com/llvm/llvm-project/pull/138649
>From 8bba07d817d4dd583122d7ed831de276da8801e0 Mon Sep 17 00:00:00 2001
From: SAKSHAM JOSHI
Date: Tue, 6 May 2025 12:43:34 +0530
Subject: [PATCH] FEAT: one byte for "true" & "false"
In C programming langu
ChuanqiXu9 wrote:
hmmm for `test case`, I mean a test case in clang, either a unit test or a lit
test, so we can understand the effect on users more clearly.
And if you're talking about the general code policy, I'll suggest you to
https://discourse.llvm.org/ to make it more verbose.
https://
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/138293
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,san
@@ -4169,11 +4169,16 @@ static Value *emitPointerArithmetic(CodeGenFunction
&CGF,
// The index is not pointer-sized.
// The pointer type is not byte-sized.
//
- if (BinaryOperator::isNullPointerArithmeticExtension(CGF.getContext(),
-
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
https://github.com/saksham-joshi created
https://github.com/llvm/llvm-project/pull/138649
### 1 byte is enough for bool
- In C programming language,
- the size of **"true"** and **"false"** macros is 4 byte and size of macro
"bool" (typedef of _Bool) is 1 byte.
- By this simple change, we ca
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/138640
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: SAKSHAM JOSHI (saksham-joshi)
Changes
### 1 byte is enough for bool
- In C programming language,
- the size of **"true"** and **"false"** macros is 4 byte and size of macro
"bool" (typedef of _Bool) is 1 byte.
- By this simple cha
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: SAKSHAM JOSHI (saksham-joshi)
Changes
### 1 byte is enough for bool
- In C programming language,
- the size of **"true"** and **"false"** macros is 4 byte and size of macro
"bool" (typedef of _Bool) is 1 byte.
- By this simple change, w
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/137610
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tbaederr wrote:
@jmmartinez Ping.
https://github.com/llvm/llvm-project/pull/137306
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/guy-david edited
https://github.com/llvm/llvm-project/pull/136792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -15,7 +15,7 @@ enum SwitchTwo {
};
void testSwitchTwo(enum SwitchTwo st) {
- switch (st) {} // expected-warning{{enumeration values 'Vim' and 'Emacs' not
handled in switch}}
+ switch (st) {} // expected-warning{{enumeration values 'Ed', 'Vim' and
'Emacs' not handled in
https://github.com/balazske closed
https://github.com/llvm/llvm-project/pull/128735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Balázs Kéri
Date: 2025-05-06T10:17:26+02:00
New Revision: 4b30b3f901b00da1bd1f70e9ca9086e349c7cca0
URL:
https://github.com/llvm/llvm-project/commit/4b30b3f901b00da1bd1f70e9ca9086e349c7cca0
DIFF:
https://github.com/llvm/llvm-project/commit/4b30b3f901b00da1bd1f70e9ca9086e349c7cca0.diff
L
zmodem wrote:
I wonder if we should also do something special about the
`-Wdeprecated-declarations` diagnostic in switches.
If the user fixes the -Wswitch / -Wreturn-type warnings from `testSwitchFour`
the natural way:
```
int testSwitchFour(enum SwitchFour e) {
switch (e) {
case Red:
zmodem wrote:
> Fly-by comment: I drafted the following alternative comment in another
> thread. Please feel free to disregard as I don't know the tone and style of
> comments in these files. (But I appreciate your consideration for the
> contents in the message I'd like to see!)
>
> ```
>
https://github.com/k-arrows commented:
I am not a developer, so I won't review the implementation, but I have one
comment. The way I originally wrote the bug report was poor, but it is not
necessary to limit the fix and its test to OpenMP SIMD. For example, the code
below also crashes:
https:/
https://github.com/mgoudar updated
https://github.com/llvm/llvm-project/pull/134985
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-s
carlosgalvezp wrote:
> because the cases handled by it are covered by CaptureByRefExplicit.
Strange, can you explain why that's the case? Are we missing tests?
https://github.com/llvm/llvm-project/pull/138504
___
cfe-commits mailing list
cfe-commits@l
https://github.com/ArtSin updated
https://github.com/llvm/llvm-project/pull/136549
>From f5c8b4ea6539976e759e8fe183e009475d895052 Mon Sep 17 00:00:00 2001
From: Artem Sinkevich
Date: Mon, 21 Apr 2025 12:48:51 +0400
Subject: [PATCH] [sanitizer] Fix empty string in unsupported argument error
for
ayushpareek2003 wrote:
> I feel `.clear()` should have a similar semantic with `= {};`. It will be
> confusing for people to understand this. It will be helpful if you can
> provide a test to show the difference.
I’ve tried to demonstrate the difference between .clear() and = {}, not a
formal
https://github.com/ayushpareek2003 updated
https://github.com/llvm/llvm-project/pull/138256
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Aria
ayushpareek2003 wrote:
> hmmm for `test case`, I mean a test case in clang, either a unit test or a
> lit test, so we can understand the effect on users more clearly.
>
> And if you're talking about the general code policy, I'll suggest you to
> https://discourse.llvm.org/ to make it more verb
@@ -4169,11 +4169,16 @@ static Value *emitPointerArithmetic(CodeGenFunction
&CGF,
// The index is not pointer-sized.
// The pointer type is not byte-sized.
//
- if (BinaryOperator::isNullPointerArithmeticExtension(CGF.getContext(),
-
https://github.com/dtcxzyw updated
https://github.com/llvm/llvm-project/pull/137849
>From 5b3a9ed3ed2f258a178e0c17891e2d9ae4f21446 Mon Sep 17 00:00:00 2001
From: Yingwei Zheng
Date: Tue, 6 May 2025 17:47:07 +0800
Subject: [PATCH] [Clang][CodeGen] Enable pointer overflow check for GCC
workaroun
https://github.com/dtcxzyw edited
https://github.com/llvm/llvm-project/pull/137849
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/dtcxzyw edited
https://github.com/llvm/llvm-project/pull/137849
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DaanDeMeyer created
https://github.com/llvm/llvm-project/pull/138669
None
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Ar
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Daan De Meyer (DaanDeMeyer)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/138669.diff
2 Files Affected:
- (modified) clang-tools-extra/include-cleaner/lib/WalkAST.cpp (+5)
- (modified) clang-tools-extra/
https://github.com/jmorse closed
https://github.com/llvm/llvm-project/pull/111836
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jmorse wrote:
/me squints -- this was a minor improvement and I've lost all the context
behind it, abandoning as it's not worth revisiting IMO
https://github.com/llvm/llvm-project/pull/111836
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
https://github.com/balazske updated
https://github.com/llvm/llvm-project/pull/132242
From e3064b600ea726ab7b3dea054e9f11e1ce028297 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?=
Date: Wed, 19 Mar 2025 16:09:04 +0100
Subject: [PATCH 1/5] [clang-tidy] Add check
bugprone-misle
@@ -39,6 +39,29 @@ AST_MATCHER(clang::ParmVarDecl, isArgvOfMain) {
return FD ? FD->isMain() : false;
}
+AST_MATCHER(clang::TypeLoc, isInImplicitTemplateInstantiation) {
+ const auto IsImplicitTemplateInstantiation = [](const auto *Node) {
+return (Node != nullptr) &&
+
@@ -39,6 +39,29 @@ AST_MATCHER(clang::ParmVarDecl, isArgvOfMain) {
return FD ? FD->isMain() : false;
}
+AST_MATCHER(clang::TypeLoc, isInImplicitTemplateInstantiation) {
+ const auto IsImplicitTemplateInstantiation = [](const auto *Node) {
+return (Node != nullptr) &&
+
https://github.com/stmuench updated
https://github.com/llvm/llvm-project/pull/132924
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/138293
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,san
stmuench wrote:
@PiotrZSL would have any further remarks for this PR?
https://github.com/llvm/llvm-project/pull/132924
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tbaederr wrote:
That fixes compile-time computation, but this example still breaks in codegen:
```c++
#include
using namespace std;
constexpr size_t kMemoryChunk = 1024 * 8;
constexpr size_t kNumberOfIterations = 200;
constexpr size_t kThreadsNumber = 2 * kMemoryChunk;
struct S {};
int mai
@@ -0,0 +1,58 @@
+//===--- MisleadingSetterOfReferenceCheck.cpp -
clang-tidy-===//
+//
+// 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
Author: Paul Walker
Date: 2025-05-06T11:57:04+01:00
New Revision: cb9683fad12101417a46b35452cb23dfb7c6c367
URL:
https://github.com/llvm/llvm-project/commit/cb9683fad12101417a46b35452cb23dfb7c6c367
DIFF:
https://github.com/llvm/llvm-project/commit/cb9683fad12101417a46b35452cb23dfb7c6c367.diff
L
https://github.com/paulwalker-arm closed
https://github.com/llvm/llvm-project/pull/138288
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
fangyi-zhou wrote:
gentle ping
https://github.com/llvm/llvm-project/pull/137355
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/balazs-benics-sonarsource edited
https://github.com/llvm/llvm-project/pull/136720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/balazs-benics-sonarsource commented:
Looks good. There were two points unaddressed:
- Finding a name for the flag without the `legacy-` prefix
- Find out if we can ever have multiple root nodes in an exploded graph.
https://github.com/llvm/llvm-project/pull/136720
_
@@ -2523,6 +2523,20 @@ bool ExprEngine::replayWithoutInlining(ExplodedNode *N,
return true;
}
+/// Return the innermost location context which is inlined at `Node`, unless
+/// it's the top-level (entry point) location context.
+static const LocationContext *getInlinedLocati
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/127636
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-
https://github.com/DenisGZM updated
https://github.com/llvm/llvm-project/pull/133107
>From c46eda67cd7434dcce5c1f29125a940dc4ff64ba Mon Sep 17 00:00:00 2001
From: Denis Gerasimov
Date: Wed, 26 Mar 2025 18:29:38 +0300
Subject: [PATCH 1/8] [CLANG] Enable alignas after GNU attributes
---
clang/l
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/127636
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-arm-ubuntu` running
on `linaro-lldb-arm-ubuntu` while building `clang,flang` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/18/builds/15564
Here is the relevant piece of the bu
https://github.com/AaronBallman approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/138613
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/citymarina created
https://github.com/llvm/llvm-project/pull/138677
It was enabled for "aarch64", which covers arm64e but not arm64_32.
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-
@@ -4169,11 +4169,16 @@ static Value *emitPointerArithmetic(CodeGenFunction
&CGF,
// The index is not pointer-sized.
// The pointer type is not byte-sized.
//
- if (BinaryOperator::isNullPointerArithmeticExtension(CGF.getContext(),
-
@@ -2523,6 +2523,20 @@ bool ExprEngine::replayWithoutInlining(ExplodedNode *N,
return true;
}
+/// Return the innermost location context which is inlined at `Node`, unless
+/// it's the top-level (entry point) location context.
+static const LocationContext *getInlinedLocati
@@ -0,0 +1,45 @@
+// RUN: %clang_analyze_cc1 -analyzer-checker=cplusplus -verify %s
+// RUN: %clang_analyze_cc1 -analyzer-checker=cplusplus -verify %s -DEMPTY_CLASS
+
+// expected-no-diagnostics
+
+// This test reproduces the issue that previously the static analyzer
+// initializ
anutosh491 wrote:
/cherry-pick
https://github.com/llvm/llvm-project/commit/529b6fcb00aabbed17365e5fb3abbc2ae127c967
https://github.com/llvm/llvm-project/pull/138091
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bi
anutosh491 wrote:
Deleting the branch and cherry picking seems problematic
(https://github.com/llvm/llvm-project/pull/138460#issuecomment-2853718367)
hence I've restored it and tried cherry picking again !
https://github.com/llvm/llvm-project/pull/138091
___
Author: cor3ntin
Date: 2025-05-06T14:13:32+02:00
New Revision: 300d4026f77683aae490361d250a51904dd19ed2
URL:
https://github.com/llvm/llvm-project/commit/300d4026f77683aae490361d250a51904dd19ed2
DIFF:
https://github.com/llvm/llvm-project/commit/300d4026f77683aae490361d250a51904dd19ed2.diff
LOG:
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/127636
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tambry wrote:
@anutosh491 Yep, precisely. That's what I did here.
https://github.com/llvm/llvm-project/pull/138460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -12240,16 +12240,16 @@ class Sema final : public SemaBase {
bool PrevLastDiagnosticIgnored;
public:
-explicit SFINAETrap(Sema &SemaRef, bool AccessCheckingSFINAE = false)
+explicit SFINAETrap(Sema &SemaRef, bool TestWellformedSFINAE = false)
a
anutosh491 wrote:
I did the same and worked pretty smoothly. Thank you :)
https://github.com/llvm/llvm-project/pull/138460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zahiraam wrote:
May be an [NFC] in the title of the PR?
https://github.com/llvm/llvm-project/pull/138374
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tambry wrote:
Someone probably ought to open an issue for that issue.
https://github.com/llvm/llvm-project/pull/138460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
anutosh491 wrote:
> Someone probably ought to open an issue for this caveat.
I don't know who would be responsible for looking into this just yet :|
Can we raise something and tag someone responsible for this ?
https://github.com/llvm/llvm-project/pull/138460
__
https://github.com/zahiraam approved this pull request.
Looks good. Thanks.
https://github.com/llvm/llvm-project/pull/138374
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -12240,16 +12240,16 @@ class Sema final : public SemaBase {
bool PrevLastDiagnosticIgnored;
public:
-explicit SFINAETrap(Sema &SemaRef, bool AccessCheckingSFINAE = false)
+explicit SFINAETrap(Sema &SemaRef, bool TestWellformedSFINAE = false)
a
@@ -101,9 +103,11 @@ T tmain(T argc, T *argv) {
a = 2;
#pragma omp task default(none), private(argc, b) firstprivate(argv) shared(d)
if (argc > 0) final(S::TS > 0) priority(argc) affinity(argc, argv[b:argc],
arr[:], ([argc][sizeof(T)])argv)
foo();
+#ifndef OMP60
-
https://github.com/erichkeane updated
https://github.com/llvm/llvm-project/pull/138576
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,san
https://github.com/erichkeane approved this pull request.
What do we think @AaronBallman ? I think the diagnostic column is as best
effort as we are going to get, so I'm OK with this as-is. WE could perhaps
improve that, but I don't think doing that here is worth the effort.
I'll approve, bu
@@ -2056,9 +2056,21 @@ void NeonEmitter::createIntrinsic(const Record *R,
auto &Entry = IntrinsicMap[Name];
for (auto &I : NewTypeSpecs) {
+
+// MFloat8 type is only available on AArch64. If encountered set ArchGuard
+// correctly.
+std::string savedArchGuard =
@@ -5464,6 +5464,15 @@ RValue CodeGenFunction::EmitCall(const CGFunctionInfo
&CallInfo,
Builder.CreateStore(errorValue, swiftErrorTemp);
}
+// Mfloat8 type is loaded as scalar type, but is treated as single
+// vector type for other operation
NagyDonat wrote:
The commit looks promising, my only remaining remarks are about phrasing in the
documentation and comments.
https://github.com/llvm/llvm-project/pull/132242
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
https://github.com/saksham-joshi updated
https://github.com/llvm/llvm-project/pull/138649
>From 8bba07d817d4dd583122d7ed831de276da8801e0 Mon Sep 17 00:00:00 2001
From: SAKSHAM JOSHI
Date: Tue, 6 May 2025 12:43:34 +0530
Subject: [PATCH 1/2] FEAT: one byte for "true" & "false"
In C programming l
https://github.com/paulwalker-arm updated
https://github.com/llvm/llvm-project/pull/138262
>From 5da6f3e5ff07977294d82721c4bd37984daace47 Mon Sep 17 00:00:00 2001
From: Paul Walker
Date: Fri, 2 May 2025 12:41:06 +0100
Subject: [PATCH 1/2] Add test showing x86 functions when targeting AArch64.
paulwalker-arm wrote:
Re-review request after rebasing to pull in driver fix.
https://github.com/llvm/llvm-project/pull/138262
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -29,7 +29,7 @@ static cl::opt
ClVectorLibrary(
"Accelerate framework"),
clEnumValN(TargetLibraryInfoImpl::DarwinLibSystemM,
"Darwin_libsystem_m", "Darwin libsystem_m"),
- clEnumValN(TargetLibra
https://github.com/saksham-joshi updated
https://github.com/llvm/llvm-project/pull/138649
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,
https://github.com/alexey-bataev commented:
Need to update OpenMPSupport.rst and release notes
https://github.com/llvm/llvm-project/pull/134709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
@@ -4899,6 +4899,238 @@ void
CGOpenMPRuntime::emitSingleReductionCombiner(CodeGenFunction &CGF,
}
}
+void CGOpenMPRuntime::emitPrivateReduction(
alexey-bataev wrote:
1. Add the comments, describing the logic of the function.
2. How does it differ from the
https://github.com/alexey-bataev edited
https://github.com/llvm/llvm-project/pull/134709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/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
https://github.com/saksham-joshi created
https://github.com/llvm/llvm-project/pull/138713
# In C language,
- the size of "true" and "false" macro is 4 bytes (for x64 arch)
- but the size of "bool" (typedef _Bool) is 1 byte.
- In order to decrease the memory usage, we can set the size of "true
https://github.com/MacDue approved this pull request.
LGTM, thanks :+1:
https://github.com/llvm/llvm-project/pull/138262
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MacDue edited
https://github.com/llvm/llvm-project/pull/138262
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1360,8 +1360,15 @@ void
TargetLibraryInfoImpl::addVectorizableFunctionsFromVecLib(
addVectorizableFunctions(VecFuncs_DarwinLibSystemM);
break;
}
- case LIBMVEC_X86: {
-addVectorizableFunctions(VecFuncs_LIBMVEC_X86);
+ case LIBMVEC: {
+switch (TargetTrip
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: SAKSHAM JOSHI (saksham-joshi)
Changes
# In C language,
- the size of "true" and "false" macro is 4 bytes (for x64 arch)
- but the size of "bool" (typedef _Bool) is 1 byte.
- In order to decrease the memory usage, we can set the size of "
1 - 100 of 194 matches
Mail list logo
