Author: Kazu Hirata
Date: 2024-10-10T08:15:48-07:00
New Revision: 77c842f44cc06951975fd4a85761e0bc830d185a
URL:
https://github.com/llvm/llvm-project/commit/77c842f44cc06951975fd4a85761e0bc830d185a
DIFF:
https://github.com/llvm/llvm-project/commit/77c842f44cc06951975fd4a85761e0bc830d185a.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/111784
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Kazu Hirata
Date: 2024-10-10T08:16:51-07:00
New Revision: d2a96d170a4faa0a6c42fe5f23c073891d6118b8
URL:
https://github.com/llvm/llvm-project/commit/d2a96d170a4faa0a6c42fe5f23c073891d6118b8
DIFF:
https://github.com/llvm/llvm-project/commit/d2a96d170a4faa0a6c42fe5f23c073891d6118b8.diff
L
Author: Kazu Hirata
Date: 2024-10-10T08:17:25-07:00
New Revision: 670a4613fc5f29036f23fe357b0dbf017d019717
URL:
https://github.com/llvm/llvm-project/commit/670a4613fc5f29036f23fe357b0dbf017d019717
DIFF:
https://github.com/llvm/llvm-project/commit/670a4613fc5f29036f23fe357b0dbf017d019717.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/111785
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bricknerb created
https://github.com/llvm/llvm-project/pull/111856
https://eel.is/c++draft/class.virtual#8.1
This prevents overriding methods with non class return types that have less
cv-qualification.
Fixes: #111742
>From 786d31e2657964e578cd1fdf2006b0fb3b19fab6 Mon Sep
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
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Boaz Brickner (bricknerb)
Changes
https://eel.is/c++draft/class.virtual#8.1
This prevents overriding methods with non class return types that have less
cv-qualification.
Fixes: #111742
---
Full diff: https://github.com/llvm/llvm-projec
https://github.com/sdkrystian edited
https://github.com/llvm/llvm-project/pull/111852
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
⚠️ We detected that you are using a GitHub private e-mail address to contribute
to the repo. Please turn off [Keep my email addresses
private](https://github.com/settings/emails) setting in your account. See
[LLVM
Discourse](https://discourse.llvm.org/t/hidden-email
@@ -4493,28 +4493,34 @@ void CXXNameMangler::mangleType(const
ArrayParameterType *T) {
}
void CXXNameMangler::mangleType(const HLSLAttributedResourceType *T) {
- mangleType(T->getWrappedType());
+ llvm::SmallString<64> Str("_Res");
const HLSLAttributedResourceType::Attri
@@ -11533,6 +11539,18 @@ QualType ASTContext::mergeTypes(QualType LHS, QualType
RHS, bool OfBlockPointer,
return {};
return LHS;
}
+ case Type::HLSLAttributedResource: {
+const HLSLAttributedResourceType *LHSTy =
+LHS->castAs();
+const HLSLAttribut
https://github.com/frasercrmck updated
https://github.com/llvm/llvm-project/pull/109985
>From 454110573c6fcb472f5a87d3de31d2a0addd546d Mon Sep 17 00:00:00 2001
From: Fraser Cormack
Date: Mon, 23 Sep 2024 12:54:02 +0100
Subject: [PATCH 1/2] [libclc] Split off library build system into helpers
T
@@ -683,6 +706,59 @@ struct AAAMDAttributesFunction : public AAAMDAttributes {
return !A.checkForAllCallLikeInstructions(DoesNotRetrieve, *this,
UsedAssumedInformation);
}
+
+ // Returns true if FlatScratchInit is needed, i.e
topperc wrote:
> > > Personally I don't like to add a privileged extension if it doesn't
> > > introduce new CSRs/instructions.
> >
> >
> > I'd actually put that on the
> > [agenda](https://docs.google.com/document/d/1G3ocHm2zE6AYTS2N3_3w2UxFnSEyKkcF57siLWe-NVs/edit?tab=t.0)
> > for today's
@@ -439,6 +439,26 @@ struct AAAMDAttributesFunction : public AAAMDAttributes {
indicatePessimisticFixpoint();
return;
}
+
+SmallPtrSet VisitedConsts;
+
+for (Instruction &I : instructions(F)) {
arsenm wrote:
Should use checkForAllInstru
@@ -48,11 +48,13 @@ define <4 x i32> @load_static_array_of_vec_test(i32 %index)
{
}
; CHECK-LABEL: multid_load_test
-define <4 x i32> @multid_load_test() {
+define <4 x i32> @multid_load_test() #0 {
; CHECK-COUNT-8: load i32, ptr addrspace(3)
{{(.*@groushared2dArrayofVec
@@ -931,6 +931,10 @@ def FeatureStdExtSmepmp
: RISCVExtension<"smepmp", 1, 0,
"'Smepmp' (Enhanced Physical Memory Protection)">;
+def FeatureStdExtSmrnmi
+: RISCVExtension<"smrnmi", 1, 0,
+ "'Smrnmi' (Extension for Resumable Non
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 55d51dd9dca8220ffaf9260d56dae9f5c34b7120
786d31e2657964e578cd1fdf2006b0fb3b19fab6 --e
ldionne wrote:
@ilovepi Some things that *could* be going wrong: The `cxx_shared` target is
now defined when it wasn't before, and something in your CMake has e.g. `if
(TARGET cxx_shared)` and does something it shouldn't based on that.
Otherwise, it's also possible that `libc++.so` doesn't get
@@ -860,6 +860,16 @@ class RedeclarableTemplateDecl : public TemplateDecl,
/// \endcode
bool isMemberSpecialization() const { return Common.getInt(); }
+ /// Determines whether any redeclaration of this template was
+ /// a specialization of a member template.
+ bool ha
https://github.com/mizvekov approved this pull request.
https://github.com/llvm/llvm-project/pull/111852
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman commented:
Thank you for this, I think the clang-tidy portions make a lot of sense.
However, I do have concerns about the changes to Clang. I don't think we want
constant expression evaluation to fail based on an input parameter. For one,
it's a bit odd because
everythingfunctional wrote:
@klausler , can you point me specifically at the failure messages? The failures
I've seen so far from the merge did not look related to my (admittedly
untrained) eye. I ran the tests in the repo and they passed locally. Are there
other tests somewhere that need to b
klausler wrote:
> @klausler , can you point me specifically at the failure messages? The
> failures I've seen so far from the merge did not look related to my
> (admittedly untrained) eye. I ran the tests in the repo and they passed
> locally. Are there other tests somewhere that need to be up
https://github.com/llvm-beanz edited
https://github.com/llvm/llvm-project/pull/111010
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1956,6 +1956,26 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
return true;
break;
}
+ case Builtin::BI__builtin_hlsl_wave_read_lane_at: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+
+// Ensure ind
https://github.com/dpaoliello created
https://github.com/llvm/llvm-project/pull/111879
MSVC has a set of qualifiers to allow using 32-bit signed/unsigned pointers
when building 64-bit targets. This is useful for WoW code (i.e., the part of
Windows that handles running 32-bit application on a 6
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Daniel Paoliello (dpaoliello)
Changes
MSVC has a set of qualifiers to allow using 32-bit signed/unsigned pointers
when building 64-bit targets. This is useful for WoW code (i.e., the part of
Windows that handles running 32-bit application
Artem-B wrote:
> I'm not sure why the option isn't enabled by default, personally
While it does indeed help with generating better code, using this option while
compiling CUDA code may be problematic.
Front-end is not aware of address spaces and all pointers are generic, so
`sizeof(any pointer
zmodem wrote:
> No difference at all. It's just that the PR which will take care of the
> problem is not small, so I want to keep them separate and merge them all in
> one go.
To make it easier to find, that PR is #111457 and it landed soon after.
https://github.com/llvm/llvm-project/pull/111
hanickadot wrote:
> Is there some explanation of the set of builtins you chose somewhere? How is
> "masking" a pointer different from the existing
> __builtin_align_down/__builtin_align_up?
Intention is to allow to use any bits in pointer for tagging and give freedom.
To do so. I'm not propos
llvmbot wrote:
@llvm/pr-subscribers-backend-aarch64
Author: Daniel Paoliello (dpaoliello)
Changes
MSVC has a set of qualifiers to allow using 32-bit signed/unsigned pointers
when building 64-bit targets. This is useful for WoW code (i.e., the part of
Windows that handles running 32-bit a
sheredom wrote:
Ok by specifying `-resource-dir=Z:/resource` and adding it a VFS made that
work. But we are seeing a single stray original path in the source files that
we think is related to the working directory. We tried setting
`-working-directory=Z:/working`, but the VFS requires that thi
klausler wrote:
There are multiple weird failures in llvm-test-suite where clang++ complains
about unknown options that look like they're slang-only options. Did you run
llvm-test-suite before merging? Please check. These failures are going to
cause build bot failures shortly in the clang-*
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 9459d729d22b7bfedad9d3a4237162077c6984a4
3daa49652dd1f0c0bb2b42748cee717236c32ce7 --e
https://github.com/pskrgag updated
https://github.com/llvm/llvm-project/pull/111588
>From a3805292ea37cf06d1cf227768034b30a42a685f Mon Sep 17 00:00:00 2001
From: Pavel Skripkin
Date: Mon, 7 Oct 2024 23:01:24 +0300
Subject: [PATCH 1/6] wip: initial versio
---
.../Checkers/FuchsiaHandleChecker.
kiranchandramohan wrote:
The 2 stage buildbot (https://lab.llvm.org/buildbot/#/builders/41) with the
Fortran tests passed after this merge. Its latest run is also a pass.
https://github.com/llvm/llvm-project/pull/110023
___
cfe-commits mailing list
cf
honkstar1 wrote:
Adding to @sheredom ... the precompiled headers are actually portable now
(hurray!).. the single last non-virtual path we see is in the object files
(.o). And it is related to the working dir as mentioned.
https://github.com/llvm/llvm-project/pull/106577
_
kiranchandramohan wrote:
@everythingfunctional Following are the steps to run the testsuite. Please
check and confirm if you did not run it before.
```
git clone https://github.com/llvm/llvm-test-suite.git
cd llvm-test-suite
mkdir build
cd build
cmake -DCMAKE_BUILD_TYPE=Release
-DCMAKE_C_COMPI
everythingfunctional wrote:
> @everythingfunctional Following are the steps to run the testsuite. Please
> check and confirm if you did not run it before.
>
> ```
> git clone https://github.com/llvm/llvm-test-suite.git
> cd llvm-test-suite
> mkdir build
> cd build
> cmake -DCMAKE_BUILD_TYPE=Rel
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/111209
>From 398dcb5c1a354c12d4d43af152f6d3e14f001274 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 2 Oct 2024 13:30:33 -0700
Subject: [PATCH 1/2] [HLSL] Implementation the `degrees` intrinsic
- add degre
klausler wrote:
> Which I don't think the rename is at fault for.
If you rebase to current llvm-project/main, that test will (or should) work; it
got broken yesterday and was fixed later.
https://github.com/llvm/llvm-project/pull/110023
___
cfe-commi
https://github.com/dpaoliello updated
https://github.com/llvm/llvm-project/pull/111879
>From ec2b533ef344f58f61fc59fe5a9c55043eef8718 Mon Sep 17 00:00:00 2001
From: "Daniel Paoliello (HE/HIM)"
Date: Wed, 9 Oct 2024 16:47:57 -0700
Subject: [PATCH] [clang][aarch64] Add support for the MS qualifie
AaronBallman wrote:
> I remember back in the day that adding new matchers was a problem because it
> grew the size of the files beyond some supported compiler's limit.
>
> Could @AaronBallman or someone else knowledgable about this(cc @kadircet)
> take a look and confirm adding matchers is fin
@@ -32,6 +58,8 @@ class LLVM_LIBRARY_VISIBILITY AArch64TargetInfo : public
TargetInfo {
SveMode = (1 << 2),
};
+ enum AddrSpace { ptr32_sptr = 270, ptr32_uptr = 271, ptr64 = 272 };
dpaoliello wrote:
Definitely prefer that - this whole thing has **way*
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff d36cef0b173329fa1f94ff3a92da6a50da4aff9e
ef0bcaf5528ca3bc7597ae0114618a4c4bb25e09 --e
https://github.com/mizvekov created
https://github.com/llvm/llvm-project/pull/111894
This flag has been deprecated since Clang 19, having been the default since
then.
It has remained because its negation was still useful to work around backwards
compatibility breaking changes from P0522.
How
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-clang
Author: Matheus Izvekov (mizvekov)
Changes
This flag has been deprecated since Clang 19, having been the default since
then.
It has remained because its negation was still useful to work around backwards
compati
erichkeane wrote:
I THINK I'd still prefer to keep this flag for 1 more release. Our mitigations
/P3310 haven't been in a release cycle yet, so I'd prefer we wait until after
the 20 release to flip this switch.
https://github.com/llvm/llvm-project/pull/111894
__
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/111209
>From 398dcb5c1a354c12d4d43af152f6d3e14f001274 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 2 Oct 2024 13:30:33 -0700
Subject: [PATCH 1/3] [HLSL] Implementation the `degrees` intrinsic
- add degre
https://github.com/pow2clk closed
https://github.com/llvm/llvm-project/pull/111632
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/haoNoQ updated
https://github.com/llvm/llvm-project/pull/111624
>From b5c9082e36efcc7be2cabc73c985749f2fd41725 Mon Sep 17 00:00:00 2001
From: Artem Dergachev
Date: Tue, 8 Oct 2024 20:24:00 -0700
Subject: [PATCH 1/3] [-Wunsafe-buffer-usage] Add user documentation.
---
clang/
efriedma-quic wrote:
unnamed_addr doesn't allow cloning; there's no way to make it work in a
reasonable way even if we wanted it. See, for example,
https://github.com/llvm/llvm-project/issues/32127 .
https://github.com/llvm/llvm-project/pull/111343
@@ -0,0 +1,47 @@
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.0-compute -S -o - %s |
FileCheck %s
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library -S -o - %s |
FileCheck %s
+
+// Verify that internal linkage unused functions are removed
+
+RWBuffer buf;
+
+// Never
@@ -19,20 +19,20 @@
using namespace llvm;
static bool finalizeLinkage(Module &M) {
- SmallPtrSet EntriesAndExports;
+ SmallPtrSet Funcs;
// Find all entry points and export functions
for (Function &EF : M.functions()) {
-if (!EF.hasFnAttribute("hlsl.shader") && !
@@ -0,0 +1,80 @@
+; RUN: opt -S -dxil-finalize-linkage
-mtriple=dxil-unknown-shadermodel6.5-compute %s | FileCheck %s
+; RUN: llc %s --filetype=asm -o - | FileCheck %s
+
+target triple = "dxilv1.5-pc-shadermodel6.5-compute"
+
+; Confirm that DXILFinalizeLinkage will remove functi
https://github.com/hanickadot updated
https://github.com/llvm/llvm-project/pull/111861
From e1c8d5e689fe8d3d0338eb64220aaf6371aed48a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Hana=20Dusi=CC=81kova=CC=81?=
Date: Thu, 10 Oct 2024 21:05:55 +0200
Subject: [PATCH] [clang & libcxx] constexpr pointer t
@@ -1,13 +1,15 @@
// Check CUID generated by hash.
// The same CUID is generated for the same file with the same options.
+// RUN: cd %S
+
// RUN: %clang -### -x hip --target=x86_64-unknown-linux-gnu
--no-offload-new-driver \
// RUN: --offload-arch=gfx906 -c -nogpuinc -no
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/110435
>From c52634882631a71fad956a70179b480abf13006a Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Sun, 29 Sep 2024 22:01:38 +0300
Subject: [PATCH 1/3] [Clang] fix overload resolution for object parameters
with
@@ -526,6 +526,11 @@ llvm::Type *CodeGenTypes::ConvertType(QualType T) {
}
case BuiltinType::SveCount:
return llvm::TargetExtType::get(getLLVMContext(), "aarch64.svcount");
+case BuiltinType::ArmMFloat8:
+ ResultType =
+ llvm::VectorType::get(l
@@ -5828,6 +5832,7 @@ bool Parser::isKnownToBeTypeSpecifier(const Token &Tok)
const {
case tok::kw__ExtInt:
case tok::kw__BitInt:
case tok::kw___bf16:
+ case tok::kw___mfp8:
paulwalker-arm wrote:
Do you know why this and the similar changes are needed?
@@ -2218,6 +2219,11 @@ TypeInfo ASTContext::getTypeInfoImpl(const Type *T)
const {
Width = 0;
\
Align = 16;
\
break;
+#define AARCH64_SC
@@ -1391,7 +1391,8 @@ void ASTContext::InitBuiltinTypes(const TargetInfo
&Target,
}
if (Target.hasAArch64SVETypes() ||
- (AuxTarget && AuxTarget->hasAArch64SVETypes())) {
+ (AuxTarget && AuxTarget->hasAArch64SVETypes()) ||
+ Target.hasArmMFloat8Type()) {
--
@@ -3437,8 +3445,8 @@ StringRef BuiltinType::getName(const PrintingPolicy
&Policy) const {
case Id: \
return #ExtType;
#include "clang/Basic/OpenCLExtensionTypes.def"
-#define SVE_TYPE(Name, Id, SingletonId) \
- case Id: \
+#define SVE_TYPE(Name, Id, SingletonId)
@@ -700,6 +701,9 @@ class TargetInfo : public TransferrableTargetInfo,
return HasBFloat16 || HasFullBFloat16;
}
+ /// Determine whether the _mfp8 type is supported on this target.
+ virtual bool hasArmMFloat8Type() const { return HasMFloat8; }
paulwalk
@@ -476,7 +476,6 @@ llvm::Type *CodeGenTypes::ConvertType(QualType T) {
Context.getFloatTypeSemantics(T),
/* UseNativeHalf = */ false);
break;
-
paulwalker-arm wrote:
Unnecessary whi
@@ -4302,6 +4308,7 @@ ASTContext::getBuiltinVectorTypeInfo(const BuiltinType
*Ty) const {
case BuiltinType::Id:
\
return {BoolTy, llvm::ElementCount::getScalable(NumEls), NF};
#define SVE_OPAQUE_TYPE(Name, MangledNa
@@ -1381,7 +1381,7 @@ void ASTContext::InitBuiltinTypes(const TargetInfo
&Target,
if (LangOpts.OpenACC && !LangOpts.OpenMP) {
InitBuiltinType(ArraySectionTy, BuiltinType::ArraySection);
}
- if (LangOpts.MatrixTypes)
+ if (LangOpts.MatrixTypes || LangOpts.HLSL)
--
ldionne wrote:
Also, this is another reminder that we would benefit from having Fuchsia
builders in the pre-commit CI. We've asked before and we'll keep asking until
we get them: Fuchsia is amongst the last platform that frequently reports
errors but doesn't provide us with any pre-commit visi
https://github.com/bogner approved this pull request.
https://github.com/llvm/llvm-project/pull/111752
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/adam-yang created
https://github.com/llvm/llvm-project/pull/111883
partially fixes #70103
### Changes
* Implemented `GroupMemoryBarrierWithGroupSync` clang builtin
* Linked `GroupMemoryBarrierWithGroupSync` clang builtin with
`hlsl_intrinsics.h`
* Added sema checks for `Gro
https://github.com/adam-yang edited
https://github.com/llvm/llvm-project/pull/111883
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/dpaoliello updated
https://github.com/llvm/llvm-project/pull/111879
>From 281a05c90b1b7df7e75743012d93efeb484901fd Mon Sep 17 00:00:00 2001
From: "Daniel Paoliello (HE/HIM)"
Date: Wed, 9 Oct 2024 16:47:57 -0700
Subject: [PATCH] [clang][aarch64] Add support for the MS qualifie
https://github.com/zygoloid edited
https://github.com/llvm/llvm-project/pull/111343
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zygoloid commented:
Hmm. Looking at https://reviews.llvm.org/D20348 it seems like `unnamed_addr`
and `local_unnamed_addr` don't just permit merging, but also cloning of the
global (so that the same global could have different addresses in different
modules). Cloning wouldn't
@@ -4097,6 +4096,7 @@ llvm::Constant *ItaniumRTTIBuilder::BuildTypeInfo(
// And the name.
llvm::GlobalVariable *TypeName = GetAddrOfTypeName(Ty, Linkage);
llvm::Constant *TypeNameField;
+ llvm::GlobalValue::UnnamedAddr Scope;
zygoloid wrote:
`Scope` isn
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff fabe7e39df624c09d399f5f61d5450c2021c2357
628e7d8991f3e8114bb1afbb28bf3eebba622be8 --e
@@ -19,20 +19,20 @@
using namespace llvm;
static bool finalizeLinkage(Module &M) {
- SmallPtrSet EntriesAndExports;
+ SmallPtrSet Funcs;
// Find all entry points and export functions
pow2clk wrote:
Updated
https://github.com/llvm/llvm-project/pull/106
efriedma-quic wrote:
> Intention is to allow to use any bits in pointer for tagging and give freedom
Hmm, so needs to somehow allow accessing bits that aren't low bits?
As a general comment not specific to this patch, I'd prefer to build around
llvm.ptrmask and geps as much as possible, as opp
zygoloid wrote:
> unnamed_addr doesn't allow cloning; there's no way to make it work in a
> reasonable way even if we wanted it. See, for example, #32127 .
OK. The LangRef is very vague on its semantics, would be nice to get that
clarified by someone who knows the intent.
Is the same true for
@@ -503,17 +503,16 @@ bool Sema::checkLiteralOperatorId(const CXXScopeSpec &SS,
const IdentifierInfo *II = Name.Identifier;
ReservedIdentifierStatus Status = II->isReserved(PP.getLangOpts());
SourceLocation Loc = Name.getEndLoc();
-if (!PP.getSourceManager().isI
hanickadot wrote:
> Hmm, so needs to somehow allow accessing bits that aren't low bits?
yes, for example upper byte on aarch64
> As a general comment not specific to this patch, I'd prefer to build around
> llvm.ptrmask and geps as much as possible, as opposed to relying on ptrtoint.
I 100% a
https://github.com/AaronBallman approved this pull request.
LGTM, please keep an eye out for fallout in the wild though. It may be
interesting to see how many instances of `-Wno-deprecated-literal-operator`
exist now and do a search again before release to see how many new instances
show up.
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
https://github.com/steakhal requested changes to this pull request.
I'm sharing my unfinished review as I have it now to leave you a feedback.
In the LLVM community we
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/109636
___
cfe-commits mailing list
cfe-commit
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,244 @@
+//===--- MutexModelingAPI.h - API for modeling mutexes
===//
+//
+// Part of the LLVM Project, under the Apache
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,169 @@
+//===--- MutexModelingGDM.h - Modeling of mutexes in GDM
--===//
+//
+// Part of the LLVM Project, under the Apache
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,773 @@
+//===--- MutexModeling.cpp - Modeling of mutexes
--===//
+//
+// Part of the LLVM Project, under the Apache
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,244 @@
+//===--- MutexModelingAPI.h - API for modeling mutexes
===//
+//
+// Part of the LLVM Project, under the Apache
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,169 @@
+//===--- MutexModelingGDM.h - Modeling of mutexes in GDM
--===//
+//
+// Part of the LLVM Project, under the Apache
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-x86_64-debian`
running on `lldb-x86_64-debian` while building `clang` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/162/builds/8122
Here is the relevant piece of the build log
https://github.com/ChuanqiXu9 approved this pull request.
LGTM now
https://github.com/llvm/llvm-project/pull/111662
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,244 @@
+//===--- MutexModelingAPI.h - API for modeling mutexes
===//
+//
+// Part of the LLVM Project, under the Apache
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?= ,
Endre =?utf-8?q?Fülöp?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,169 @@
+//===--- MutexModelingGDM.h - Modeling of mutexes in GDM
--===//
+//
+// Part of the LLVM Project, under the Apache
@@ -522,6 +522,39 @@ def FeatureTLBIW : ExtensionWithMArch<"tlbiw", "TLBIW",
"FEAT_TLBIW",
// Armv9.6 Architecture Extensions
//===--===//
+def FeatureCMPBR : ExtensionWithMArch<"cmpbr", "CMPBR", "FEAT_CMPB
@@ -522,6 +522,39 @@ def FeatureTLBIW : ExtensionWithMArch<"tlbiw", "TLBIW",
"FEAT_TLBIW",
// Armv9.6 Architecture Extensions
//===--===//
+def FeatureCMPBR : ExtensionWithMArch<"cmpbr", "CMPBR", "FEAT_CMPB
PankajDwivedi-25 wrote:
> This does not seem to be the right fix. I tends to think the test
> https://github.com/ROCm/hip-tests/tree/amd-staging/samples/2_Cookbook/16_assembly_to_executable
> needs fix. Since it does not expect host-only compilation to use CUID, it
> should add `-fuse-cuid=non
Author: Oleksandr T.
Date: 2024-10-10T10:02:21+02:00
New Revision: 1fa3c857f011bfbca788d481680229accbf6e350
URL:
https://github.com/llvm/llvm-project/commit/1fa3c857f011bfbca788d481680229accbf6e350
DIFF:
https://github.com/llvm/llvm-project/commit/1fa3c857f011bfbca788d481680229accbf6e350.diff
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/107786
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
201 - 300 of 443 matches
Mail list logo
