@@ -12,7 +12,7 @@ __launch_bounds__(0x1) void
TestWayTooBigArg(void); // expected-
__launch_bounds__(-128, 7) void TestNegArg1(void); // expected-warning
{{'launch_bounds' attribute parameter 0 is negative and will be ignored}}
__launch_bounds__(128, -7) void
https://github.com/jchlanda updated
https://github.com/llvm/llvm-project/pull/66496
>From 9c8caed3c8def15ccdbfdf831f36d0befed1fc84 Mon Sep 17 00:00:00 2001
From: Jakub Chlanda
Date: Fri, 15 Sep 2023 12:08:04 +0100
Subject: [PATCH 1/5] [NVPTX] Add support for maxclusterrank in launch_bounds
https://github.com/eopXD updated https://github.com/llvm/llvm-project/pull/67109
>From dc77e5c7bca87badecc314b24cab4c10be0e02fa Mon Sep 17 00:00:00 2001
From: eopXD
Date: Thu, 21 Sep 2023 06:34:57 -0700
Subject: [PATCH 1/3] [Clang][RISCV] Handle RVV tuple types correctly as
OutputOperand for
https://github.com/eopXD updated https://github.com/llvm/llvm-project/pull/67018
>From dc77e5c7bca87badecc314b24cab4c10be0e02fa Mon Sep 17 00:00:00 2001
From: eopXD
Date: Thu, 21 Sep 2023 06:34:57 -0700
Subject: [PATCH] [Clang][RISCV] Handle RVV tuple types correctly as
OutputOperand for
https://github.com/ktf updated https://github.com/llvm/llvm-project/pull/66430
>From dd75b18a8497381f478a85f1bf7b622583c5ca29 Mon Sep 17 00:00:00 2001
From: Giulio Eulisse <10544+...@users.noreply.github.com>
Date: Thu, 14 Sep 2023 21:58:21 +0200
Subject: [PATCH] Introduce PagedVector class
The
https://github.com/spaits updated
https://github.com/llvm/llvm-project/pull/66481
From ce62d3e1924b497b3e7160579a87557119c9e35d Mon Sep 17 00:00:00 2001
From: Gabor Spaits
Date: Fri, 15 Sep 2023 10:21:30 +0200
Subject: [PATCH] [analyzer] Add std::variant checker
Adding a checker that checks
https://github.com/eopXD edited https://github.com/llvm/llvm-project/pull/67109
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -537,59 +537,46 @@ void NVPTXAsmPrinter::emitKernelFunctionDirectives(const
Function ,
raw_ostream ) const {
// If the NVVM IR has some of reqntid* specified, then output
// the reqntid directive, and set the
@@ -11836,6 +11836,10 @@ def err_sycl_special_type_num_init_method : Error<
"types with 'sycl_special_class' attribute must have one and only one
'__init' "
"method defined">;
+def warn_cuda_maxclusterrank_sm_90 : Warning<
+ "maxclusterrank requires sm_90 or higher,
@@ -5607,6 +5607,21 @@ bool Sema::CheckRegparmAttr(const ParsedAttr ,
unsigned ) {
return false;
}
+// Helper to get CudaArch.
+static CudaArch getCudaArch(const TargetInfo ) {
jchlanda wrote:
Is that the kind of thing you had in mind:
```diff
diff --git
sam-mccall wrote:
> I just meant to change [...] It's a local fix, but all the code here is
> pretty inconsistent anyway, so I don't feel too bad about it.
Oops, of course. Done & I worked out the testing.
https://github.com/llvm/llvm-project/pull/66947
https://github.com/sam-mccall updated
https://github.com/llvm/llvm-project/pull/66947
>From 00ef242c40dd453157ee3b31e367fd6240d68a5a Mon Sep 17 00:00:00 2001
From: Sam McCall
Date: Wed, 20 Sep 2023 21:19:03 +0200
Subject: [PATCH 1/2] [Driver] Fix detection of libc++ with empty sysroot.
https://github.com/jchlanda updated
https://github.com/llvm/llvm-project/pull/66496
>From 9c8caed3c8def15ccdbfdf831f36d0befed1fc84 Mon Sep 17 00:00:00 2001
From: Jakub Chlanda
Date: Fri, 15 Sep 2023 12:08:04 +0100
Subject: [PATCH 1/4] [NVPTX] Add support for maxclusterrank in launch_bounds
https://github.com/jchlanda resolved
https://github.com/llvm/llvm-project/pull/66496
___
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/67103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Alcaro
Date: 2023-09-22T11:58:32+02:00
New Revision: 10217b9d9b4814eb086cffd7f008cb72967cd739
URL:
https://github.com/llvm/llvm-project/commit/10217b9d9b4814eb086cffd7f008cb72967cd739
DIFF:
https://github.com/llvm/llvm-project/commit/10217b9d9b4814eb086cffd7f008cb72967cd739.diff
LOG:
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
This PR is based on #67018. This PR fixes compilation issue for RVV
tuple types as InputOperands for inline asm.
---
Currently the compiler generates https://godbolt.org/z/djebPfqxf for tuple type
as inline asm inputs and cannot be code
https://github.com/eopXD created https://github.com/llvm/llvm-project/pull/67109
This PR is based on #67018. This PR fixes compilation issue for RVV tuple types
as InputOperands for inline asm.
---
Currently the compiler generates https://godbolt.org/z/djebPfqxf for tuple type
as inline asm
https://github.com/eopXD edited https://github.com/llvm/llvm-project/pull/67018
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/eopXD edited https://github.com/llvm/llvm-project/pull/67018
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/eopXD edited https://github.com/llvm/llvm-project/pull/67018
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rorth wrote:
The different behaviour can easily be seen since Solaris `clang` supports
linker selection at compile time:
```
$ clang -r -o hello-r.o hello.o -fuse-ld=gld
/usr/gnu/bin/ld: warning: cannot find entry symbol _start; not setting start
address
$ clang -r -o hello-r.o hello.o
ld:
@@ -587,6 +588,36 @@ TEST_F(HeadersForSymbolTest, AmbiguousStdSymbols) {
}
}
+TEST_F(HeadersForSymbolTest, AmbiguousStdSymbolsUsingShadow) {
kadircet wrote:
might be easier to test this in AnalysisTest, rather than writing a new
ASTVisitor, but up to you.
https://github.com/kadircet approved this pull request.
thanks, lgtm!
https://github.com/llvm/llvm-project/pull/66485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet edited
https://github.com/llvm/llvm-project/pull/66485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sam-mccall closed
https://github.com/llvm/llvm-project/pull/66424
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Sam McCall
Date: 2023-09-22T11:24:24+02:00
New Revision: 3f78d6ab146874d20144f9f5fcbb894931279c7d
URL:
https://github.com/llvm/llvm-project/commit/3f78d6ab146874d20144f9f5fcbb894931279c7d
DIFF:
https://github.com/llvm/llvm-project/commit/3f78d6ab146874d20144f9f5fcbb894931279c7d.diff
Author: Timm Bäder
Date: 2023-09-22T11:23:18+02:00
New Revision: b2bbf6947cae64934ff35a6a467bad34c1096155
URL:
https://github.com/llvm/llvm-project/commit/b2bbf6947cae64934ff35a6a467bad34c1096155
DIFF:
https://github.com/llvm/llvm-project/commit/b2bbf6947cae64934ff35a6a467bad34c1096155.diff
Author: Sam McCall
Date: 2023-09-22T11:23:11+02:00
New Revision: 0f05096540bc90125df47983d7013dd440617055
URL:
https://github.com/llvm/llvm-project/commit/0f05096540bc90125df47983d7013dd440617055
DIFF:
https://github.com/llvm/llvm-project/commit/0f05096540bc90125df47983d7013dd440617055.diff
https://github.com/sam-mccall closed
https://github.com/llvm/llvm-project/pull/66933
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sam-mccall closed
https://github.com/llvm/llvm-project/pull/66757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Alcaro created
https://github.com/llvm/llvm-project/pull/67103
None
>From 59f00d06056ff88acf014c339d64ac3c3ec85466 Mon Sep 17 00:00:00 2001
From: Alcaro
Date: Fri, 22 Sep 2023 11:08:56 +0200
Subject: [PATCH] docs: Fix misplaced apostrophe
---
Author: Sam McCall
Date: 2023-09-22T11:09:18+02:00
New Revision: 01d3045d1261c72abcf9cc1071be87434cf7086d
URL:
https://github.com/llvm/llvm-project/commit/01d3045d1261c72abcf9cc1071be87434cf7086d
DIFF:
https://github.com/llvm/llvm-project/commit/01d3045d1261c72abcf9cc1071be87434cf7086d.diff
sam-mccall wrote:
Thanks! This will fix the double-slash issue (despite the name, `add_dots`
normalizes those).
symlinks and progra~1 mean normalization based on actual filesystem access, and
choosing between multiple forms that are in some sense equally valid.
I don't think I'm opposed if
ilya-biryukov added a comment.
@cor3ntin any suggestions on how to proceed here?
I hope the approach taken in the current patch should already provide
meaningful and improved results in most cases.
It only affects the diagnostics output, so changing or reverting it should be
relatively easy in
https://github.com/asl resolved https://github.com/llvm/llvm-project/pull/66333
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
If you are doing more contributions you can get commit access by doing
https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access. Though with
the switch to Github it's super easy to ask the reviewer to click the merge
button, so it's not required.
tbaeder added a comment.
Ping
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155572/new/
https://reviews.llvm.org/D155572
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/66715
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Balint Cristian
Date: 2023-09-22T08:43:17+01:00
New Revision: 73779bb2093310bf129017f2f504ec93b1ca4a1d
URL:
https://github.com/llvm/llvm-project/commit/73779bb2093310bf129017f2f504ec93b1ca4a1d
DIFF:
https://github.com/banach-space requested changes to this pull request.
Please re-enable the tests in linker-flags.f90.
If these options don't work on Windows then we should try to document it. Also,
tests for `-rpath` can be moved to a dedicated file if we need to disable them
on a
Author: Heejin Ahn
Date: 2023-09-22T00:35:37-07:00
New Revision: 058222b2316615194c089f2bc68d11341f39d26e
URL:
https://github.com/llvm/llvm-project/commit/058222b2316615194c089f2bc68d11341f39d26e
DIFF:
https://github.com/llvm/llvm-project/commit/058222b2316615194c089f2bc68d11341f39d26e.diff
Prabhuk updated this revision to Diff 557227.
Prabhuk added a comment.
Addressed review comments about the inline comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159541/new/
https://reviews.llvm.org/D159541
Files:
banach-space wrote:
> Thanks @MaskRay and @banach-space for taking a look at this. I have marked
> the test as unsupported on windows because of a failing buildkite. I don't
> have access to a Windows machine so I can't add an equivalent test for that
> platform.
Thanks for working for this!
phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152206/new/
https://reviews.llvm.org/D152206
Prabhuk added a comment.
@phosek -- Regarding the abandoning commit. I had messed up the commit
splitting process earlier. Uploaded the correct set of files for this change
now. Please take a look.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
@@ -136,6 +136,17 @@ int main(void) {
// Cleanup happens automatically -> no warning.
}
+ /// Function pointers
+ {
+int __attribute__((requires_capability())) (*function_ptr)(int) =
Foo_fun1;
tbaederr wrote:
This assignment also works if
Prabhuk updated this revision to Diff 557223.
Prabhuk added a comment.
Update patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152206/new/
https://reviews.llvm.org/D152206
Files:
llvm/lib/IR/DataLayout.cpp
llvm/lib/TargetParser/Triple.cpp
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
This was much easier than expected actually.
I'm sure I'm missing a lot of changes and test coverage though.
@aaronpuchert I'm starting a new review here instead of continuing the
one in Phab since this is a completely different attempt.
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/67095
This was much easier than expected actually.
I'm sure I'm missing a lot of changes and test coverage though.
@aaronpuchert I'm starting a new review here instead of continuing the one in
Phab since this is a
Prabhuk updated this revision to Diff 557221.
Prabhuk added a comment.
Datalayout update.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152206/new/
https://reviews.llvm.org/D152206
Files:
llvm/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp
apelete added a comment.
I guess it can be closed, thanks.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D19385/new/
https://reviews.llvm.org/D19385
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/XChy updated https://github.com/llvm/llvm-project/pull/67093
>From e7314709fc812736a357b50654b08ef722fb89a4 Mon Sep 17 00:00:00 2001
From: XChy
Date: Mon, 31 Jul 2023 17:10:45 +0800
Subject: [PATCH 1/2] [InstCombine] Tests for simplifying icmp(constants[x])
---
@@ -52,6 +52,7 @@ set(ENABLE_LINKER_BUILD_ID ON CACHE BOOL "")
set(ENABLE_X86_RELAX_RELOCATIONS ON CACHE BOOL "")
set(CMAKE_BUILD_TYPE Release CACHE STRING "")
+set(CMAKE_CXX_VISIBILITY_PRESET default CACHE STRING "")
petrhosek wrote:
Should we also set
phosek added a comment.
Is it possible to abandon this change?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152206/new/
https://reviews.llvm.org/D152206
___
cfe-commits mailing list
phosek added inline comments.
Comment at: clang/lib/Driver/ToolChains/UEFI.cpp:58-59
+
+ // Other UEFI binary subsystems that are currently unsupported:
+ // efi_boot_service_driver, efi_rom, efi_runtime_driver.
+ CmdArgs.push_back("-subsystem:efi_application");
@@ -463,3 +463,38 @@ static_assert(bit_cast(ld539) ==
fivehundredandthirtynine, "");
static_assert(round_trip<__int128_t>(34.0L));
#endif
}
+
+namespace test_vector {
+
+typedef unsigned uint2 __attribute__((vector_size(2 * sizeof(unsigned;
+typedef char byte8
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
`isRVVSizelessBuiltinType` and `isRVVType` has the same functionality. This
commit removes the former since we have more variants available in `isRVVType`.
---
Full diff: https://github.com/llvm/llvm-project/pull/67089.diff
5 Files
https://github.com/eopXD created https://github.com/llvm/llvm-project/pull/67089
`isRVVSizelessBuiltinType` and `isRVVType` has the same functionality. This
commit removes the former since we have more variants available in `isRVVType`.
>From 483240e16d800e52783dcfaf52eae81fb1dfca7c Mon Sep 17
@@ -156,10 +156,11 @@ define void @storesTIInit(double %Val) #0 {
; SMALL32: # %bb.0: # %entry
; SMALL32-NEXT:mflr 0
; SMALL32-NEXT:stwu 1, -32(1)
-; SMALL32-NEXT:lwz 3, L..C4(2) # target-flags(ppc-lo) @TIInit
-; SMALL32-NEXT:lwz 4, L..C5(2) #
kasuga-fj wrote:
Thank you for your review! Here are some comments and my thoughts.
> Note that you should probably rebase your patch onto main and force-push to
> update the PR
Does it mean that I can rebase and force-push since (it should be avoided in
normal case but) this is a special
https://github.com/eopXD edited https://github.com/llvm/llvm-project/pull/67018
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/eopXD edited https://github.com/llvm/llvm-project/pull/67018
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
201 - 263 of 263 matches
Mail list logo