Author: martinboehme
Date: 2023-09-19T09:03:20+02:00
New Revision: 1d7b59ca8ddb0b189a036a8f7e26f7e6deb73038
URL:
https://github.com/llvm/llvm-project/commit/1d7b59ca8ddb0b189a036a8f7e26f7e6deb73038
DIFF:
https://github.com/llvm/llvm-project/commit/1d7b59ca8ddb0b189a036a8f7e26f7e6deb73038.diff
https://github.com/martinboehme closed
https://github.com/llvm/llvm-project/pull/66742
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dtcxzyw wrote:
> We should use rebase instead of merge. And I recommend force push to change
> the code before the review starts.
I am confused with the LLVM GitHub user guide. I witnessed someone mess up by
doing rebase + force push.
Examples: #65853 #65543 #65535
https://github.com/llvm/ll
jcsxky added a comment.
ping~ @donat.nagy @steakhal @balazske
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159519/new/
https://reviews.llvm.org/D159519
___
cfe-commits mailing list
cfe-commits@lists.l
@@ -20,8 +20,8 @@
extern "C" {
kbeyls wrote:
Thanks, all of these updates look correct to me.
Since these all refer to sections in a document, I think it may be useful too
to add a hyperlink at the top of the document to where the ACLE lives.
I think the correc
@@ -20,8 +20,8 @@
extern "C" {
vhscampos wrote:
Thanks @Blue-Dot for the patch. I agree with @kbeyls that it would be a good
idea.
https://github.com/llvm/llvm-project/pull/2
___
cfe-commits mailing list
cfe-comm
qiucf abandoned this revision.
qiucf added a comment.
Determining float ABI by system library without explicit options may cause
confusion. Since Fedora has successfully switched using ieeelongdouble on
ppc64le
(https://developers.redhat.com/articles/2023/05/16/benefits-fedora-38-long-double-tr
qiucf added a comment.
Gentle ping?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116016/new/
https://reviews.llvm.org/D116016
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.o
petrhosek wrote:
To give you a concrete example, I recently encountered this issue when
compiling [libffi](https://github.com/libffi/libffi).
https://github.com/llvm/llvm-project/pull/66698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/2
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett commented:
If these section refs weren't already here I'd suggest not adding numbers at
all given that they will change later, and Firefox and Chrome seem hopeless at
searching for them in the HTML document.
But we have them already so I won't suggest that :) T
@@ -32,7 +32,7 @@ extern "C" {
#define __isb(i) __builtin_arm_isb(i)
#endif
-/* 8.4 Hints */
+/* 7.4 Hints */
DavidSpickett wrote:
If we're matching the section titles then they should have the extra `.` on the
end of the number, so here `7.4. Hints`. Just t
@@ -20,8 +20,8 @@
extern "C" {
DavidSpickett wrote:
You may want to note that section numbers were correct at time of writing, if
they've changed then look for the section title.
https://github.com/llvm/llvm-project/pull/2
_
https://github.com/martinboehme created
https://github.com/llvm/llvm-project/pull/66746
Previously, post-visit state changes were indistinguishable from ordinary
iterations, which could give a confusing picture of how many iterations a block
needs to converge.
Now, post-visit state changes are
llvmbot wrote:
@llvm/pr-subscribers-clang-analysis
Changes
Previously, post-visit state changes were indistinguishable from ordinary
iterations, which could give a confusing picture of how many iterations a block
needs to converge.
Now, post-visit state changes are marked with "post-visit"
https://github.com/martinboehme edited
https://github.com/llvm/llvm-project/pull/66746
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jh7370 edited https://github.com/llvm/llvm-project/pull/65812
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1696,6 +1696,40 @@ TEST(Support, ReplacePathPrefix) {
EXPECT_EQ(Path, "C:\\old/foo\\bar");
}
+TEST(Support, FindProgramName) {
+ StringRef WindowsProgName =
+ path::program_name("C:\\Test\\foo.exe", path::Style::windows);
+ EXPECT_EQ(WindowsProgName, "foo");
+
+
@@ -5,11 +5,14 @@
# RUN: mkdir %t
# RUN: ln -s llvm-ranlib %t/llvm-ranlib-9
# RUN: ln -s llvm-ranlib %t/ranlib.exe
+# RUN: ln -s llvm-ranlib %t/x86_64-unknown-freebsd13.2-llvm-ranlib
jh7370 wrote:
Let's put the new test files and deletion of this old test in a
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/66749
We can't load anything from them, but we can still take their address or return
them as lvalues.
>From 9952d03bf25766ada5a41c13c3be45f9bd1e8bf2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
We can't load anything from them, but we can still take their address or return
them as lvalues.
---
Full diff: https://github.com/llvm/llvm-project/pull/66749.diff
5 Files Affected:
- (modified) clang/lib/AST/Interp/ByteCodeExprGen.cpp
https://github.com/legrosbuffle created
https://github.com/llvm/llvm-project/pull/66750
None
>From 3679b5c95a8de88a5cabc9ecdacdbbfef9c68ea8 Mon Sep 17 00:00:00 2001
From: Clement Courbet
Date: Tue, 19 Sep 2023 10:19:49 +0200
Subject: [PATCH] [NFC] Preparatory work for D153131
---
clang/lib/A
tom-anders added a comment.
(friendly ping)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141800/new/
https://reviews.llvm.org/D141800
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
nikic wrote:
I think this is the right canonicalization at the IR level, as select allows
better analysis than zext(icmp). However, this seems to be universally worse
for the backend: https://llvm.godbolt.org/z/Yh7brfc8b So I think we would need
an SDAG undo transform. Interestingly, this alre
Author: Clement Courbet
Date: 2023-09-19T11:02:00+02:00
New Revision: c8090512157ab0aad1e43e398598e689b7e55a52
URL:
https://github.com/llvm/llvm-project/commit/c8090512157ab0aad1e43e398598e689b7e55a52
DIFF:
https://github.com/llvm/llvm-project/commit/c8090512157ab0aad1e43e398598e689b7e55a52.dif
https://github.com/legrosbuffle closed
https://github.com/llvm/llvm-project/pull/66750
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Daniel Krupp
Date: 2023-09-19T11:04:50+02:00
New Revision: 97495d3159799677c2dea8516f2246854c19d007
URL:
https://github.com/llvm/llvm-project/commit/97495d3159799677c2dea8516f2246854c19d007
DIFF:
https://github.com/llvm/llvm-project/commit/97495d3159799677c2dea8516f2246854c19d007.diff
https://github.com/dkrupp closed https://github.com/llvm/llvm-project/pull/66086
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
courbet updated this revision to Diff 557015.
courbet marked an inline comment as done.
courbet added a comment.
Rebase on NFC changes
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153131/new/
https://reviews.llvm.org/D153131
Files:
clang/includ
https://github.com/sdkrystian updated
https://github.com/llvm/llvm-project/pull/66636
>From d11d546f3190936ba45c57b4825073026d817878 Mon Sep 17 00:00:00 2001
From: Krystian Stasiowski
Date: Fri, 25 Aug 2023 14:07:32 -0400
Subject: [PATCH 1/2] [clang] remove ClassScopeFunctionSpecializationDecl
https://github.com/sdkrystian updated
https://github.com/llvm/llvm-project/pull/66636
>From d11d546f3190936ba45c57b4825073026d817878 Mon Sep 17 00:00:00 2001
From: Krystian Stasiowski
Date: Fri, 25 Aug 2023 14:07:32 -0400
Subject: [PATCH 1/2] [clang] remove ClassScopeFunctionSpecializationDecl
https://github.com/sdkrystian resolved
https://github.com/llvm/llvm-project/pull/66636
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sdkrystian resolved
https://github.com/llvm/llvm-project/pull/66636
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sdkrystian resolved
https://github.com/llvm/llvm-project/pull/66636
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sdkrystian resolved
https://github.com/llvm/llvm-project/pull/66636
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sdkrystian updated
https://github.com/llvm/llvm-project/pull/66636
>From d11d546f3190936ba45c57b4825073026d817878 Mon Sep 17 00:00:00 2001
From: Krystian Stasiowski
Date: Fri, 25 Aug 2023 14:07:32 -0400
Subject: [PATCH 1/2] [clang] remove ClassScopeFunctionSpecializationDecl
https://github.com/DavidSpickett requested changes to this pull request.
This looks really nice. Two overall things:
Now that ARM and AArch64 print more than just names, they should have column
headers like RISC-V does. Name / Description
The clang test case `clang/test/Driver/print-supported-
@@ -600,11 +601,17 @@ StringRef ARM::getARMCPUForArch(const llvm::Triple
&Triple, StringRef MArch) {
llvm_unreachable("invalid arch name");
}
-void ARM::PrintSupportedExtensions() {
+void ARM::PrintSupportedExtensions(std::map llvmDescMap)
{
outs() << "All available -ma
@@ -600,11 +601,17 @@ StringRef ARM::getARMCPUForArch(const llvm::Triple
&Triple, StringRef MArch) {
llvm_unreachable("invalid arch name");
}
-void ARM::PrintSupportedExtensions() {
+void ARM::PrintSupportedExtensions(std::map llvmDescMap)
{
outs() << "All available -ma
@@ -11,11 +11,13 @@
//
//===--===//
-#include "llvm/TargetParser/AArch64TargetParser.h"
+#include "llvm/Support/Error.h"
DavidSpickett wrote:
Why do we need to include Support/Error now?
Th
@@ -11,11 +11,13 @@
//
//===--===//
-#include "llvm/TargetParser/AArch64TargetParser.h"
+#include "llvm/Support/Error.h"
#include "llvm/Support/raw_ostream.h"
+#include "llvm/TargetParser/AArch64TargetParser
https://github.com/DavidSpickett edited
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
@@ -1016,7 +1017,8 @@ TEST(TargetParserTest, ARMPrintSupportedExtensions) {
outs().flush();
testing::internal::CaptureStdout();
- ARM::PrintSupportedExtensions();
+ std::map EmptyMap;
DavidSpickett wrote:
All tests for this should include at least 1 des
@@ -600,11 +601,17 @@ StringRef ARM::getARMCPUForArch(const llvm::Triple
&Triple, StringRef MArch) {
llvm_unreachable("invalid arch name");
}
-void ARM::PrintSupportedExtensions() {
+void ARM::PrintSupportedExtensions(std::map llvmDescMap)
{
outs() << "All available -ma
@@ -198,13 +199,21 @@ static int PrintSupportedExtensions(std::string
TargetStr) {
std::unique_ptr TheTargetMachine(
TheTarget->createTargetMachine(TargetStr, "", "", Options,
std::nullopt));
const llvm::Triple &MachineTriple = TheTargetMachine->getTargetTriple();
+
@@ -198,13 +199,21 @@ static int PrintSupportedExtensions(std::string
TargetStr) {
std::unique_ptr TheTargetMachine(
TheTarget->createTargetMachine(TargetStr, "", "", Options,
std::nullopt));
const llvm::Triple &MachineTriple = TheTargetMachine->getTargetTriple();
+
@@ -210,24 +210,33 @@ static void verifyTables() {
#endif
}
-void llvm::riscvExtensionsHelp() {
+void llvm::riscvExtensionsHelp(std::map llvmDescMap) {
+
outs() << "All available -march extensions for RISC-V\n\n";
- outs() << '\t' << left_justify("Name", 20) << "Version\n"
https://github.com/sam-mccall resolved
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
@@ -87,6 +87,9 @@ class alignas(const Formula *) Formula {
ArrayRef Operands,
unsigned Value = 0);
+ // Parse Formulas using Arena rather than caling this function directly.
+ static Formula *parse(llvm::BumpPtrAllocator &All
https://github.com/sam-mccall resolved
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
https://github.com/sam-mccall resolved
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
https://github.com/sam-mccall resolved
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
@@ -95,4 +98,94 @@ BoolValue &Arena::makeBoolValue(const Formula &F) {
return *It->second;
}
+namespace {
+const Formula *parse(Arena &A, llvm::StringRef &In) {
+ auto EatWhitespace = [&] { In = In.ltrim(' '); };
sam-mccall wrote:
Renamed, hard to know if
https://github.com/sam-mccall resolved
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
donat.nagy accepted this revision.
donat.nagy added a comment.
This revision is now accepted and ready to land.
LGTM. I'm not very familiar with this area, but the change seems to be a very
clean improvement.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llv
@@ -258,114 +276,71 @@ TEST(SolverTest, IffWithUnits) {
}
TEST(SolverTest, IffWithUnitsConflict) {
- ConstraintContext Ctx;
- auto X = Ctx.atom();
- auto Y = Ctx.atom();
- auto XEqY = Ctx.iff(X, Y);
- auto NotY = Ctx.neg(Y);
-
- // (X <=> Y) ^ X !Y
- EXPECT_THAT(solve(
https://github.com/jp4a50 created
https://github.com/llvm/llvm-project/pull/66755
By default, OuterScope aligns lambdas to the beginning of the current line.
This makes sense for most types of statements within code blocks but leads to
unappealing and misleading indentation for lambdas within
@@ -95,4 +98,94 @@ BoolValue &Arena::makeBoolValue(const Formula &F) {
return *It->second;
}
+namespace {
+const Formula *parse(Arena &A, llvm::StringRef &In) {
+ auto EatWhitespace = [&] { In = In.ltrim(' '); };
+ EatWhitespace();
+
+ if (In.consume_front("!")) {
+if
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Changes
By default, OuterScope aligns lambdas to the beginning of the current line.
This makes sense for most types of statements within code blocks but leads to
unappealing and misleading indentation for lambdas within constructor
initial
@@ -95,4 +98,94 @@ BoolValue &Arena::makeBoolValue(const Formula &F) {
return *It->second;
}
+namespace {
+const Formula *parse(Arena &A, llvm::StringRef &In) {
+ auto EatWhitespace = [&] { In = In.ltrim(' '); };
+ EatWhitespace();
+
+ if (In.consume_front("!")) {
+if
https://github.com/sam-mccall updated
https://github.com/llvm/llvm-project/pull/66424
>From 35b33561af918fc2e13af9de18af6b181535ce49 Mon Sep 17 00:00:00 2001
From: Sam McCall
Date: Thu, 14 Sep 2023 21:36:50 +0200
Subject: [PATCH 1/2] [dataflow] Parse formulas from text
My immediate use for thi
https://github.com/sam-mccall resolved
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
https://github.com/sam-mccall resolved
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
https://github.com/sam-mccall resolved
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
https://github.com/sam-mccall updated
https://github.com/llvm/llvm-project/pull/66424
>From 35b33561af918fc2e13af9de18af6b181535ce49 Mon Sep 17 00:00:00 2001
From: Sam McCall
Date: Thu, 14 Sep 2023 21:36:50 +0200
Subject: [PATCH 1/3] [dataflow] Parse formulas from text
My immediate use for thi
Author: Timm Bäder
Date: 2023-09-19T11:56:44+02:00
New Revision: ad4a513026d8f15f40ac5a6dede6ad3aabfb
URL:
https://github.com/llvm/llvm-project/commit/ad4a513026d8f15f40ac5a6dede6ad3aabfb
DIFF:
https://github.com/llvm/llvm-project/commit/ad4a513026d8f15f40ac5a6dede6ad3aabfb.diff
LO
This revision was automatically updated to reflect the committed changes.
Closed by commit rGad4a51302777: [clang][CFG] Cleanup functions (authored by
tbaeder).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157385/new/
https://reviews.llvm.org/D1573
@@ -210,24 +210,33 @@ static void verifyTables() {
#endif
}
-void llvm::riscvExtensionsHelp() {
+void llvm::riscvExtensionsHelp(std::map llvmDescMap) {
+
outs() << "All available -march extensions for RISC-V\n\n";
- outs() << '\t' << left_justify("Name", 20) << "Version\n"
@@ -11,11 +11,13 @@
//
//===--===//
-#include "llvm/TargetParser/AArch64TargetParser.h"
+#include "llvm/Support/Error.h"
cbalint13 wrote:
It was for format()```, will double-check to see
https://github.com/cbalint13 edited
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
@@ -11,11 +11,13 @@
//
//===--===//
-#include "llvm/TargetParser/AArch64TargetParser.h"
+#include "llvm/Support/Error.h"
#include "llvm/Support/raw_ostream.h"
+#include "llvm/TargetParser/AArch64TargetParser
@@ -600,11 +601,17 @@ StringRef ARM::getARMCPUForArch(const llvm::Triple
&Triple, StringRef MArch) {
llvm_unreachable("invalid arch name");
}
-void ARM::PrintSupportedExtensions() {
+void ARM::PrintSupportedExtensions(std::map llvmDescMap)
{
outs() << "All available -ma
@@ -1016,7 +1017,8 @@ TEST(TargetParserTest, ARMPrintSupportedExtensions) {
outs().flush();
testing::internal::CaptureStdout();
- ARM::PrintSupportedExtensions();
+ std::map EmptyMap;
cbalint13 wrote:
Will address this, even by adding a dummy descriptio
cbalint13 wrote:
@DavidSpickett
Thank you for the review time, will address all the concerns and be back with a
re-review request.
https://github.com/llvm/llvm-project/pull/66715
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
https://github.com/cbalint13 edited
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
https://github.com/cbalint13 edited
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
https://github.com/cbalint13 edited
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
https://github.com/sam-mccall created
https://github.com/llvm/llvm-project/pull/66757
(In addition to the un-normalized form, so this is back-compatible)
>From 1923a5142f9cbd3556bba599b611ae8a1bbec776 Mon Sep 17 00:00:00 2001
From: Sam McCall
Date: Tue, 19 Sep 2023 12:16:54 +0200
Subject: [PA
@@ -6309,7 +6309,69 @@ Instruction
*InstCombinerImpl::foldICmpUsingBoolRange(ICmpInst &I) {
Y->getType()->isIntOrIntVectorTy(1) && Pred == ICmpInst::ICMP_ULE)
return BinaryOperator::CreateOr(Builder.CreateIsNull(X), Y);
+ ICmpInst::Predicate Pred1, Pred2;
const
sam-mccall wrote:
@kadircet this is internal b/294113850
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
llvmbot wrote:
@llvm/pr-subscribers-clangd
Changes
(In addition to the un-normalized form, so this is back-compatible)
---
Full diff: https://github.com/llvm/llvm-project/pull/66757.diff
1 Files Affected:
- (modified) clang-tools-extra/clangd/SystemIncludeExtractor.cpp (+7-1)
``
sam-mccall wrote:
I didn't add a test here because it seems fairly fiddly and fragile (would be a
lit test) and unlikely to actually regress in practice. But I'm just making
excuses, happy to add one if you think it's worthwhile.
https://github.com/llvm/llvm-project/pull/66757
https://github.com/cbalint13 edited
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
@@ -600,11 +601,17 @@ StringRef ARM::getARMCPUForArch(const llvm::Triple
&Triple, StringRef MArch) {
llvm_unreachable("invalid arch name");
}
-void ARM::PrintSupportedExtensions() {
+void ARM::PrintSupportedExtensions(std::map llvmDescMap)
{
outs() << "All available -ma
@@ -1016,7 +1017,8 @@ TEST(TargetParserTest, ARMPrintSupportedExtensions) {
outs().flush();
testing::internal::CaptureStdout();
- ARM::PrintSupportedExtensions();
+ std::map EmptyMap;
DavidSpickett wrote:
Yeah that's fine. All it should check is that th
https://github.com/cbalint13 edited
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
https://github.com/sam-mccall updated
https://github.com/llvm/llvm-project/pull/66757
>From 1923a5142f9cbd3556bba599b611ae8a1bbec776 Mon Sep 17 00:00:00 2001
From: Sam McCall
Date: Tue, 19 Sep 2023 12:16:54 +0200
Subject: [PATCH 1/2] [clangd] Allow --query-driver to match a dot-normalized
form
@@ -210,24 +210,33 @@ static void verifyTables() {
#endif
}
-void llvm::riscvExtensionsHelp() {
+void llvm::riscvExtensionsHelp(std::map llvmDescMap) {
+
outs() << "All available -march extensions for RISC-V\n\n";
- outs() << '\t' << left_justify("Name", 20) << "Version\n"
@@ -2325,6 +2325,7 @@ void
CodeGenModule::SetLLVMFunctionAttributesForDefinition(const Decl *D,
B.addAttribute(llvm::Attribute::Naked);
// OptimizeNone wins over OptimizeForSize and MinSize.
+F->removeFnAttr(llvm::Attribute::OptimizeForDebugging);
--
Timm =?utf-8?q?B=C3=A4der?=
Message-ID:
In-Reply-To:
tbaederr wrote:
Review continued on Phabricator.
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RKSimon approved this pull request.
LGTM - please update the description to mention that you're updating the movnti
i32/i64 scalar integer nt ops as well as the vector ops
https://github.com/llvm/llvm-project/pull/66310
___
cfe-comm
budimirarandjelovicsyrmia added a comment.
Ping.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141192/new/
https://reviews.llvm.org/D141192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinf
https://github.com/martinboehme resolved
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
@@ -0,0 +1,312 @@
+//===- StdVariantChecker.cpp -*- C++
-*-==//
+//
+// 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: Ap
DonatNagyE wrote:
The impact is limited, because `core.BitwiseShift` is a `PreStmt` callback, so
when I merged it, it turned the logic tweaked/affected in this commit into
"almost dead" code.
The most significant change is that some "pedantic-only" issues (i.e. code like
`-1 >> 3` that's unde
https://github.com/KanRobert approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/66310
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/martinboehme created
https://github.com/llvm/llvm-project/pull/66764
I've received a report of a null pointer dereference happening on the
`LocDst->getType()` dereference. I wasn't unfortunately able to find a repro,
but I'd argue the new version is better for the reduced inde
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
I've received a report of a null pointer dereference happening on the
`LocDst->getType()` dereference. I wasn't unfortunately able to find a repro,
but I'd argue the new version is better for the reduced indentation alone.
---
Full diff: h
kadircet wrote:
thanks, no need for tests, the logic is rather trivial (might be worth
mentioning in change notes though, so that we can build them up more naturally
for the future and encourage others to do so as well)
https://github.com/llvm/llvm-project/pull/66757
__
1 - 100 of 401 matches
Mail list logo