https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/106714
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/106826
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-31T08:06:06+02:00
New Revision: e4cf0a047d30fcd6cf606af0764883ca896a5153
URL:
https://github.com/llvm/llvm-project/commit/e4cf0a047d30fcd6cf606af0764883ca896a5153
DIFF:
https://github.com/llvm/llvm-project/commit/e4cf0a047d30fcd6cf606af0764883ca896a5153.diff
L
mizvekov wrote:
@DavidTruby @DavidSpickett A fix for this issue is up at
https://github.com/llvm/llvm-project/pull/106829
https://github.com/llvm/llvm-project/pull/100692
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
mizvekov wrote:
FYI @DavidTruby @DavidSpickett
https://github.com/llvm/llvm-project/pull/106829
___
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
Author: Matheus Izvekov (mizvekov)
Changes
This applies to function template non-call partial ordering the same
provisional wording change applied in the call context: Don't perform the
consistency check on return type and parameters which didn't
https://github.com/mizvekov created
https://github.com/llvm/llvm-project/pull/106829
This applies to function template non-call partial ordering the same
provisional wording change applied in the call context: Don't perform the
consistency check on return type and parameters which didn't have
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
The comment already explains that this is only true for _some_ builtin
functions. This also brings the time it takes me to run the
builtin-functions.cpp test locally from 50s down to 47s.
---
Full diff: htt
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/106826
The comment already explains that this is only true for _some_ builtin
functions. This also brings the time it takes me to run the
builtin-functions.cpp test locally from 50s down to 47s.
>From e38266cd0fa28e
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/106825
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-31T07:26:02+02:00
New Revision: 6f81c878ecd40acf1a0364e0ea5c9e6ea87407a2
URL:
https://github.com/llvm/llvm-project/commit/6f81c878ecd40acf1a0364e0ea5c9e6ea87407a2
DIFF:
https://github.com/llvm/llvm-project/commit/6f81c878ecd40acf1a0364e0ea5c9e6ea87407a2.diff
L
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/3304
Here is the relevant piece of the b
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/106825
>From fd47ce2d92865a5c9e5307478a10dd2c8b91ac44 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Sat, 31 Aug 2024 02:56:46 +0200
Subject: [PATCH] [clang][bytecode][NFC] Implement
MemberPointer
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/106825.diff
3 Files Affected:
- (modified) clang/lib/AST/ByteCode/Interp.h (+2-2)
- (modified) clang/lib/AST/ByteCode/MemberPointer.h (+6-6)
- (m
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/106825
None
>From 7d3a2ba57c5eefe47c2ccc57584881efdef41262 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Sat, 31 Aug 2024 02:56:46 +0200
Subject: [PATCH] [clang][bytecode][NFC] Implement
MemberP
https://github.com/owenca milestoned
https://github.com/llvm/llvm-project/pull/106662
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/106734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-31T06:24:36+02:00
New Revision: 360e4abfc8c7298283041e8f5a07f1829a888d18
URL:
https://github.com/llvm/llvm-project/commit/360e4abfc8c7298283041e8f5a07f1829a888d18
DIFF:
https://github.com/llvm/llvm-project/commit/360e4abfc8c7298283041e8f5a07f1829a888d18.diff
L
https://github.com/HighCommander4 approved this pull request.
https://github.com/llvm/llvm-project/pull/106354
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
c8ef wrote:
I'm curious about why Clang emits atomic-to-non-atomic and non-atomic-to-atomic
conversions when the type of the expression is atomic and the input is also
atomic.
(I'm not very familiar with C++ expression rules, so I may have missed
something in the standards.)
https://github.c
tbaederr wrote:
Of course not!
The reason that `AtomicToNonAtomic` is handled but `NonAtomicToAtomic` isn't,
is that this is in `IntExprEvaluator`, meaning the result must be a
(non-atomic) integer. If it was atomic, it'd be in `AtomicExprEvaluator`.
This AST for the LHS of the binary operato
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/106734
>From af84a7c4effc0372386b598bf323783d0e7fd132 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 30 Aug 2024 15:51:17 +0200
Subject: [PATCH] [clang][bytecode] Diagnose comparisons with lit
@@ -324,9 +329,14 @@ def add_release_notes(module_path, module, check_name,
description):
# Adds a test for the check.
-def write_test(module_path, module, check_name, test_extension, test_standard):
-if test_standard:
-test_standard = f"-std={test_standard}-or-l
https://github.com/nicovank edited
https://github.com/llvm/llvm-project/pull/106801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/nicovank updated
https://github.com/llvm/llvm-project/pull/106801
>From 6d4241a1858a0c46ceedf3034191a606af2e3941 Mon Sep 17 00:00:00 2001
From: Nicolas van Kempen
Date: Fri, 30 Aug 2024 23:18:37 -0400
Subject: [PATCH] [clang-tidy] Add type annotations to add_new_check.py, fix
https://github.com/nicovank updated
https://github.com/llvm/llvm-project/pull/106801
>From 6d4241a1858a0c46ceedf3034191a606af2e3941 Mon Sep 17 00:00:00 2001
From: Nicolas van Kempen
Date: Fri, 30 Aug 2024 23:18:37 -0400
Subject: [PATCH] [clang-tidy] Add type annotations to add_new_check.py, fix
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`clang-x86_64-linux-abi-test` running on `sie-linux-worker2` while building
`clang` at step 7 "abi-test-suite".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/8/builds/3716
Here is the relevant piece
https://github.com/shafik commented:
@tbaederr I am not confident this is the right approach.
If we compare the non-statement-expression case: https://godbolt.org/z/vseWeh5jP
We don't have the same issue as the original case:
https://godbolt.org/z/Gh13WcMEz
So I suspect there is something abo
https://github.com/llvm-beanz updated
https://github.com/llvm/llvm-project/pull/106811
>From 4abb34b1b21e666e20a9b3bad8af146d3bf322f0 Mon Sep 17 00:00:00 2001
From: Chris Bieneman
Date: Fri, 30 Aug 2024 23:54:41 +
Subject: [PATCH 1/2] Fix implicit conversion rank ordering
DXC prefers dimen
@@ -722,7 +722,8 @@ static void ValidateMultipleRegisterAnnotations(Sema &S,
Decl *TheDecl,
}
static void DiagnoseHLSLRegisterAttribute(Sema &S, SourceLocation &ArgLoc,
- Decl *TheDecl, RegisterType regType)
{
+
@@ -795,6 +796,9 @@ static void DiagnoseHLSLRegisterAttribute(Sema &S,
SourceLocation &ArgLoc,
S.Diag(ArgLoc, diag::warn_hlsl_deprecated_register_type_b);
else if (regType != RegisterType::C)
S.Diag(ArgLoc, diag::err_hlsl_binding_type_mismatch) << regType
@@ -817,8 +821,12 @@ static void DiagnoseHLSLRegisterAttribute(Sema &S,
SourceLocation &ArgLoc,
S.Diag(TheDecl->getLocation(),
diag::warn_hlsl_user_defined_type_missing_member)
<< regTypeNum;
-
-return;
+// Space argument cannot be specifie
@@ -722,7 +722,8 @@ static void ValidateMultipleRegisterAnnotations(Sema &S,
Decl *TheDecl,
}
static void DiagnoseHLSLRegisterAttribute(Sema &S, SourceLocation &ArgLoc,
- Decl *TheDecl, RegisterType regType)
{
+
https://github.com/damyanp requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/106782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
c8ef wrote:
```
| # \|
/var/lib/buildkite-agent/builds/linux-56-59b8f5d88-c4bpt-1/llvm-project/github-pull-requests/libcxx/test/std/utilities/charconv/charconv.to.chars/integral.pass.cpp:25:64:
error: declaration of 'uint64_t' must be imported from module 'std_cstdint'
before it is required
llvmbot wrote:
/pull-request llvm/llvm-project#106819
https://github.com/llvm/llvm-project/pull/106654
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
/cherry-pick e0f2368cdeb7312973a92fb2d22199d1de540db8
https://github.com/llvm/llvm-project/pull/106654
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca closed
https://github.com/llvm/llvm-project/pull/106654
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Owen Pan
Date: 2024-08-30T19:23:45-07:00
New Revision: e0f2368cdeb7312973a92fb2d22199d1de540db8
URL:
https://github.com/llvm/llvm-project/commit/e0f2368cdeb7312973a92fb2d22199d1de540db8
DIFF:
https://github.com/llvm/llvm-project/commit/e0f2368cdeb7312973a92fb2d22199d1de540db8.diff
LOG:
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/106680
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/c8ef edited https://github.com/llvm/llvm-project/pull/106817
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/c8ef updated https://github.com/llvm/llvm-project/pull/106751
>From 6ce4604725d36afaea17cf533d422a978c4389ff Mon Sep 17 00:00:00 2001
From: c8ef
Date: Fri, 30 Aug 2024 15:34:34 +
Subject: [PATCH] fix cast
---
clang/lib/AST/ExprConstant.cpp | 2 +-
clang/test/AST/atomic-
c8ef wrote:
The libc++ test also appears to be unrelated.
https://github.com/llvm/llvm-project/pull/106751
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rjmccall approved this pull request.
Thanks, LGTM.
https://github.com/llvm/llvm-project/pull/101083
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/python3kgae updated
https://github.com/llvm/llvm-project/pull/101240
>From 65b4ab94bc533c8dee9733761947671a4d326e90 Mon Sep 17 00:00:00 2001
From: Xiang Li
Date: Tue, 30 Jul 2024 16:34:40 -0400
Subject: [PATCH 1/7] [HLSL] AST support for WaveSize attribute.
First step for su
MaskRay wrote:
> If a function is declared without a prototype (pre-C23), it doesn't mean it
> has a void argument list; it means the argument list is undeclared. So you
> can't just assume the argument list is void; you have to compute the argument
> types from the actual arguments passed to
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: None (c8ef)
Changes
Fixes #106761.
---
Full diff: https://github.com/llvm/llvm-project/pull/106817.diff
4 Files Affected:
- (modified) clang/lib/Tooling/Refactoring/AtomicChange.cpp (+4-4)
- (modified) clang/tools/clang-format/C
https://github.com/c8ef created https://github.com/llvm/llvm-project/pull/106817
Fixes #106761.
>From dc9e84136e33c6ab35b1263382b740ff9a9dcd68 Mon Sep 17 00:00:00 2001
From: c8ef
Date: Sat, 31 Aug 2024 09:30:49 +0800
Subject: [PATCH] fix formating typo
---
clang/lib/Tooling/Refactoring/Atomic
https://github.com/damyanp approved this pull request.
https://github.com/llvm/llvm-project/pull/106811
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2764,10 +2771,59 @@ void AsmPrinter::emitJumpTableInfo() {
for (const MachineBasicBlock *MBB : JTBBs)
emitJumpTableEntry(MJTI, MBB, JTI);
}
+
+ if (EmitJumpTableSizesSection)
+emitJumpTableSizesSection(MJTI, F);
+
if (!JTInDiffSection)
OutStreamer->
@@ -2764,10 +2771,59 @@ void AsmPrinter::emitJumpTableInfo() {
for (const MachineBasicBlock *MBB : JTBBs)
emitJumpTableEntry(MJTI, MBB, JTI);
}
+
+ if (EmitJumpTableSizesSection)
+emitJumpTableSizesSection(MJTI, F);
+
if (!JTInDiffSection)
OutStreamer->
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/106734
>From 0786e92abfceb279ad79b3da3c33b3c645aa8754 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 30 Aug 2024 15:51:17 +0200
Subject: [PATCH] [clang][bytecode] Diagnose comparisons with lit
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/106734
>From bbf8b0f42845ada6ddb3935d975b06f9044b3c10 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 30 Aug 2024 15:51:17 +0200
Subject: [PATCH] [clang][bytecode] Diagnose comparisons with lit
https://github.com/c8ef updated https://github.com/llvm/llvm-project/pull/106751
>From 6ce4604725d36afaea17cf533d422a978c4389ff Mon Sep 17 00:00:00 2001
From: c8ef
Date: Fri, 30 Aug 2024 15:34:34 +
Subject: [PATCH] fix cast
---
clang/lib/AST/ExprConstant.cpp | 2 +-
clang/test/AST/atomic-
c8ef wrote:
I'm not sure why the `ELF/avr-reloc.s` test failed. I will attempt to update
the branch.
https://github.com/llvm/llvm-project/pull/106751
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Chris B (llvm-beanz)
Changes
DXC prefers dimension-preserving conversions over precision-losing conversions.
This means a double4 -> float4 conversion is preferred over a double4 ->
double3 or double4 -> double conversion.
---
Full diff:
llvmbot wrote:
@llvm/pr-subscribers-hlsl
Author: Chris B (llvm-beanz)
Changes
DXC prefers dimension-preserving conversions over precision-losing conversions.
This means a double4 -> float4 conversion is preferred over a double4 ->
double3 or double4 -> double conversion.
---
Full diff:
https://github.com/llvm-beanz created
https://github.com/llvm/llvm-project/pull/106811
DXC prefers dimension-preserving conversions over precision-losing conversions.
This means a double4 -> float4 conversion is preferred over a double4 ->
double3 or double4 -> double conversion.
>From 4abb34
@@ -3978,7 +3978,9 @@ def warn_acquired_before : Warning<
def warn_acquired_before_after_cycle : Warning<
"cycle in acquired_before/after dependencies, starting with '%0'">,
InGroup, DefaultIgnore;
-
+def warn_attribute_mismatch : Warning<
+ "attribute mismatch between fun
@@ -2260,6 +2260,37 @@ static bool neverReturns(const CFGBlock *B) {
return false;
}
+template
+static SmallVector collectAttrArgs(const FunctionDecl *FD) {
+ SmallVector Args;
+ for (const AttrT *A : FD->specific_attrs()) {
+for (const Expr *E : A->args())
+ Arg
https://github.com/aaronpuchert edited
https://github.com/llvm/llvm-project/pull/67520
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/aaronpuchert commented:
This probably fits better in `Sema/SemaDeclAttr.cpp`. Maybe
`checkLockFunAttrCommon` is a good place. Add a test to
`clang/test/Sema/attr-capabilities.c{,pp}`.
https://github.com/llvm/llvm-project/pull/67520
__
https://github.com/dfukalov created
https://github.com/llvm/llvm-project/pull/106810
This is the second part.
Without these explicit includes, removing other headers, who implicitly include
llvm-config.h, may have non-trivial side effects. For example, `clagd` may
report even `llvm-config.h`
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/106714
>From b04dc6d69039eeabe177cc5d109970734e0c29ca Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 30 Aug 2024 13:45:37 +0200
Subject: [PATCH] [clang][bytecode] Implement __noop
This does n
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/106745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-31T01:50:59+02:00
New Revision: 3745a2e8ab10029f8f401f5ff3c3c76c12e94822
URL:
https://github.com/llvm/llvm-project/commit/3745a2e8ab10029f8f401f5ff3c3c76c12e94822
DIFF:
https://github.com/llvm/llvm-project/commit/3745a2e8ab10029f8f401f5ff3c3c76c12e94822.diff
L
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/106714
>From 0423c99a3b539e20c807ae4a773521a5ad5063c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 30 Aug 2024 13:45:37 +0200
Subject: [PATCH] [clang][bytecode] Implement __noop
This does n
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/106734
>From 099ae16b3e41392f8fde9d4c92b4af48486bd027 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 30 Aug 2024 15:51:17 +0200
Subject: [PATCH] [clang][bytecode] Diagnose comparisons with lit
aaronpuchert wrote:
> Maybe I misunderstood the code of this PR, but I thought it only checks
> definitions.
That's my understanding as well. I was just thinking about the usual
requirement of putting the attribute on the first declaration (like the
standard mandates it for `[[noreturn]]`). U
plotfi wrote:
@justinfargnoli Is there anything I can do to help with moving this forward,
what seems to be the blocker on this at this point (asking to re-attain
context)?
https://github.com/llvm/llvm-project/pull/68176
___
cfe-commits mailing list
@@ -1827,15 +1831,13 @@ Expected FunctionImporter::importFunctions(
if (Error Err = SrcModule->materializeMetadata())
return std::move(Err);
-auto &ImportGUIDs = FunctionsToImportPerModule->second;
-
// Find the globals to import
SetVector GlobalsToImpo
@@ -215,13 +215,49 @@ class FunctionImporter {
SmallVector getSourceModules() const;
std::optional
-getImportType(const FunctionsToImportTy &GUIDToImportType,
- GlobalValue::GUID GUID) const;
+getImportType(StringRef FromModule, GlobalValue::GU
https://github.com/kazutakahirata updated
https://github.com/llvm/llvm-project/pull/106772
>From 2013008e1e0cf7df73b735d1ecfaf927b49542fd Mon Sep 17 00:00:00 2001
From: Kazu Hirata
Date: Thu, 29 Aug 2024 09:44:16 -0700
Subject: [PATCH 1/2] [LTO] Reduce memory usage for import lists
This patch
yabinc wrote:
> It probably makes sense to add a helper like the following to CodeGenModule
> or something like that:
>
> ```
> bool shouldZeroInitPadding() {
> // Comment explaining reasoning for this rule
> return !CGM.getLangOpts().CPlusPlus;
> }
> ```
>
> So we don't have `// See c
https://github.com/petrhosek closed
https://github.com/llvm/llvm-project/pull/106806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Petr Hosek
Date: 2024-08-30T15:13:23-07:00
New Revision: 8b77aa990b5f2f75ea7128a87bdbb8b905162e90
URL:
https://github.com/llvm/llvm-project/commit/8b77aa990b5f2f75ea7128a87bdbb8b905162e90
DIFF:
https://github.com/llvm/llvm-project/commit/8b77aa990b5f2f75ea7128a87bdbb8b905162e90.diff
LO
https://github.com/michaelrj-google approved this pull request.
LGTM from libc side
https://github.com/llvm/llvm-project/pull/106806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Prabhuk approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/106806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bob80905 updated
https://github.com/llvm/llvm-project/pull/106782
>From 99408f31a8946df7ef9efa223d0dba2ab876fcd0 Mon Sep 17 00:00:00 2001
From: Joshua Batista
Date: Fri, 30 Aug 2024 12:08:37 -0700
Subject: [PATCH 1/3] add diag and testing for space and global constants
---
@@ -215,13 +215,49 @@ class FunctionImporter {
SmallVector getSourceModules() const;
std::optional
-getImportType(const FunctionsToImportTy &GUIDToImportType,
- GlobalValue::GUID GUID) const;
+getImportType(StringRef FromModule, GlobalValue::GU
@@ -1827,15 +1831,13 @@ Expected FunctionImporter::importFunctions(
if (Error Err = SrcModule->materializeMetadata())
return std::move(Err);
-auto &ImportGUIDs = FunctionsToImportPerModule->second;
-
// Find the globals to import
SetVector GlobalsToImpo
llvmbot wrote:
@llvm/pr-subscribers-libc
Author: Petr Hosek (petrhosek)
Changes
This addresses issue introduced in #105718.
---
Full diff: https://github.com/llvm/llvm-project/pull/106806.diff
2 Files Affected:
- (modified) clang/cmake/caches/Fuchsia-stage2.cmake (+2-2)
- (modified) l
https://github.com/petrhosek created
https://github.com/llvm/llvm-project/pull/106806
This addresses issue introduced in #105718.
>From ef70fea3ffa9fc1787c4f8aa1fde591adda5fd6b Mon Sep 17 00:00:00 2001
From: Petr Hosek
Date: Fri, 30 Aug 2024 14:57:29 -0700
Subject: [PATCH] [libc] Use correct n
@@ -11427,6 +11427,19 @@ static void AnalyzeImplicitConversions(
return;
}
+ if (auto *OutArgE = dyn_cast(E)) {
+// The base expression is only used to initialize the parameter for
+// arguments to `inout` parameters, so we only traverse down the base
+// ex
@@ -5915,6 +5916,16 @@ bool Sema::GatherArgumentsForCall(SourceLocation
CallLoc, FunctionDecl *FDecl,
ProtoArgType->isBlockPointerType())
if (auto *BE = dyn_cast(Arg->IgnoreParenNoopCasts(Context)))
BE->getBlockDecl()->setDoesNotEscape();
+ //
@@ -5432,6 +5434,57 @@ LValue CodeGenFunction::EmitOpaqueValueLValue(const
OpaqueValueExpr *e) {
return getOrCreateOpaqueLValueMapping(e);
}
+LValue CodeGenFunction::BindHLSLOutArgExpr(const HLSLOutArgExpr *E,
+ Address OutTemp) {
+
@@ -285,6 +285,13 @@ class CallArgList : public SmallVector {
/// A value to "use" after the writeback, or null.
llvm::Value *ToUse;
+
+/// An Expression representing a cast from the temporary's type to the
+/// source l-value's type.
+const Expr *CastExpr;
@@ -7071,6 +7071,102 @@ class ArraySectionExpr : public Expr {
void setRBracketLoc(SourceLocation L) { RBracketLoc = L; }
};
+/// This class represents temporary values used to represent inout and out
+/// arguments in HLSL. From the callee perspective these parameters are m
@@ -7071,6 +7071,102 @@ class ArraySectionExpr : public Expr {
void setRBracketLoc(SourceLocation L) { RBracketLoc = L; }
};
+/// This class represents temporary values used to represent inout and out
+/// arguments in HLSL. From the callee perspective these parameters are m
@@ -7071,6 +7071,102 @@ class ArraySectionExpr : public Expr {
void setRBracketLoc(SourceLocation L) { RBracketLoc = L; }
};
+/// This class represents temporary values used to represent inout and out
+/// arguments in HLSL. From the callee perspective these parameters are m
https://github.com/yabinc updated
https://github.com/llvm/llvm-project/pull/97121
>From d7e0361ed91032154aafee3cf2f9a7cdc233a395 Mon Sep 17 00:00:00 2001
From: Yabin Cui
Date: Thu, 27 Jun 2024 17:11:52 -0700
Subject: [PATCH 1/3] [clang][CodeGen] Zero init unspecified fields in
initializers in
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Nicolas van Kempen (nicovank)
Changes
```
python3 -m mypy --strict clang-tools-extra/clang-tidy/add_new_check.py
Success: no issues found in 1 source file
```
---
Full diff: https://github.com/llvm/llvm-project/pull/106801.diff
1
https://github.com/nicovank created
https://github.com/llvm/llvm-project/pull/106801
```
python3 -m mypy --strict clang-tools-extra/clang-tidy/add_new_check.py
Success: no issues found in 1 source file
```
>From 0213feed46b0cb7ba38410c26eba4d1d7226bd5b Mon Sep 17 00:00:00 2001
From: Nicolas v
@@ -143,19 +143,3 @@ float test_dot_float3(float3 p0, float3 p1) { return
dot(p0, p1); }
// CHECK: %dx.dot = call float @llvm.dx.dot4.v4f32(<4 x float> %0, <4 x float>
%1)
// CHECK: ret float %dx.dot
float test_dot_float4(float4 p0, float4 p1) { return dot(p0, p1); }
-
-// CH
https://github.com/llvm-beanz closed
https://github.com/llvm/llvm-project/pull/104474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Chris B
Date: 2024-08-30T16:18:46-05:00
New Revision: 02654f7370638889b989b4d776d35c3d47c87cdd
URL:
https://github.com/llvm/llvm-project/commit/02654f7370638889b989b4d776d35c3d47c87cdd
DIFF:
https://github.com/llvm/llvm-project/commit/02654f7370638889b989b4d776d35c3d47c87cdd.diff
LOG:
https://github.com/hekota closed
https://github.com/llvm/llvm-project/pull/106673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hekota wrote:
> Is there some other way to slice this so that there's a smaller part of your
> larger change that can actually be tested?
The thing is that we are already using resource attributes in many places, and
they all need to be updated when we switch to having the attributes on types
@@ -174,51 +174,33 @@ std::string llvm::computeLTOCacheKey(
for (auto GUID : ExportsGUID)
Hasher.update(ArrayRef((uint8_t *)&GUID, sizeof(GUID)));
- // Include the hash for every module we import functions from. The set of
- // imported symbols for each module may affe
@@ -480,6 +480,9 @@ struct RegisterBindingFlags {
bool ContainsNumeric = false;
bool DefaultGlobals = false;
+
+ // used only when Resource == true
+ llvm::dxil::ResourceClass ResourceClass = llvm::dxil::ResourceClass::UAV;
hekota wrote:
Yes, the indivi
1 - 100 of 456 matches
Mail list logo
