https://github.com/bogner updated
https://github.com/llvm/llvm-project/pull/75674
>From 9d6e00bd972a563daefd67b544614e2bb609cc42 Mon Sep 17 00:00:00 2001
From: Justin Bogner
Date: Fri, 15 Dec 2023 16:29:09 -0800
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
=?UTF
@@ -1600,6 +1600,25 @@ void SVEEmitter::createSMEHeader(raw_ostream &OS) {
OS << "extern \"C\" {\n";
OS << "#endif\n\n";
+ OS << "void __arm_za_disable(void) __arm_streaming_compatible;\n\n";
+
+ OS << "__ai bool __arm_has_sme(void) __arm_streaming_compatible {\n";
+ OS
@@ -10570,6 +10570,26 @@ Value
*CodeGenFunction::EmitAArch64BuiltinExpr(unsigned BuiltinID,
return Builder.CreateCall(F, llvm::ConstantInt::get(Int32Ty, HintID));
}
+ if (BuiltinID == clang::AArch64::BI__builtin_arm_get_sme_state) {
+// Create call to __arm_sme_sta
shafik wrote:
> > Why remove the support for ParenListExpr entirely? Could you not fallback
> > on looking at the init list expression only when ParenListExpr is null?
>
> We could keep `ParenListExpr`, but I'm not sure the benefit (IMO, removing it
> seems like an API improvement, and make co
@@ -362,14 +362,20 @@ mlir::scf::tileUsingSCFForOp(RewriterBase &rewriter,
TilingInterface op,
auto clonedOp = cast(
cloneOpAndUpdateDestinationArgs(rewriter, op, clonedOpDestination));
- // 5b. Tile the cloned operation.
+ // 5b. Early return cloned op if tiling is
https://github.com/shafik approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/75779
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chelini closed
https://github.com/llvm/llvm-project/pull/75494
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hanhanW updated
https://github.com/llvm/llvm-project/pull/75410
>From c07f7e1c5c6f8bbc7189e96096004d39a0a1aa3f Mon Sep 17 00:00:00 2001
From: hanhanW
Date: Wed, 13 Dec 2023 15:59:48 -0800
Subject: [PATCH 1/3] [mlir][TilingInterface] Early return cloned ops if tile
sizes are
https://github.com/momchil-velikov updated
https://github.com/llvm/llvm-project/pull/75596
>From fc5c82e61efef3f1cd2f6606b12c358637a687f5 Mon Sep 17 00:00:00 2001
From: Momchil Velikov
Date: Fri, 15 Dec 2023 12:18:53 +
Subject: [PATCH 1/2] [AArch64] Update target feature requirements of SVE
@@ -2066,7 +2066,7 @@ let TargetGuard = "sve2p1|sme2" in {
def SVPFALSE_COUNT_ALIAS : SInst<"svpfalse_c", "}v", "", MergeNone, "",
[IsOverloadNone, IsStreamingCompatible]>;
}
-let TargetGuard = "sve2p1,b16b16" in {
+let TargetGuard = "(sve2|sme2),b16b16" in {
--
Author: Ziqing Luo
Date: 2023-12-18T10:39:15-08:00
New Revision: 4123b95949727cb171bfcac6fcec89ea8cb76ef2
URL:
https://github.com/llvm/llvm-project/commit/4123b95949727cb171bfcac6fcec89ea8cb76ef2
DIFF:
https://github.com/llvm/llvm-project/commit/4123b95949727cb171bfcac6fcec89ea8cb76ef2.diff
LO
https://github.com/ziqingluo-90 closed
https://github.com/llvm/llvm-project/pull/75665
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ldionne created
https://github.com/llvm/llvm-project/pull/75841
This removes a long standing piece of technical debt. Most other platforms have
moved all their header search path logic to the driver, but Darwin still had
some logic for setting framework search paths present
Author: Justin Bogner
Date: 2023-12-18T11:43:52-07:00
New Revision: 4f54d71501815877898aaa6d764b8468d0708ed6
URL:
https://github.com/llvm/llvm-project/commit/4f54d71501815877898aaa6d764b8468d0708ed6
DIFF:
https://github.com/llvm/llvm-project/commit/4f54d71501815877898aaa6d764b8468d0708ed6.diff
https://github.com/bogner closed https://github.com/llvm/llvm-project/pull/75674
___
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: Louis Dionne (ldionne)
Changes
This removes a long standing piece of technical debt. Most other platforms have
moved all their header search path logic to the driver, but Darwin still had
some logic for setting framework search paths pres
@@ -758,9 +758,14 @@ void darwin::Linker::ConstructJob(Compilation &C, const
JobAction &JA,
}
}
- // Add non-standard, platform-specific search paths, e.g., for DriverKit:
- // -L/System/DriverKit/usr/lib
- // -F/System/DriverKit/System/Library/Framework
+ // Add
https://github.com/searlmc1 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/75249
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
pogo59 wrote:
>From a UI perspective, "add a visibility attribute" doesn't seem all that
>descriptive. What visibility does it add? Therefore, I suggest
>`-fvisibility-global-new-delete[=]` so the user can ask for what
>they want, and `-fvisibility-global-new-delete-hidden` equals
>`-fvisibil
@@ -1687,7 +1687,8 @@ CharUnits ASTContext::getDeclAlign(const Decl *D, bool
ForAlignof) const {
if (VD->hasGlobalStorage() && !ForAlignof) {
uint64_t TypeSize =
!BaseT->isIncompleteType() ? getTypeSize(T.getTypePtr()) : 0;
-Align = std::max(A
pogo59 wrote:
Ben and I talked about this offline. I was a little skittish about these
power-user options letting you do whatever you wanted, and maybe a small number
(2? 3?) of predefined combinations would be more straightforward. He convinced
me that (a) these wouldn't be needed often, and
https://github.com/rampitec closed
https://github.com/llvm/llvm-project/pull/75249
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rampitec closed
https://github.com/llvm/llvm-project/pull/75247
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
banach-space wrote:
No need for a dedicated test file, please use
https://github.com/llvm/llvm-project/blob/main/flang/test/Driver/dynamic-linker.f90.
https://github.com/llvm/llvm-project/pull/75598
___
cfe-commits m
@@ -0,0 +1,10 @@
+! Verify that rdynamic flag adds -export-dynamic flag and passes it on to the
linker.
+
+! RUN: %flang -### --target=x86_64-linux-gnu -rdynamic %s 2>&1 | FileCheck
--check-prefixes=GNU-LINKER-OPTIONS %s
+! RUN: %flang -### --target=aarch64-linux-none -rdynamic
https://github.com/ilovepi updated
https://github.com/llvm/llvm-project/pull/66915
>From ce9772dd519a62025cf545ded306bf40c75f2924 Mon Sep 17 00:00:00 2001
From: Paul Kirth
Date: Tue, 19 Sep 2023 20:53:54 +
Subject: [PATCH 1/4] [RISCV] Support Global Dynamic TLSDESC in the RISC-V
backend
T
banach-space wrote:
Thanks! Mostly looks good, but I have a question:
> This leads to the problem that _QQmain and _QQEnvironmentDefaults (as of the
> time of this PR) are symbols marked as used, while main is being defined.
Sorry for being pedantic, but not sure I follow. `FortranMain` defin
@@ -1133,6 +1133,16 @@ static bool isWholeArchivePresent(const ArgList &Args) {
return WholeArchiveActive;
}
+/// Determine if driver is invoked to create a shared object library (-static)
+static bool isSharedLinkage(const ArgList &Args) {
+ return Args.hasArg(options::OPT
@@ -3,18 +3,22 @@
! RUN: %flang -### --target=x86_64-linux-gnu -rpath /path/to/dir -shared \
! RUN: -static %s 2>&1 | FileCheck \
-! RUN: --check-prefixes=GNU-LINKER-OPTIONS %s
+! RUN: --check-prefixes=GNU-LINKER-OPTIONS \
+! RUN: --implicit-check-not=GNU-LINKE
hokein wrote:
Friendly ping.
https://github.com/llvm/llvm-project/pull/72428
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rampitec updated
https://github.com/llvm/llvm-project/pull/74537
>From 7e382620cdc5999c645ed0746f242595f0294c58 Mon Sep 17 00:00:00 2001
From: Stanislav Mekhanoshin
Date: Mon, 4 Dec 2023 16:11:53 -0800
Subject: [PATCH 1/9] [AMDGPU] Use alias info to relax waitcounts for LDS D
rampitec wrote:
All split off parts were merged and this patch is merged with main. Only
waitcount insertion pass changes remained here.
https://github.com/llvm/llvm-project/pull/74537
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
vgvassilev wrote:
@capfredf ping.
https://github.com/llvm/llvm-project/pull/75556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ilovepi updated
https://github.com/llvm/llvm-project/pull/66915
>From ce9772dd519a62025cf545ded306bf40c75f2924 Mon Sep 17 00:00:00 2001
From: Paul Kirth
Date: Tue, 19 Sep 2023 20:53:54 +
Subject: [PATCH 1/5] [RISCV] Support Global Dynamic TLSDESC in the RISC-V
backend
T
nathanchance wrote:
The latest version of this change causes test failures for me:
```
$ cmake \
-B build \
-G Ninja \
-S llvm \
--log-level=NOTICE \
-Wno-dev \
-DCMAKE_BUILD_TYPE=Release \
-DCMAKE_C_COMPILER=clang \
-DCMAKE_CXX_COM
rnk wrote:
> If we land this as-is, it'll tank build time on Windows.
While this is true, I don't think it's the right tradeoff for us to leave Intel
intrinsics inaccessible for users who don't want to enable new
microarchitectural features globally with command line flags. You may recall
the
@@ -3107,6 +3136,41 @@ void RISCVAsmParser::emitLoadTLSGDAddress(MCInst &Inst,
SMLoc IDLoc,
RISCV::ADDI, IDLoc, Out);
}
+void RISCVAsmParser::emitLoadTLSDescAddress(MCInst &Inst, SMLoc IDLoc,
+MCStreamer &Out) {
@@ -3246,6 +3310,18 @@ bool RISCVAsmParser::checkPseudoAddTPRel(MCInst &Inst,
return false;
}
+bool RISCVAsmParser::checkPseudoTLSDESCCall(MCInst &Inst,
ilovepi wrote:
done
https://github.com/llvm/llvm-project/pull/66915
___
mjklemm wrote:
> Thanks! Mostly looks good, but I have a question:
>
> > This leads to the problem that _QQmain and _QQEnvironmentDefaults (as of
> > the time of this PR) are symbols marked as used, while main is being
> > defined.
>
> Sorry for being pedantic, but not sure I follow. `Fortran
mjklemm wrote:
> > Also, not sure what defines `_QQEnvironmentDefaults ` and what exactly
> > makes it problematic?
>
> The `main` entrypoint also calls `_QQEnvironmentDefaults` to treat `argc`,
> `argv`, etc. So, when `main` is included in the shared object, so will be
> those symbols are us
@@ -6843,6 +6845,24 @@ SDValue
RISCVTargetLowering::getDynamicTLSAddr(GlobalAddressSDNode *N,
return LowerCallTo(CLI).first;
}
+SDValue
+RISCVTargetLowering::getGeneralDynamicTLSDescAddr(GlobalAddressSDNode *N,
MaskRay wrote:
`GeneralDynamicTLSDesc` in the
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/66915
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -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 (SanOp
bwendling wrote:
> I don't mean for this next bit to sound as aggressive or irritated as it
> might but #73168 has been open for almost a month now and it is actively
> preventing us from testing the Linux kernel against tip of tree LLVM because
> (for better or worse...) the kernel has starte
@@ -740,3 +1058,589 @@ void test10(struct union_of_fams *p, int index) {
void test11(struct annotated *p, int index) {
p->array[index] = __builtin_dynamic_object_size(&p->count, 1);
}
+
+// SANITIZE-WITH-ATTR-LABEL: define dso_local i64 @test11_bdos(
+// SANITIZE-WITH-ATTR-SA
@@ -57,6 +57,14 @@
// CHECK-MODULE-MAP-FILES: "-fmodule-map-file=foo.map"
// CHECK-MODULE-MAP-FILES: "-fmodule-map-file=bar.map"
+// RUN: %clang -fno-modules -fmodule-name=foo -c -### %s 2>&1 | FileCheck
-check-prefix=CHECK-PROPAGATE-MODULE-NAME %s
MaskRay wr
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/75827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/75827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -6843,6 +6845,24 @@ SDValue
RISCVTargetLowering::getDynamicTLSAddr(GlobalAddressSDNode *N,
return LowerCallTo(CLI).first;
}
+SDValue
+RISCVTargetLowering::getGeneralDynamicTLSDescAddr(GlobalAddressSDNode *N,
ilovepi wrote:
Well, AArch64 handles lowering
@@ -6843,6 +6845,24 @@ SDValue
RISCVTargetLowering::getDynamicTLSAddr(GlobalAddressSDNode *N,
return LowerCallTo(CLI).first;
}
+SDValue
+RISCVTargetLowering::getGeneralDynamicTLSDescAddr(GlobalAddressSDNode *N,
MaskRay wrote:
getTLSDescAddr sounds good.
h
https://github.com/cachemeifyoucan approved this pull request.
https://github.com/llvm/llvm-project/pull/74782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/karka228 updated
https://github.com/llvm/llvm-project/pull/74440
>From a80bf9d03f19d48c0aca4af7758dc49516da8825 Mon Sep 17 00:00:00 2001
From: Karl-Johan Karlsson
Date: Tue, 5 Dec 2023 10:03:00 +0100
Subject: [PATCH 1/7] [Sema] Implement support for -Wformat-signedness
In gc
Author: Jan Svoboda
Date: 2023-12-18T12:11:27-08:00
New Revision: f0691bcdf90bc44d0737e3395423e84b075ab84a
URL:
https://github.com/llvm/llvm-project/commit/f0691bcdf90bc44d0737e3395423e84b075ab84a
DIFF:
https://github.com/llvm/llvm-project/commit/f0691bcdf90bc44d0737e3395423e84b075ab84a.diff
L
https://github.com/jansvoboda11 closed
https://github.com/llvm/llvm-project/pull/74782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ilovepi updated
https://github.com/llvm/llvm-project/pull/66915
>From ce9772dd519a62025cf545ded306bf40c75f2924 Mon Sep 17 00:00:00 2001
From: Paul Kirth
Date: Tue, 19 Sep 2023 20:53:54 +
Subject: [PATCH 1/7] [RISCV] Support Global Dynamic TLSDESC in the RISC-V
backend
T
https://github.com/mehnadnerd updated
https://github.com/llvm/llvm-project/pull/69685
>From 56081a5bfe14605a692c3b49dd5c37625673aadf Mon Sep 17 00:00:00 2001
From: brs
Date: Thu, 19 Oct 2023 17:16:45 -0500
Subject: [PATCH 1/2] [RISCV][MC] MC layer support for the experimental zalasr
extension
@@ -6843,6 +6845,24 @@ SDValue
RISCVTargetLowering::getDynamicTLSAddr(GlobalAddressSDNode *N,
return LowerCallTo(CLI).first;
}
+SDValue
+RISCVTargetLowering::getGeneralDynamicTLSDescAddr(GlobalAddressSDNode *N,
ilovepi wrote:
done.
https://github.com/llvm
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 bf2e05c7fbc739cd9b9086163303f846ee2806d2
89b78a0af5547fc4c0c85368698c1bd2b0151993 --
bd1976bris wrote:
> From a UI perspective, "add a visibility attribute" doesn't seem all that
> descriptive. What visibility does it add? Therefore, I suggest
> `-fvisibility-global-new-delete[=]` so the user can ask for what
> they want, and `-fvisibility-global-new-delete-hidden` equals
> `
nickdesaulniers wrote:
> > Can we please come to some consensus so that issue can be resolved?
>
> This supposed bug fix snowballed out of control. I'd be happy to revert to a
> previous version and then submit the newer work as a improvements.
Linux kernel builds have been broken since the in
bwendling wrote:
> > > Can we please come to some consensus so that issue can be resolved?
> >
> >
> > This supposed bug fix snowballed out of control. I'd be happy to revert to
> > a previous version and then submit the newer work as a improvements.
>
> Linux kernel builds have been broken s
banach-space wrote:
> Sure, I was planning to do this for the previous PR already but got
> distracted. Can you point me to the right place to add this? Thanks!
Somewhere here:
* https://github.com/llvm/llvm-project/blob/main/flang/docs/FlangDriver.md
I would probably create a section on build
https://github.com/kadircet updated
https://github.com/llvm/llvm-project/pull/75612
From 8801af07ad4e469f832570b5027e1522f41a6d06 Mon Sep 17 00:00:00 2001
From: Kadir Cetinkaya
Date: Fri, 15 Dec 2023 15:14:05 +0100
Subject: [PATCH 1/2] [clangd] Track IWYU pragmas for non-preamble includes
---
@@ -113,7 +113,7 @@ class PragmaIncludes {
llvm::DenseSet ShouldKeep;
/// Owns the strings.
- llvm::BumpPtrAllocator Arena;
+ std::shared_ptr Arena;
kadircet wrote:
you're right in theory, but in practice we actually never build ASTs
concurrently from
https://github.com/jamesETsmith updated
https://github.com/llvm/llvm-project/pull/68494
>From c4a3ccfbad090ad8314aa8ad53092edc8d5432bc Mon Sep 17 00:00:00 2001
From: James Smith
Date: Thu, 28 Sep 2023 10:11:15 -0400
Subject: [PATCH 01/18] [libc++] Implement ranges::iota and
ranges::out_value_r
Author: Paul Kirth
Date: 2023-12-18T13:03:13-08:00
New Revision: d1e2b96b606e5bf8329ab5821f7a614163a56e16
URL:
https://github.com/llvm/llvm-project/commit/d1e2b96b606e5bf8329ab5821f7a614163a56e16
DIFF:
https://github.com/llvm/llvm-project/commit/d1e2b96b606e5bf8329ab5821f7a614163a56e16.diff
LO
https://github.com/ilovepi closed
https://github.com/llvm/llvm-project/pull/75079
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bryanpkc approved this pull request.
This looks reasonable to me, but I'll wait for other reviewers to look at this.
Please rebase on main and make sure the CI workflows succeed.
https://github.com/llvm/llvm-project/pull/75516
___
c
https://github.com/bwendling created
https://github.com/llvm/llvm-project/pull/75857
There are many issues that popped up with the counted_by feature. The patch has
grown too large and approval is blocking Linux testing.
Includes reverts of:
commit 769bc11f684d ("[Clang] Implement the 'counted
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Bill Wendling (bwendling)
Changes
There are many issues that popped up with the counted_by feature. The patch has
grown too large and approval is blocking Linux testing.
Includes reverts of:
commit 769bc11f684d ("[Clang] Implement
bwendling wrote:
Here's a revert of the feature.
https://github.com/llvm/llvm-project/pull/75857
https://github.com/llvm/llvm-project/pull/73730
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Bill Wendling (bwendling)
Changes
There are many issues that popped up with the counted_by feature. The patch has
grown too large and approval is blocking Linux testing.
Includes reverts of:
commit 769bc11f684d ("[Clang] Implement the 'co
nickdesaulniers wrote:
@nathanchance can you test this please to verify it's unbreaking the linux
kernel builds?
https://github.com/llvm/llvm-project/pull/75857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
@@ -758,9 +758,14 @@ void darwin::Linker::ConstructJob(Compilation &C, const
JobAction &JA,
}
}
- // Add non-standard, platform-specific search paths, e.g., for DriverKit:
- // -L/System/DriverKit/usr/lib
- // -F/System/DriverKit/System/Library/Framework
+ // Add
dtemirbulatov wrote:
> Should these builtins be `IsStreamingCompatible` until we add
> `IsStreamingOrSVE2p1`?
yes, it looks like this is the case. LGTM.
https://github.com/llvm/llvm-project/pull/75821
___
cfe-commits mailing list
cfe-commits@lists.ll
nathanchance wrote:
> @nathanchance can you test this please to verify it's unbreaking the linux
> kernel builds?
A quick initial test shows this resolves the two cases that I found in
https://github.com/llvm/llvm-project/issues/73168. I can do a fuller set of
builds if necessary but since th
https://github.com/dtemirbulatov approved this pull request.
https://github.com/llvm/llvm-project/pull/75821
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
efriedma-quic wrote:
It doesn't seem like a good idea to make target-independent logic behave in
target-specific ways; that's going to confusing for both people hacking on
clang, and for users if it's user-visible. Is there some way we can make this
logic consistent across targets?
Maybe we
MaxEW707 wrote:
> Is intrin0.h a header that ships with MSVC's compiler, or with MS's STL? If
> the latter, shouldn't everything just work already?
It ships with MSVC.
`intrin0.h` from MSVC doesn't work currently due to some declarations not
matching declarations in clang's own `intrin.h`.
Fo
https://github.com/nickdesaulniers approved this pull request.
https://github.com/llvm/llvm-project/pull/75857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,303 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
DanielCChen wrote:
@mjklemm This PR caused some regressions of C-interop test cases in our local
test run. The test cases typically have a Fortran main (compiled with Flang)
that calls a C function (compiled with clang). The linking is by `flang-new`.
The error looks like:
```
ld.lld: error: d
https://github.com/bwendling edited
https://github.com/llvm/llvm-project/pull/75857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bwendling edited
https://github.com/llvm/llvm-project/pull/75857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,145 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,145 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
banach-space wrote:
> @mjklemm This PR caused some regressions of C-interop test cases in our local
> test run. The test cases typically have a Fortran main (compiled with Flang)
> that calls a C function (compiled with clang).
Sorry that you are hitting this, but please note that compiling m
@@ -8522,6 +8522,22 @@ bool Sema::CheckCountedByAttr(Scope *S, const FieldDecl
*FD) {
}
}
+ // We don't support 'counted_by' on flexible array members in substructures.
+ const DeclContext *DC = FD->getParent();
+ while (const auto *RD = dyn_cast(DC)) {
+if (!RD-
@@ -8522,6 +8522,22 @@ bool Sema::CheckCountedByAttr(Scope *S, const FieldDecl
*FD) {
}
}
+ // We don't support 'counted_by' on flexible array members in substructures.
+ const DeclContext *DC = FD->getParent();
+ while (const auto *RD = dyn_cast(DC)) {
+if (!RD-
@@ -3946,6 +4080,32 @@ static Address emitArraySubscriptGEP(CodeGenFunction
&CGF, Address addr,
return Address(eltPtr, CGF.ConvertTypeForMem(eltType), eltAlign);
}
+static bool GetFieldOffsetInBits(CodeGenFunction &CGF, const RecordDecl *RD,
+
https://github.com/mariusz-sikora-at-amd updated
https://github.com/llvm/llvm-project/pull/75625
>From de5303eb8a9e061dbd365922f85cad02bca5ec26 Mon Sep 17 00:00:00 2001
From: Stanislav Mekhanoshin
Date: Tue, 5 Jul 2022 11:41:29 -0700
Subject: [PATCH 1/3] GFX12: Add LoopDataPrefetchPass
It is c
https://github.com/mariusz-sikora-at-amd updated
https://github.com/llvm/llvm-project/pull/75625
>From de5303eb8a9e061dbd365922f85cad02bca5ec26 Mon Sep 17 00:00:00 2001
From: Stanislav Mekhanoshin
Date: Tue, 5 Jul 2022 11:41:29 -0700
Subject: [PATCH 1/4] GFX12: Add LoopDataPrefetchPass
It is c
https://github.com/rapidsna approved this pull request.
https://github.com/llvm/llvm-project/pull/75857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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 33d5f4314f4fd83be21e22054e662220a62fe40b
6296613d1d3f3e8b63e1af1d12d28f2000facc0e --
@@ -8522,6 +8522,22 @@ bool Sema::CheckCountedByAttr(Scope *S, const FieldDecl
*FD) {
}
}
+ // We don't support 'counted_by' on flexible array members in substructures.
+ const DeclContext *DC = FD->getParent();
+ while (const auto *RD = dyn_cast(DC)) {
+if (!RD-
@@ -8522,6 +8522,22 @@ bool Sema::CheckCountedByAttr(Scope *S, const FieldDecl
*FD) {
}
}
+ // We don't support 'counted_by' on flexible array members in substructures.
+ const DeclContext *DC = FD->getParent();
+ while (const auto *RD = dyn_cast(DC)) {
+if (!RD-
@@ -3946,6 +4080,32 @@ static Address emitArraySubscriptGEP(CodeGenFunction
&CGF, Address addr,
return Address(eltPtr, CGF.ConvertTypeForMem(eltType), eltAlign);
}
+static bool GetFieldOffsetInBits(CodeGenFunction &CGF, const RecordDecl *RD,
+
@@ -8522,6 +8522,22 @@ bool Sema::CheckCountedByAttr(Scope *S, const FieldDecl
*FD) {
}
}
+ // We don't support 'counted_by' on flexible array members in substructures.
+ const DeclContext *DC = FD->getParent();
+ while (const auto *RD = dyn_cast(DC)) {
+if (!RD-
@@ -758,9 +758,14 @@ void darwin::Linker::ConstructJob(Compilation &C, const
JobAction &JA,
}
}
- // Add non-standard, platform-specific search paths, e.g., for DriverKit:
- // -L/System/DriverKit/usr/lib
- // -F/System/DriverKit/System/Library/Framework
+ // Add
@@ -3946,6 +4080,32 @@ static Address emitArraySubscriptGEP(CodeGenFunction
&CGF, Address addr,
return Address(eltPtr, CGF.ConvertTypeForMem(eltType), eltAlign);
}
+static bool GetFieldOffsetInBits(CodeGenFunction &CGF, const RecordDecl *RD,
+
201 - 300 of 419 matches
Mail list logo
