https://github.com/pizzud updated
https://github.com/llvm/llvm-project/pull/67467
>From 6d5d35e1273f595e8a0382053d5183cbce7a9d8a Mon Sep 17 00:00:00 2001
From: David Pizzuto
Date: Tue, 26 Sep 2023 10:45:42 -0700
Subject: [PATCH 1/5] [clang-tidy] Add bugprone-move-shared-pointer-contents
https://github.com/knightXun updated
https://github.com/llvm/llvm-project/pull/74885
>From 12cc1fe332fbab94c1368ea95374d5a1289a22f8 Mon Sep 17 00:00:00 2001
From: knightXun
Date: Sat, 9 Dec 2023 04:57:15 +0800
Subject: [PATCH] [Clang][Sema] Check the number of lambda non-concept tempate
https://github.com/knightXun updated
https://github.com/llvm/llvm-project/pull/74885
>From 12cc1fe332fbab94c1368ea95374d5a1289a22f8 Mon Sep 17 00:00:00 2001
From: knightXun
Date: Sat, 9 Dec 2023 04:57:15 +0800
Subject: [PATCH 1/2] [Clang][Sema] Check the number of lambda non-concept
tempate
https://github.com/urnathan created
https://github.com/llvm/llvm-project/pull/74893
An 8 bit bitfield with preferred bool type? Seems confused.
>From f7cac332123f6ea6c78dcffbcac2d58f356b6396 Mon Sep 17 00:00:00 2001
From: Nathan Sidwell
Date: Fri, 8 Dec 2023 17:43:22 -0500
Subject: [PATCH]
https://github.com/knightXun updated
https://github.com/llvm/llvm-project/pull/74885
>From 12cc1fe332fbab94c1368ea95374d5a1289a22f8 Mon Sep 17 00:00:00 2001
From: knightXun
Date: Sat, 9 Dec 2023 04:57:15 +0800
Subject: [PATCH] [Clang][Sema] Check the number of lambda non-concept tempate
david-xl wrote:
> > > . For IR PGO, there is basically no need to do so as the instrumentation
> > > and profile-use should be in-sync. For front-end instrumentation, there
> > > seem to be some use cases to use out of sync profile:
> > > https://reviews.llvm.org/D51240.
> >
> >
> > Thanks
https://github.com/jansvoboda11 edited
https://github.com/llvm/llvm-project/pull/74892
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -441,22 +434,19 @@ void
ModuleManager::visit(llvm::function_ref Visitor,
bool ModuleManager::lookupModuleFile(StringRef FileName, off_t ExpectedSize,
time_t ExpectedModTime,
OptionalFileEntryRef ) {
minglotus-6 wrote:
> > . For IR PGO, there is basically no need to do so as the instrumentation
> > and profile-use should be in-sync. For front-end instrumentation, there
> > seem to be some use cases to use out of sync profile:
> > https://reviews.llvm.org/D51240.
>
> Thanks for double
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jan Svoboda (jansvoboda11)
Changes
AFAICT, `ModuleFile::File` can be `std::nullopt` only for PCM files loaded from
the standard input. This patch starts setting that variable to
`FileManager::getSTDIN()` in that case, which makes it
https://github.com/jansvoboda11 created
https://github.com/llvm/llvm-project/pull/74892
AFAICT, `ModuleFile::File` can be `std::nullopt` only for PCM files loaded from
the standard input. This patch starts setting that variable to
`FileManager::getSTDIN()` in that case, which makes it
@@ -2822,13 +2825,18 @@ bool UnwindCursor::setInfoForSigReturn(Registers_s390x &) {
// onto the stack.
const pint_t pc = static_cast(this->getReg(UNW_REG_IP));
// The PC might contain an invalid address if the unwind info is bad, so
- // directly accessing it could
https://github.com/ajordanr-google updated
https://github.com/llvm/llvm-project/pull/74791
>From 1f4df1b82970c95684eed93c8f6bcaa6d6507b88 Mon Sep 17 00:00:00 2001
From: Jordan R Abrahams-Whitehead
Date: Fri, 8 Dec 2023 00:09:59 +
Subject: [PATCH 1/2] [libunwind] Replace process_vm_readv
https://github.com/banach-space updated
https://github.com/llvm/llvm-project/pull/74377
From 1eefb9136ea042e44ada4be66a18b4bf69c18fe3 Mon Sep 17 00:00:00 2001
From: Andrzej Warzynski
Date: Sat, 2 Dec 2023 14:01:02 +
Subject: [PATCH 1/3] [flang][driver] Rename `flang-new` as `flang`
This
@@ -4022,8 +4169,36 @@ LValue CodeGenFunction::EmitArraySubscriptExpr(const
ArraySubscriptExpr *E,
ArrayLV = EmitArraySubscriptExpr(ASE, /*Accessed*/ true);
else
ArrayLV = EmitLValue(Array);
+
auto *Idx = EmitIdxAfterBase(/*Promote*/true);
+if
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/73686
>From bc152095691b32d1ad8539dfd60f5089df5eed8d Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 28 Nov 2023 10:39:44 -0800
Subject: [PATCH 01/11] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Piotr Zegar (PiotrZSL)
Changes
Check now more properly add missing parentheses to code like this: 'bool bar =
true ? 1 : 0 != 0;'.
Closes #71867
---
Patch is 31.07 KiB, truncated to 20.00 KiB below, full version:
https://github.com/PiotrZSL created
https://github.com/llvm/llvm-project/pull/74891
Check now more properly add missing parentheses to code like this: 'bool bar =
true ? 1 : 0 != 0;'.
Closes #71867
>From 8c3b797f1f34d18c1e9211898f7d1a5697251317 Mon Sep 17 00:00:00 2001
From: Piotr Zegar
https://github.com/knightXun updated
https://github.com/llvm/llvm-project/pull/74885
>From 4b5f4c6a15e84a9c3025a241cc5e11ad768dfdda Mon Sep 17 00:00:00 2001
From: knightXun
Date: Sat, 9 Dec 2023 04:57:15 +0800
Subject: [PATCH] [Clang][Sema] Check the number of lambda non-concept tempate
MaskRay wrote:
> > > Probably would be good to introduce the `-v1` version and require it
> > > first, then eventually change the default - so people don't get a silent
> > > behavior change? Even the existing users only using `*` and `.` need to
> > > change their syntax to migrate to v2,
jansvoboda11 wrote:
My first version of the test put backslashes into the source file (i.e.
`#include "C:\foo\bar"`) which doesn't really work, because Clang treats those
as string escape sequences. Instead of trying to replace `\` with `\\` in the
test, I chose to use forward slashes, which
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 05420a17547e495f5748e9662150d6eb931e2c28
65707b837a8bb7283896d2c9d4933a17e02a20b9 --
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Philip Reames (preames)
Changes
d80e46d added support for the target function attribute. However, it turns out
that commit has a nasty bug/oversight. As the tests in that revision show,
everything works if clang -cc1 is directly
https://github.com/preames created
https://github.com/llvm/llvm-project/pull/74889
d80e46d added support for the target function attribute. However, it turns out
that commit has a nasty bug/oversight. As the tests in that revision show,
everything works if clang -cc1 is directly invoked. I
https://github.com/jansvoboda11 updated
https://github.com/llvm/llvm-project/pull/74782
>From 6ab18edae7b86ca216848b7fcaff5e58fb3e186c Mon Sep 17 00:00:00 2001
From: Jan Svoboda
Date: Thu, 7 Dec 2023 15:15:16 -0800
Subject: [PATCH 1/4] [clang][lex] Fix non-portability diagnostics with
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/73686
>From bc152095691b32d1ad8539dfd60f5089df5eed8d Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 28 Nov 2023 10:39:44 -0800
Subject: [PATCH 01/10] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/73686
>From bc152095691b32d1ad8539dfd60f5089df5eed8d Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 28 Nov 2023 10:39:44 -0800
Subject: [PATCH 01/10] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
https://github.com/zixu-w closed https://github.com/llvm/llvm-project/pull/74886
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Zixu Wang
Date: 2023-12-08T13:07:29-08:00
New Revision: c9b4bb9ff9b65a741c558bfb93719df95272c2e1
URL:
https://github.com/llvm/llvm-project/commit/c9b4bb9ff9b65a741c558bfb93719df95272c2e1
DIFF:
https://github.com/llvm/llvm-project/commit/c9b4bb9ff9b65a741c558bfb93719df95272c2e1.diff
https://github.com/ian-twilightcoder approved this pull request.
https://github.com/llvm/llvm-project/pull/74886
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/74886
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/fhahn approved this pull request.
LGTM, thanks!
For reference, this is breaking llvm-test-suite builds, e.g. see
https://green.lab.llvm.org/green/job/lnt-ctmark-aarch64-Os/15893/console
https://github.com/llvm/llvm-project/pull/74886
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: flyingcat (knightXun)
Changes
Check that the number of non-concept template parameters is greater than zero
during lambda template instantiation to aviod panic Fix issue:
https://github.com/llvm/llvm-project/issues/70601
---
Full diff:
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-driver
Author: Zixu Wang (zixu-w)
Changes
https://github.com/llvm/llvm-project/pull/74676 landed the work to implement
`-fdefine-target-os-macros` and enabled the extension for the Darwin driver.
However it is breaking
https://github.com/zixu-w created
https://github.com/llvm/llvm-project/pull/74886
https://github.com/llvm/llvm-project/pull/74676 landed the work to implement
`-fdefine-target-os-macros` and enabled the extension for the Darwin driver.
However it is breaking some test builds. Leave the
https://github.com/knightXun created
https://github.com/llvm/llvm-project/pull/74885
Check that the number of non-concept template parameters is greater than zero
during lambda template instantiation to aviod panic Fix issue:
https://github.com/llvm/llvm-project/issues/70601
>From
@@ -1,39 +1,45 @@
-; Do setup work for all below tests: generate bitcode and combined index
-; RUN: opt -module-summary %s -o %t.bc
-; RUN: opt -module-summary %p/Inputs/thinlto_indirect_call_promotion.ll -o
%t2.bc
+; The raw profiles and reduced IR inputs are generated from
fhahn wrote:
Please also see the failures on GreenDragon:
https://green.lab.llvm.org/green/job/lnt-ctmark-aarch64-Os/15893/console
https://github.com/llvm/llvm-project/pull/74676
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
fhahn wrote:
It looks like this breaks building at least `MultiSource` from
https://github.com/llvm/llvm-test-suite/. The first failure I see is when
building `llvm-test-suite/MultiSource/Applications/ClamAV/zlib_zutil.c`
```
In file included from
RIscRIpt wrote:
Thank you!
https://github.com/llvm/llvm-project/pull/71300
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2884,6 +2884,12 @@ def warn_cxx11_compat_constexpr_body_multiple_return :
Warning<
InGroup, DefaultIgnore;
def note_constexpr_body_previous_return : Note<
"previous return statement is here">;
+def err_ms_constexpr_not_distinct : Error<
+ "[[msvc::constexpr]] cannot
@@ -6,14 +7,80 @@ float z();
#pragma float_control(except, on)
class ON {
float w = 2 + y() * z();
- // CHECK-LABEL: define {{.*}} @_ZN2ONC2Ev{{.*}}
- // CHECK: llvm.experimental.constrained.fmul{{.*}}tonearest{{.*}}strict
};
ON on;
#pragma float_control(except, off)
@@ -5587,10 +5593,12 @@ RValue CodeGenFunction::EmitCall(const CGFunctionInfo
,
!isa_and_nonnull(TargetDecl))
EmitKCFIOperandBundle(ConcreteCallee, BundleList);
+#if 0 // XXX Why is this here? Duplicate!
if (const FunctionDecl *FD =
@@ -6,6 +6,37 @@
//
//===--===//
+// character.h
yi-wu-arm wrote:
Replaced by a comment
https://github.com/llvm/llvm-project/pull/74628
___
@@ -37,5 +77,30 @@ void FORTRAN_PROCEDURE_NAME(getarg)(
(void)RTNAME(GetCommandArgument)(
n, , nullptr, nullptr, __FILE__, __LINE__);
}
+
+// CALL GETLOG(USRNAME)
+void FORTRAN_PROCEDURE_NAME(getlog)(std::int8_t *arg, std::int64_t length) {
yi-wu-arm
https://github.com/yi-wu-arm updated
https://github.com/llvm/llvm-project/pull/74628
>From 0e98aa7ca15b05b91813eaeeb6ae1305e5f5384d Mon Sep 17 00:00:00 2001
From: Yi Wu
Date: Mon, 6 Nov 2023 19:49:13 +
Subject: [PATCH 01/26] GETLOG runtime and extension implementation: get login
username
@@ -6,6 +6,37 @@
//
//===--===//
+// character.h
yi-wu-arm wrote:
It actually meant to match the `CHARCHTER_H` in the end of `#endif`, but I
guess that create a confusion and should be
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kevin P. Neal (kpneal)
Changes
Some C++ constructors require the strictfp attribute on all function calls but
don't get it because the comstructor lacks a FunctionDecl. Use the IRBuilder's
strictfp mode to communicate that the strictfp
https://github.com/kpneal created
https://github.com/llvm/llvm-project/pull/74883
Some C++ constructors require the strictfp attribute on all function calls but
don't get it because the comstructor lacks a FunctionDecl. Use the IRBuilder's
strictfp mode to communicate that the strictfp
@@ -10,10 +10,50 @@
// extensions that will eventually be implemented in Fortran.
#include "flang/Runtime/extensions.h"
+#include "flang/Runtime/character.h"
#include "flang/Runtime/command.h"
#include "flang/Runtime/descriptor.h"
#include "flang/Runtime/io-api.h"
https://github.com/yi-wu-arm updated
https://github.com/llvm/llvm-project/pull/74628
>From 0e98aa7ca15b05b91813eaeeb6ae1305e5f5384d Mon Sep 17 00:00:00 2001
From: Yi Wu
Date: Mon, 6 Nov 2023 19:49:13 +
Subject: [PATCH 01/25] GETLOG runtime and extension implementation: get login
username
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/71300
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3657,6 +3657,21 @@ an error:
}];
}
+def MSConstexprDocs : Documentation {
+ let Category = DocCatStmt;
+ let Content = [{
+The ``[[msvc::constexpr]]`` attribute can be applied only to a function
+definition or a ``return`` statement. It does not impact function
@@ -5546,11 +5563,14 @@ static EvalStmtResult EvaluateStmt(StmtResult ,
EvalInfo ,
case Stmt::LabelStmtClass:
return EvaluateStmt(Result, Info, cast(S)->getSubStmt(), Case);
- case Stmt::AttributedStmtClass:
-// As a general principle, C++11 attributes can be
@@ -235,6 +235,7 @@ Non-comprehensive list of changes in this release
except that it returns the size of a type ignoring tail padding.
* ``__builtin_classify_type()`` now classifies ``_BitInt`` values as the
return value ``18``
and vector types as return value ``19``, to
RIscRIpt wrote:
> Also, please don't 'force-push', just push an additional commit to your
> review. It erases history and makes reviewing it about 3x harder since
> history/context gets lost.
Sorry. I did this, because I am used to such workflow and reviewing
https://github.com/RIscRIpt updated
https://github.com/llvm/llvm-project/pull/71300
>From 3be36c6100801195f8f1f5167bdaa289bc8cb175 Mon Sep 17 00:00:00 2001
From: Richard Dzenis
Date: Thu, 20 Jul 2023 00:18:50 +0300
Subject: [PATCH 1/4] [clang-cl] Add support for [[msvc::constexpr]] C++11
@@ -1,39 +1,45 @@
-; Do setup work for all below tests: generate bitcode and combined index
-; RUN: opt -module-summary %s -o %t.bc
-; RUN: opt -module-summary %p/Inputs/thinlto_indirect_call_promotion.ll -o
%t2.bc
+; The raw profiles and reduced IR inputs are generated from
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/74794
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
I'm okay landing these changes without test coverage, though tests are always
preferred. LGTM
https://github.com/llvm/llvm-project/pull/73955
___
cfe-commits mailing list
@@ -1,39 +1,45 @@
-; Do setup work for all below tests: generate bitcode and combined index
-; RUN: opt -module-summary %s -o %t.bc
-; RUN: opt -module-summary %p/Inputs/thinlto_indirect_call_promotion.ll -o
%t2.bc
+; The raw profiles and reduced IR inputs are generated from
https://github.com/rnk commented:
> I assume that /volatile:ms is a legacy behavior that non-Windows OSes likely
> don't want to adopt.
That is correct, it is legacy behavior. However, I think this flag has
potential porting applications, similar to `-fshort-wchar`. I also think
operating
@@ -1,39 +1,45 @@
-; Do setup work for all below tests: generate bitcode and combined index
-; RUN: opt -module-summary %s -o %t.bc
-; RUN: opt -module-summary %p/Inputs/thinlto_indirect_call_promotion.ll -o
%t2.bc
+; The raw profiles and reduced IR inputs are generated from
https://github.com/ajordanr-google edited
https://github.com/llvm/llvm-project/pull/74791
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dwblaikie wrote:
> > Probably would be good to introduce the `-v1` version and require it first,
> > then eventually change the default - so people don't get a silent behavior
> > change? Even the existing users only using `*` and `.` need to change their
> > syntax to migrate to v2, right?
@@ -8,10 +8,7 @@
// UNSUPPORTED: c++03, c++11, c++14, c++17
-// sentinel() = default;
-// constexpr explicit sentinel(sentinel_t end);
-// constexpr sentinel(sentinel s)
-// requires Const && convertible_to, sentinel_t>;
+// constexpr sentinel_t base() const;
ellishg wrote:
> Probably would be good to introduce the `-v1` version and require it first,
> then eventually change the default - so people don't get a silent behavior
> change? Even the existing users only using `*` and `.` need to change their
> syntax to migrate to v2, right? They'll
https://github.com/fhahn closed https://github.com/llvm/llvm-project/pull/74761
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/petrhosek approved this pull request.
https://github.com/llvm/llvm-project/pull/74809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ziqingluo-90 closed
https://github.com/llvm/llvm-project/pull/74020
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ziqing Luo
Date: 2023-12-08T10:29:37-08:00
New Revision: a341e177cea1cee800793d357264f6f46a3b4979
URL:
https://github.com/llvm/llvm-project/commit/a341e177cea1cee800793d357264f6f46a3b4979
DIFF:
https://github.com/llvm/llvm-project/commit/a341e177cea1cee800793d357264f6f46a3b4979.diff
dwblaikie wrote:
Probably would be good to introduce the `-v1` version and require it first,
then eventually change the default - so people don't get a silent behavior
change? Even the existing users only using `*` and `.` need to change their
syntax to migrate to v2, right? They'll need to
@@ -959,6 +967,32 @@ def : GCNPat <
}
} // let OtherPredicates = [HasShaderCyclesRegister]
+def SIMM24bitPtr : ImmLeaf (Imm);}]
+>;
+
+multiclass SMPrefetchPat {
+ def : GCNPat <
+(smrd_prefetch (SMRDImm i64:$sbase, i32:$offset), timm, timm, (i32
cache_type)),
+
@@ -2700,19 +2701,18 @@ bool UnwindCursor::setInfoForSigReturn(Registers_arm64 &) {
// [1]
https://github.com/torvalds/linux/blob/master/arch/arm64/kernel/vdso/sigreturn.S
const pint_t pc = static_cast(this->getReg(UNW_REG_IP));
// The PC might contain an invalid
urnathan wrote:
> I'm planning to take a closer look at this when I have more time. Sorry I
> haven't been more responsive here.
When might that be?
https://github.com/llvm/llvm-project/pull/65742
___
cfe-commits mailing list
Author: Paul Walker
Date: 2023-12-08T18:01:12Z
New Revision: 94c837345c27e173284a85471d4efda19eded08e
URL:
https://github.com/llvm/llvm-project/commit/94c837345c27e173284a85471d4efda19eded08e
DIFF:
https://github.com/llvm/llvm-project/commit/94c837345c27e173284a85471d4efda19eded08e.diff
LOG:
Timm =?utf-8?q?B=C3=A4der?=
Message-ID:
In-Reply-To:
https://github.com/AaronBallman approved this pull request.
LGTM assuming no surprises come up from the additional test coverage.
https://github.com/llvm/llvm-project/pull/70772
___
cfe-commits
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
@@ -378,3 +378,92 @@ namespace Packs {
static_assert(foo() == 2, "");
static_assert(foo<>() == 0, "");
}
+
+namespace std {
+template struct remove_reference { using type = T; };
+template struct remove_reference { using
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
@@ -1005,12 +1008,23 @@ bool SetThisField(InterpState , CodePtr OpPC,
uint32_t I) {
template ::T>
bool GetGlobal(InterpState , CodePtr OpPC, uint32_t I) {
const Block *B = S.P.getGlobal(I);
+
+ if (!CheckConstant(S, OpPC,
Author: Shafik Yaghmour
Date: 2023-12-08T09:38:59-08:00
New Revision: bbb8a0df7367068e1cf2fc54edd376beb976b430
URL:
https://github.com/llvm/llvm-project/commit/bbb8a0df7367068e1cf2fc54edd376beb976b430
DIFF:
https://github.com/AaronBallman approved this pull request.
LGTM with a testing nit
https://github.com/llvm/llvm-project/pull/72660
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -27,6 +27,9 @@ static_assert(foo[2][3] == , "");
static_assert(foo[2][4] == nullptr, "");
+const int SomeInt[] = {1};
+int getSomeInt() { return *SomeInt; }
AaronBallman wrote:
```suggestion
constexpr int SomeInt[] = {1};
constexpr int getSomeInt() {
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/72660
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/72658
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
LGTM, but please wait until sometime mid-next week to land in case Richard has
concerns.
https://github.com/llvm/llvm-project/pull/70829
___
cfe-commits mailing list
Author: erichkeane
Date: 2023-12-08T09:32:49-08:00
New Revision: 731361cd1540d0e729633833e6f3a670443c4b84
URL:
https://github.com/llvm/llvm-project/commit/731361cd1540d0e729633833e6f3a670443c4b84
DIFF:
https://github.com/llvm/llvm-project/commit/731361cd1540d0e729633833e6f3a670443c4b84.diff
@@ -500,6 +506,12 @@ bool ClangTableGenMain(raw_ostream , RecordKeeper
) {
case GenArmSmeRangeChecks:
EmitSmeRangeChecks(Records, OS);
break;
+ case GenArmSmeStreamingAttrs:
SamTebbs33 wrote:
Done
https://github.com/llvm/llvm-project/pull/74064
https://github.com/CarolineConcatto updated
https://github.com/llvm/llvm-project/pull/73258
>From e0f245e8d6a395afac5de471b55358c7b730a170 Mon Sep 17 00:00:00 2001
From: Caroline Concatto
Date: Wed, 22 Nov 2023 10:03:50 +
Subject: [PATCH 1/8] [Clang][AArch64] Add fix vector types to
yxsamliu wrote:
It seems that is the only HIP test that still uses %T.
https://github.com/llvm/llvm-project/pull/74872
___
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: Yaxun (Sam) Liu (yxsamliu)
Changes
Replace %T with %t since %T is deprecated.
---
Full diff: https://github.com/llvm/llvm-project/pull/74872.diff
1 Files Affected:
- (modified) clang/test/Driver/rocm-detect.hip (+17-17)
https://github.com/yxsamliu created
https://github.com/llvm/llvm-project/pull/74872
Replace %T with %t since %T is deprecated.
>From 91160285c17ef9ba2b35e6e8c4a8eb433ec5ad8a Mon Sep 17 00:00:00 2001
From: "Yaxun (Sam) Liu"
Date: Fri, 8 Dec 2023 12:24:19 -0500
Subject: [PATCH] Fix test
https://github.com/ellishg approved this pull request.
Looks good to me. Thanks for following up!
https://github.com/llvm/llvm-project/pull/74809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
asb wrote:
> > I think the conclusion from the LLVM sync-up call was that everyone happy
> > to move in this direction, so please add the release note and we can do a
> > final review. Thanks!
>
> Done, added release note.
Thanks! Sorry I wasn't specific about this, but we need a Clang
@@ -20,3 +21,23 @@ int16x8_t incompat_neon_smc(int16x8_t splat)
__arm_streaming_compatible {
// expected-warning@+1 {{builtin call has undefined behaviour when called
from a streaming compatible function}}
return (int16x8_t)__builtin_neon_vqaddq_v((int8x16_t)splat,
@@ -3168,11 +3168,70 @@ static void checkArmStreamingBuiltin(Sema , CallExpr
*TheCall,
<< TheCall->getSourceRange() << "streaming compatible";
return;
}
+
+ if (FnType == ArmNonStreaming && BuiltinType == ArmStreaming) {
+S.Diag(TheCall->getBeginLoc(),
yxsamliu wrote:
> %T is deprecated in lit. You may want to migrate other `%T` related to AMDGPU.
Thanks. I will try cleaning them up.
https://github.com/llvm/llvm-project/pull/74783
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/xu-chiheng updated
https://github.com/llvm/llvm-project/pull/74868
From 680b2b48925d52c006eee5f26b1173856dfcf376 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=BE=90=E6=8C=81=E6=81=92=20Xu=20Chiheng?=
Date: Sat, 9 Dec 2023 00:59:00 +0800
Subject: [PATCH 1/2] [Cygwin] Cygwin
https://github.com/yxsamliu closed
https://github.com/llvm/llvm-project/pull/74783
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Yaxun (Sam) Liu
Date: 2023-12-08T12:04:01-05:00
New Revision: b842b1b65aab3bff2c3dbf439054aa8fe63f8400
URL:
https://github.com/llvm/llvm-project/commit/b842b1b65aab3bff2c3dbf439054aa8fe63f8400
DIFF:
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 49b27b150b97c190dedf8b45bf991c4b811ed953
680b2b48925d52c006eee5f26b1173856dfcf376 --
101 - 200 of 361 matches
Mail list logo