@@ -311,7 +318,10 @@ computeBlockInputState(const CFGBlock &Block,
AnalysisContext &AC) {
}
}
- JoinedStateBuilder Builder(AC);
+ // When performing the join, only retain state for those expressions that are
+ // consumed by this block. This avoids performing joins a
https://github.com/martinboehme edited
https://github.com/llvm/llvm-project/pull/72850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
martinboehme wrote:
> +1 for the simpler approach. I have a high-level question about the stability
> of the locations, its effect on convergence.
Answered in the inline comment.
> But I am happy with the `ExprToVal` part of the simple patch.
SG! Then let's continue the discussion on #72985?
https://github.com/sdesmalen-arm closed
https://github.com/llvm/llvm-project/pull/72979
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Sander de Smalen
Date: 2023-11-22T08:52:53Z
New Revision: 81b7f115fb272ef6fd6967f4121b64814b4bcf10
URL:
https://github.com/llvm/llvm-project/commit/81b7f115fb272ef6fd6967f4121b64814b4bcf10
DIFF:
https://github.com/llvm/llvm-project/commit/81b7f115fb272ef6fd6967f4121b64814b4bcf10.diff
L
https://github.com/jcsxky created
https://github.com/llvm/llvm-project/pull/73096
This patch aims to fix [error in ast-merge to new ast
file](https://github.com/llvm/llvm-project/issues/72783).
`ASTUnit` is put in `for` body and AST nodes would be deallocated by allocator.
Using these nodes la
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Qizhi Hu (jcsxky)
Changes
This patch aims to fix [error in ast-merge to new ast
file](https://github.com/llvm/llvm-project/issues/72783).
`ASTUnit` is put in `for` body and AST nodes would be deallocated by allocator.
Using these nodes la
https://github.com/gamesh411 updated
https://github.com/llvm/llvm-project/pull/71912
From 977e421008c1247d54f4cb67967ed2a353935c03 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Endre=20F=C3=BCl=C3=B6p?=
Date: Fri, 10 Nov 2023 10:08:58 +0100
Subject: [PATCH 1/2] [analyzer] Move security.cert.env.Inva
https://github.com/simpal01 closed
https://github.com/llvm/llvm-project/pull/71545
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1009,11 +1002,20 @@ let ParentPackage = ENV in {
"standard), which can lead to false positives depending on "
"implementation.",
"false",
- InAlpha>,
+ Released>,
]>,
Documentation
https://github.com/Fznamznon created
https://github.com/llvm/llvm-project/pull/73099
The implementation mostly reuses C++ code paths where possible, including
narrowing check in order to provide diagnostic messages in case initializer for
constexpr variable is not exactly representable in targ
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Mariya Podchishchaeva (Fznamznon)
Changes
The implementation mostly reuses C++ code paths where possible, including
narrowing check in order to provide diagnostic messages in case initializer for
constexpr variable is not exactly represen
Fznamznon wrote:
cc @to268 , GitHub doesn't let me add you to the reviewers list.
https://github.com/llvm/llvm-project/pull/73099
___
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 74cdb8e6f8c88b97204f540601a553b412d1cd56
1d70b7726e7d1f11622a6d5c8246b0737e024c8d --
to268 wrote:
That's probably because I have still not commit access, but I'll take a look.
https://github.com/llvm/llvm-project/pull/73099
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
Author: Jonas Paulsson
Date: 2023-11-22T11:24:45+01:00
New Revision: 2e09ea65c4f9e458ffb31fe63d84a4991704f9e6
URL:
https://github.com/llvm/llvm-project/commit/2e09ea65c4f9e458ffb31fe63d84a4991704f9e6
DIFF:
https://github.com/llvm/llvm-project/commit/2e09ea65c4f9e458ffb31fe63d84a4991704f9e6.diff
https://github.com/JonPsson1 closed
https://github.com/llvm/llvm-project/pull/72886
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4110,6 +4117,10 @@ static CompleteObject findCompleteObject(EvalInfo &Info,
const Expr *E,
}
bool IsConstant = BaseType.isConstant(Info.Ctx);
+bool ConstexprVar = false;
+if (const auto *VD = dyn_cast_or_null(
tbaederr wrote:
```suggestio
@@ -8605,6 +8620,27 @@ static bool checkForConflictWithNonVisibleExternC(Sema
&S, const T *ND,
return false;
}
+static bool CheckC23ConstexprVarTypeQualifiers(Sema &SemaRef,
+SourceLocation VarLoc, QualType T)
{
+ if (const auto
@@ -2231,6 +2231,13 @@ static bool CheckLValueConstantExpression(EvalInfo
&Info, SourceLocation Loc,
return false;
}
+ if (Info.getLangOpts().C23) {
+auto *VarD = dyn_cast_or_null(BaseVD);
+if (VarD && VarD->isConstexpr() && !LVal.isNullPointer()) {
+ Info
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,99 @@
+//===--- RedundantInlineSpecifierCheck.cpp -
+// clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/
martinboehme wrote:
Update: I've run some tests on an internal codebase, and with this PR, Crubit's
nullability check sees a reduction in SAT solver timeouts of over 40% and a
reduction in "reached maximum iterations" errors of over 60% -- similar to the
previous more complicated approach in #
@@ -0,0 +1,32 @@
+// RUN: %clang --target=s390x-linux -S -emit-llvm -o - %s | FileCheck %s
nikic wrote:
Why is this a Driver test rather than a CodeGen test using clang_cc1?
https://github.com/llvm/llvm-project/pull/72886
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,99 @@
+//===--- RedundantInlineSpecifierCheck.cpp -
+// clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,99 @@
+//===--- RedundantInlineSpecifierCheck.cpp -
+// clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,99 @@
+//===--- RedundantInlineSpecifierCheck.cpp -
+// clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,99 @@
+//===--- RedundantInlineSpecifierCheck.cpp -
+// clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,99 @@
+//===--- RedundantInlineSpecifierCheck.cpp -
+// clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -7889,7 +7889,7 @@ AST_POLYMORPHIC_MATCHER(isInline,
AST_POLYMORPHIC_SUPPORTED_TYPES(NamespaceDecl,
if (const auto *NSD = dyn_cast(&Node))
return NSD->isInline();
if (const auto *VD = dyn_cast(&Node))
-
=?utf-8?q?F=C3=A9lix-Antoine?= Constantin
Message-ID:
In-Reply-To:
https://github.com/PiotrZSL requested changes to this pull request.
Not bad, just few nits.
https://github.com/llvm/llvm-project/pull/73069
___
cfe-commits mailing list
cfe-commits@li
https://github.com/JonPsson1 updated
https://github.com/llvm/llvm-project/pull/72977
>From 000dcadc0fd118df643e3f2ecbe5fcbb2f8eaab0 Mon Sep 17 00:00:00 2001
From: Jonas Paulsson
Date: Tue, 21 Nov 2023 12:10:03 +0100
Subject: [PATCH 1/3] Refactor ASTContext::getDeclAlign() (NFC)
---
clang/lib/
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
philnik777 wrote:
@PiotrZSL could you use the "Start review" button to avoid spamming people with
dozens of E-Mails for a single review?
https://github.com/llvm/llvm-project/pull/73069
__
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/73103
https://wiki.edg.com/pub/Wg21kona2023/StrawPolls/p2308r1.html
This implements P2308R1 as a DR and resolves CWG2459, CWG2450 and CWG2049.
>From 90a9c92ce532af7c6346a381499780f158cd26ca Mon Sep 17 00:00:00 2001
F
@@ -1627,28 +1627,20 @@ const llvm::fltSemantics
&ASTContext::getFloatTypeSemantics(QualType T) const {
CharUnits ASTContext::getDeclAlign(const Decl *D, bool ForAlignof) const {
unsigned Align = Target->getCharWidth();
- bool UseAlignAttrOnly = false;
- if (unsigned Alig
https://github.com/DominikAdamski updated
https://github.com/llvm/llvm-project/pull/72638
>From e5d339c24193e4e37013b3b25460009418d6ce6d Mon Sep 17 00:00:00 2001
From: Dominik Adamski
Date: Fri, 17 Nov 2023 03:02:49 -0600
Subject: [PATCH 1/2] [Flang] Add code-object-version option
Information
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: cor3ntin (cor3ntin)
Changes
https://wiki.edg.com/pub/Wg21kona2023/StrawPolls/p2308r1.html
This implements P2308R1 as a DR and resolves CWG2459, CWG2450 and CWG2049.
---
Full diff: https://github.com/llvm/llvm-project/pull/73103.diff
11
@@ -0,0 +1,32 @@
+// RUN: %clang --target=s390x-linux -S -emit-llvm -o - %s | FileCheck %s
JonPsson1 wrote:
I found some other align tests there, like arm-alignment.c, so I thought this
should work..?
https://github.com/llvm/llvm-project/pull/72886
@@ -264,6 +263,37 @@ static void addDepdendentLibs(mlir::ModuleOp &mlirModule,
}
}
+// Add to MLIR code target specific items which are dependent on target
+// configuration specified by the user
+static void addTargetSpecificMLIRItems(mlir::ModuleOp &mlirModule,
+
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
PiotrZSL wrote:
> @PiotrZSL could you use the "Start review" button to avoid spamming people
> with dozens of E-Mails for a single review?
@philnik777 If you prefer not to receive notifications, simply unsubscribe.
Specially if yo
@@ -264,6 +263,37 @@ static void addDepdendentLibs(mlir::ModuleOp &mlirModule,
}
}
+// Add to MLIR code target specific items which are dependent on target
+// configuration specified by the user
+static void addTargetSpecificMLIRItems(mlir::ModuleOp &mlirModule,
+
@@ -264,6 +263,37 @@ static void addDepdendentLibs(mlir::ModuleOp &mlirModule,
}
}
+// Add to MLIR code target specific items which are dependent on target
+// configuration specified by the user
+static void addTargetSpecificMLIRItems(mlir::ModuleOp &mlirModule,
+
DominikAdamski wrote:
Done
https://github.com/llvm/llvm-project/pull/72638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,32 @@
+// RUN: %clang_cc1 -std=c++98 -triple x86_64-unknown-unknown %s -verify
-fexceptions -fcxx-exceptions -pedantic-errors \
Endilll wrote:
Is triple necessary for the test?
https://github.com/llvm/llvm-project/pull/73103
_
https://github.com/Endilll edited
https://github.com/llvm/llvm-project/pull/73103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Endilll approved this pull request.
DR test side looks good, except for a small nit.
https://github.com/llvm/llvm-project/pull/73103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
saiislam wrote:
> (patches like this should probably be broken up - test changes to the
> defaults in lld and llvm for instance don't depend on the change to the clang
> driver which is the only real semantic change in this patch, right? So
> probably only change the semantics of clang, and th
@@ -264,6 +263,37 @@ static void addDepdendentLibs(mlir::ModuleOp &mlirModule,
}
}
+// Add to MLIR code target specific items which are dependent on target
+// configuration specified by the user
+static void addTargetSpecificMLIRItems(mlir::ModuleOp &mlirModule,
+
@@ -85,6 +85,19 @@ class CodeGenOptions : public CodeGenOptionsBase {
RK_WithPattern, // Remark pattern specified via '-Rgroup=regexp'.
};
+ /// \brief Enumeration value for AMDGPU code object version, which is the
+ /// code object version times 100.
+ enum class Cod
@@ -264,6 +263,37 @@ static void addDepdendentLibs(mlir::ModuleOp &mlirModule,
}
}
+// Add to MLIR code target specific items which are dependent on target
+// configuration specified by the user
+static void addTargetSpecificMLIRItems(mlir::ModuleOp &mlirModule,
+
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/73105
… Enumerations
https://isocpp.org/files/papers/P2864R2.pdf
>From f548a8e11c75b3157a9337892688f5414234bff8 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Wed, 22 Nov 2023 12:25:20 +0100
Subject: [PATCH] [C
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: cor3ntin (cor3ntin)
Changes
… Enumerations
https://isocpp.org/files/papers/P2864R2.pdf
---
Full diff: https://github.com/llvm/llvm-project/pull/73105.diff
5 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+3)
- (modified) cl
@@ -1847,6 +1862,7 @@ bool
SIInsertWaitcnts::runOnMachineFunction(MachineFunction &MF) {
TrackedWaitcntSet.clear();
BlockInfos.clear();
+
jayfoad wrote:
Remove this
https://github.com/llvm/llvm-project/pull/68932
@@ -1708,6 +1710,19 @@ bool
SIInsertWaitcnts::insertWaitcntInBlock(MachineFunction &MF,
}
++Iter;
+if (ST->isPreciseMemoryEnabled() && Inst.mayLoadOrStore()) {
+ auto Builder =
+ BuildMI(Block, Iter, DebugLoc(), TII->get(AMDGPU::S_WAITCNT))
+
@@ -1708,6 +1710,19 @@ bool
SIInsertWaitcnts::insertWaitcntInBlock(MachineFunction &MF,
}
++Iter;
+if (ST->isPreciseMemoryEnabled() && Inst.mayLoadOrStore()) {
+ auto Builder =
+ BuildMI(Block, Iter, DebugLoc(), TII->get(AMDGPU::S_WAITCNT))
+
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
philnik777 wrote:
> > @PiotrZSL could you use the "Start review" button to avoid spamming people
> > with dozens of E-Mails for a single review?
>
> @philnik777 If you prefer not to receive notifications, simply unsubscribe.
> Spe
@@ -85,6 +85,19 @@ class CodeGenOptions : public CodeGenOptionsBase {
RK_WithPattern, // Remark pattern specified via '-Rgroup=regexp'.
};
+ /// \brief Enumeration value for AMDGPU code object version, which is the
+ /// code object version times 100.
+ enum class Cod
dmgreen added a comment.
I hadn't realized this came from someone at Arm. The performance results I had
were overall roughly flat, with some improvements and regressions. I think
there were still some people working through some fixes for some of the
knock-on effects but with those nothing larg
https://github.com/JonPsson1 updated
https://github.com/llvm/llvm-project/pull/72977
>From 000dcadc0fd118df643e3f2ecbe5fcbb2f8eaab0 Mon Sep 17 00:00:00 2001
From: Jonas Paulsson
Date: Tue, 21 Nov 2023 12:10:03 +0100
Subject: [PATCH 1/4] Refactor ASTContext::getDeclAlign() (NFC)
---
clang/lib/
owenpan accepted this revision.
owenpan added a comment.
This revision is now accepted and ready to land.
LGTM except the final nits.
Comment at: clang/lib/Format/ObjCPropertyAttributeOrderFixer.cpp:48
+const FormatToken *BeginTok, const FormatToken *EndTok) const {
+ // I
https://github.com/huixie90 updated
https://github.com/llvm/llvm-project/pull/68597
>From 617d6d23b2f861cd6dceb82f54a2685059b6 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Thu, 14 Sep 2023 09:09:29 +
Subject: [PATCH] [flang] Enable alias tags pass by default
Enable by default when o
https://github.com/tblah closed https://github.com/llvm/llvm-project/pull/68597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tblah wrote:
Sorry for the noise all. Closing and re-creating to avoid pinging everyone with
review
https://github.com/llvm/llvm-project/pull/68597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
Author: Utkarsh Saxena
Date: 2023-11-22T12:51:24+01:00
New Revision: fbfd2c9972695bc74a646bfa5e3e2d00477dabe2
URL:
https://github.com/llvm/llvm-project/commit/fbfd2c9972695bc74a646bfa5e3e2d00477dabe2
DIFF:
https://github.com/llvm/llvm-project/commit/fbfd2c9972695bc74a646bfa5e3e2d00477dabe2.diff
https://github.com/usx95 closed https://github.com/llvm/llvm-project/pull/72851
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah created https://github.com/llvm/llvm-project/pull/73111
Enable by default when optimizing for speed.
For simplicity, only forward the flag to the frontend driver when it
contradicts what is implied by the optimization level.
Since https://github.com/llvm/llvm-project/p
llvmbot wrote:
@llvm/pr-subscribers-mc
@llvm/pr-subscribers-backend-aarch64
Author: None (CarolineConcatto)
Changes
…d LLVM intrinisc (#70362)""
This reverts commit e1ee0e85104eed2c68b6821d9e5a2066e4154099.
The patch https://github.com/llvm/llvm-project/pull/70362 had a test in LLDB
fai
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (CarolineConcatto)
Changes
…d LLVM intrinisc (#70362)""
This reverts commit e1ee0e85104eed2c68b6821d9e5a2066e4154099.
The patch https://github.com/llvm/llvm-project/pull/70362 had a test in LLDB
failing. The Feature sve2p1 in AArch6
llvmbot wrote:
@llvm/pr-subscribers-flang-driver
Author: Tom Eccles (tblah)
Changes
Enable by default when optimizing for speed.
For simplicity, only forward the flag to the frontend driver when it
contradicts what is implied by the optimization level.
Since https://github.com/llvm/llvm
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/73105
>From 6c02a92f2d2dbd426bf9e5fd34cc0f32203a0283 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Wed, 22 Nov 2023 12:25:20 +0100
Subject: [PATCH] [Clang] Implement P28646R2 Remove Deprecated Arithmetic
Conver
@@ -311,7 +311,7 @@ inline void createDefaultFIRCodeGenPassPipeline(
if (config.VScaleMin != 0)
pm.addPass(fir::createVScaleAttrPass({config.VScaleMin,
config.VScaleMax}));
- fir::addFIRToLLVMPass(pm, config.OptLevel);
+ fir::addFIRToLLVMPass(pm, config.OptLevel, conf
@@ -242,10 +242,24 @@ static void
parseCodeGenArgs(Fortran::frontend::CodeGenOptions &opts,
clang::driver::options::OPT_fno_loop_versioning, false))
opts.LoopVersioning = 1;
- opts.AliasAnalysis =
- args.hasFlag(clang::driver::options::OPT_falias_
@@ -242,10 +242,24 @@ static void
parseCodeGenArgs(Fortran::frontend::CodeGenOptions &opts,
clang::driver::options::OPT_fno_loop_versioning, false))
opts.LoopVersioning = 1;
- opts.AliasAnalysis =
- args.hasFlag(clang::driver::options::OPT_falias_
https://github.com/DavidSpickett approved this pull request.
LGTM, thank you for the fix!
You'll know within ~15 minutes of landing whether there is still an issue, but
I doubt there will be.
https://github.com/llvm/llvm-project/pull/73110
___
cfe-co
@@ -1355,6 +1362,10 @@ unsigned UnwrappedLineFormatter::format(
bool FixIndentation = (FixBadIndentation || ContinueFormatting) &&
Indent != TheLine.First->OriginalColumn;
bool ShouldFormat = TheLine.Affected || FixIndentation;
+
+if (Style
https://github.com/tomekpaszek updated
https://github.com/llvm/llvm-project/pull/70338
>From b5ba0b3fde2c6662e19dfdf96a787621ec767460 Mon Sep 17 00:00:00 2001
From: Tomek Paszek
Date: Sat, 11 Nov 2023 19:38:00 +0100
Subject: [PATCH 01/11] Added an option to ignore macro definitions.
---
clang
https://github.com/ymand approved this pull request.
https://github.com/llvm/llvm-project/pull/72985
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
FlameTop wrote:
@pogo59 I agree that stating the PS ABI has an explicit function alignment
unless overridden does negate the need for a new attribute. I will re-work the
code to remove the new attribute and instead enforce it for the PS ABI target.
https://github.com/llvm/llvm-project/pull/7
https://github.com/CarolineConcatto closed
https://github.com/llvm/llvm-project/pull/73110
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yi-wu-arm updated
https://github.com/llvm/llvm-project/pull/70917
>From 0e98aa7ca15b05b91813eaeeb6ae1305e5f5384d Mon Sep 17 00:00:00 2001
From: Yi Wu
Date: Mon, 6 Nov 2023 19:49:13 +
Subject: [PATCH 1/7] GETLOG runtime and extension implementation: get login
username
Ge
https://github.com/bjosv created https://github.com/llvm/llvm-project/pull/73119
This check implements the [rule
17.5.1](https://www.perforce.com/resources/qac/high-integrity-cpp-coding-standard/standard-library)
of the HICPP standard which states:
- Do not ignore the result of std::remove, st
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Björn Svensson (bjosv)
Changes
This check implements the [rule
17.5.1](https://www.perforce.com/resources/qac/high-integrity-cpp-coding-standard/standard-library)
of the HICPP standard which states:
- Do not ignore the result of std
@@ -37,5 +80,17 @@ void FORTRAN_PROCEDURE_NAME(getarg)(
(void)RTNAME(GetCommandArgument)(
n, &value, nullptr, nullptr, __FILE__, __LINE__);
}
+
+void FORTRAN_PROCEDURE_NAME(getlog)(std::int8_t *arg, std::int64_t length) {
+ std::array str = {};
+
+ int error = getlogi
@@ -10,9 +10,52 @@
// extensions that will eventually be implemented in Fortran.
#include "flang/Runtime/extensions.h"
+#include "terminator.h"
#include "flang/Runtime/command.h"
#include "flang/Runtime/descriptor.h"
#include "flang/Runtime/io-api.h"
+#include "flang/Runtim
https://github.com/qcolombet approved this pull request.
Thanks for your patience iterating through this.
LGTM
https://github.com/llvm/llvm-project/pull/70778
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm
https://github.com/egorzhdan created
https://github.com/llvm/llvm-project/pull/73120
This upstreams more of the Clang API Notes functionality that is currently
implemented in the Apple fork:
https://github.com/apple/llvm-project/tree/next/clang/lib/APINotes
>From 41f0d8ce31e656288cb5753c37282
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Egor Zhdan (egorzhdan)
Changes
This upstreams more of the Clang API Notes functionality that is currently
implemented in the Apple fork:
https://github.com/apple/llvm-project/tree/next/clang/lib/APINotes
---
Full diff: https://git
Author: Phoebe Wang
Date: 2023-11-22T21:59:30+08:00
New Revision: e5cc3da6a9077548f613eee3aacc5e7b017c81f3
URL:
https://github.com/llvm/llvm-project/commit/e5cc3da6a9077548f613eee3aacc5e7b017c81f3
DIFF:
https://github.com/llvm/llvm-project/commit/e5cc3da6a9077548f613eee3aacc5e7b017c81f3.diff
L
@@ -1026,6 +1030,71 @@ void SystemZAsmPrinter::emitADASection() {
OutStreamer->popSection();
}
+static uint32_t getProductVersion(Module &M) {
+ if (auto *VersionVal = mdconst::extract_or_null(
+ M.getModuleFlag("zos_product_major_version")))
+return VersionVal
@@ -1026,6 +1030,71 @@ void SystemZAsmPrinter::emitADASection() {
OutStreamer->popSection();
}
+static uint32_t getProductVersion(Module &M) {
+ if (auto *VersionVal = mdconst::extract_or_null(
+ M.getModuleFlag("zos_product_major_version")))
+return VersionVal
@@ -1026,6 +1030,78 @@ void SystemZAsmPrinter::emitADASection() {
OutStreamer->popSection();
}
+static uint32_t getProductVersion(Module &M) {
+ if (auto *VersionVal = cast_or_null(
+ M.getModuleFlag("zos_product_major_version")))
+return cast(VersionVal->getVa
https://github.com/yi-wu-arm updated
https://github.com/llvm/llvm-project/pull/70917
>From 0e98aa7ca15b05b91813eaeeb6ae1305e5f5384d Mon Sep 17 00:00:00 2001
From: Yi Wu
Date: Mon, 6 Nov 2023 19:49:13 +
Subject: [PATCH 1/8] GETLOG runtime and extension implementation: get login
username
Ge
@@ -37,5 +80,17 @@ void FORTRAN_PROCEDURE_NAME(getarg)(
(void)RTNAME(GetCommandArgument)(
n, &value, nullptr, nullptr, __FILE__, __LINE__);
}
+
+void FORTRAN_PROCEDURE_NAME(getlog)(std::int8_t *arg, std::int64_t length) {
+ std::array str = {};
+
+ int error = getlogi
https://github.com/yi-wu-arm updated
https://github.com/llvm/llvm-project/pull/70917
>From 0e98aa7ca15b05b91813eaeeb6ae1305e5f5384d Mon Sep 17 00:00:00 2001
From: Yi Wu
Date: Mon, 6 Nov 2023 19:49:13 +
Subject: [PATCH 1/9] GETLOG runtime and extension implementation: get login
username
Ge
@@ -976,6 +976,46 @@ void CodeGenModule::Release() {
Context.getTypeSizeInChars(Context.getWideCharType()).getQuantity();
getModule().addModuleFlag(llvm::Module::Error, "wchar_size", WCharWidth);
+ if (getTriple().isOSzOS()) {
+getModule().addModuleFlag(llvm::Modu
https://github.com/mjklemm created
https://github.com/llvm/llvm-project/pull/73124
The flang driver was silently ignoring the `main()` function in
`Fortran_main.a` for entry into the Fortran program unit if an external
`main()` as supplied (e.g., via cross-language linkage with Fortran and C/C
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Michael Klemm (mjklemm)
Changes
The flang driver was silently ignoring the `main()` function in
`Fortran_main.a` for entry into the Fortran program unit if an external
`main()` as supplied (e.g., via cross-language linkage with Fortran an
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Michael Klemm (mjklemm)
Changes
The flang driver was silently ignoring the `main()` function in
`Fortran_main.a` for entry into the Fortran program unit if an external
`main()` as supplied (e.g., via cross-language linkage with For
@@ -1029,7 +1042,7 @@ void tools::addFortranRuntimeLibraryPath(const ToolChain
&TC,
ArgStringList &CmdArgs) {
// Default to the /../lib directory. This works fine on the
// platforms that we have tested so far. We will probably have
https://github.com/mjklemm updated
https://github.com/llvm/llvm-project/pull/73124
>From ba38aec7ac04c63fd5167908fe7f91d6ac7bceed Mon Sep 17 00:00:00 2001
From: Michael Klemm
Date: Wed, 22 Nov 2023 14:22:20 +0100
Subject: [PATCH 1/2] Let the linker fail on multiple definitions of main()
---
c
@@ -1029,7 +1042,7 @@ void tools::addFortranRuntimeLibraryPath(const ToolChain
&TC,
ArgStringList &CmdArgs) {
// Default to the /../lib directory. This works fine on the
// platforms that we have tested so far. We will probably have
1 - 100 of 361 matches
Mail list logo
