https://github.com/joker-eph approved this pull request.
https://github.com/llvm/llvm-project/pull/81240
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/joker-eph approved this pull request.
https://github.com/llvm/llvm-project/pull/81237
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
zero9178 wrote:
ping
https://github.com/llvm/llvm-project/pull/80421
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/davemgreen approved this pull request.
Looks good to get onto the branch to me
https://github.com/llvm/llvm-project/pull/81454
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
@@ -560,35 +586,50 @@ Error
InstrProfSymtab::initVTableNamesFromCompressedStrings(
std::bind(&InstrProfSymtab::addVTableName, this, std::placeholders::_1));
}
-Error InstrProfSymtab::addFuncWithName(Function &F, StringRef PGOFuncName) {
- if (Error E = addFuncName(PGOF
@@ -560,35 +586,50 @@ Error
InstrProfSymtab::initVTableNamesFromCompressedStrings(
std::bind(&InstrProfSymtab::addVTableName, this, std::placeholders::_1));
}
-Error InstrProfSymtab::addFuncWithName(Function &F, StringRef PGOFuncName) {
- if (Error E = addFuncName(PGOF
MaskRay wrote:
The current layout alleviates relocation overflow pressure for `.data` and
`.bss` but not `.lrodata`. I don't think this means that we _unsupport_
`-no-pie` use cases. We just don't make `.rodata` nicer to `-no-pie` use cases.
We could, but it would be at the cost of layout puri
llvmbot wrote:
@llvm/pr-subscribers-backend-risc-v
Author: None (llvmbot)
Changes
Backport 06c89bd
Requested by: @lukel97
---
Patch is 21.42 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/81568.diff
2 Files Affected:
- (modified) llvm/lib/
https://github.com/llvmbot updated
https://github.com/llvm/llvm-project/pull/81568
>From 42d30df56947c5464788420084baa254437a4cb9 Mon Sep 17 00:00:00 2001
From: Luke Lau
Date: Fri, 9 Feb 2024 06:51:11 +0800
Subject: [PATCH] [RISCV] Check type is legal before combining mgather to vlse
intrinsic
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/81568
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
llvmbot wrote:
@preames What do you think about merging this PR to the release branch?
https://github.com/llvm/llvm-project/pull/81568
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
https://github.com/llvmbot created
https://github.com/llvm/llvm-project/pull/81568
Backport 06c89bd
Requested by: @lukel97
>From 670b86bafd8b188a6392ac727f9aa7cac74bf0d7 Mon Sep 17 00:00:00 2001
From: Luke Lau
Date: Fri, 9 Feb 2024 06:51:11 +0800
Subject: [PATCH] [RISCV] Check type is legal b
jyknight wrote:
I don't follow.
no-pie is _uncommon_ but not _unsupported_. Making users add a linker script
when they wouldn't otherwise have one, in order to have a working no-pie build
seems unjustified.
https://github.com/llvm/llvm-project/pull/81224
__
MaskRay wrote:
> ISTM that making the layout conditional in order to not regress the
> most-common case (PIE) would be worth a minor amount of code in lld, even
> though the benefit is also relatively small.
>
> Do you think INSERT scripts are going to be a real-world issue there? It
> seems
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: None (llvmbot)
Changes
Backport 4af24d4ab765
Requested by: @owenca
---
Full diff: https://github.com/llvm/llvm-project/pull/81566.diff
2 Files Affected:
- (modified) clang/lib/Format/UnwrappedLineParser.cpp (+1-1)
- (modified)
llvmbot wrote:
@HazardyKnusperkeks What do you think about merging this PR to the release
branch?
https://github.com/llvm/llvm-project/pull/81566
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/81566
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/llvmbot created
https://github.com/llvm/llvm-project/pull/81566
Backport 4af24d4ab765
Requested by: @owenca
>From 0acacbbad0160dc697375ab657b4231012e4e4d0 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Mon, 12 Feb 2024 19:20:26 -0800
Subject: [PATCH] [clang-format] Don't rem
jyknight wrote:
ISTM that making the layout conditional in order to not regress the most-common
case (PIE) would be worth a minor amount of code in lld, even though the
benefit is also relatively small.
Do you think INSERT scripts are going to be a real-world issue there? It seems
to me like
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/81224
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/81224
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/81224
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -0,0 +1,99 @@
+#include "bolt/Rewrite/JITRewriteInstance.h"
+#include "bolt/Core/BinaryContext.h"
+#include "llvm/BinaryFormat/ELF.h"
+#include "llvm/Support/TargetSelect.h"
+#include "gtest/gtest.h"
+
+using namespace llvm;
+using namespace llvm::object;
+using namespace llvm:
@@ -0,0 +1,99 @@
+#include "bolt/Rewrite/JITRewriteInstance.h"
+#include "bolt/Core/BinaryContext.h"
+#include "llvm/BinaryFormat/ELF.h"
+#include "llvm/Support/TargetSelect.h"
+#include "gtest/gtest.h"
+
+using namespace llvm;
+using namespace llvm::object;
+using namespace llvm:
@@ -267,7 +264,6 @@ static void createRetBitCast(CallBase &CB, Type *RetTy,
CastInst **RetBitCast) {
/// Is replaced by the following:
///
/// cond_bb:
-/// %cond = icmp eq i32 ()* %ptr, @func
minglotus-6 wrote:
done.
https://github.com/llvm/llvm-proje
https://github.com/minglotus-6 updated
https://github.com/llvm/llvm-project/pull/81181
>From 7ebae253ab1808bca328453f68af2b595d07176e Mon Sep 17 00:00:00 2001
From: mingmingl
Date: Thu, 8 Feb 2024 11:32:50 -0800
Subject: [PATCH 1/2] [NFC][CallPromotionUtils]Extract a helper function
versionCal
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/81523
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81524
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81524
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81523
>From 17ab8e592a01d4402e88de21c8f66742134b64f3 Mon Sep 17 00:00:00 2001
From: Amir Ayupov
Date: Mon, 12 Feb 2024 12:42:25 -0800
Subject: [PATCH 1/2] Fix build
Created using spr 1.3.4
---
bolt/lib/Rewrite/MachORe
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81523
>From 17ab8e592a01d4402e88de21c8f66742134b64f3 Mon Sep 17 00:00:00 2001
From: Amir Ayupov
Date: Mon, 12 Feb 2024 12:42:25 -0800
Subject: [PATCH 1/2] Fix build
Created using spr 1.3.4
---
bolt/lib/Rewrite/MachORe
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/81522
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/81468
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/rafaelauler approved this pull request.
Thanks, here's what I think is a suitable commit message for this one:
Make core BOLT functionality more friendly to being used as a
library instead of in our standalone driver llvm-bolt. To
accomplish this, we augment BinaryContext with
https://github.com/rafaelauler approved this pull request.
Thanks, as commit message:
As part of the effort to refactor old error handling code that
would directly call exit(1), in this patch continue the migration
on libCore, libRewrite and libPasses to use the new BOLTError
class whenever a fa
https://github.com/rafaelauler edited
https://github.com/llvm/llvm-project/pull/81522
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/rafaelauler approved this pull request.
Thanks, as commit message we can put something like:
As part of the effort to refactor old error handling code that
would directly call exit(1), in this patch we add a new class
BOLTError and auxiliary functions createFatalBOLTError() an
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81524
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81524
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81523
>From 17ab8e592a01d4402e88de21c8f66742134b64f3 Mon Sep 17 00:00:00 2001
From: Amir Ayupov
Date: Mon, 12 Feb 2024 12:42:25 -0800
Subject: [PATCH 1/2] Fix build
Created using spr 1.3.4
---
bolt/lib/Rewrite/MachORe
minglotus-6 wrote:
> Can you pull out the part that does the refactoring into getCanonicalName and
> adds comments etc into a separate nfc patch, so that this one is just about
> adding vtables?
Sure! I'm thinking about doing the nfc patch into LLVM main branch given
`getCanonicalName` simpli
https://github.com/aaupov ready_for_review
https://github.com/llvm/llvm-project/pull/81524
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov ready_for_review
https://github.com/llvm/llvm-project/pull/81523
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/81523
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/81522
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/teresajohnson edited
https://github.com/llvm/llvm-project/pull/81181
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/teresajohnson approved this pull request.
lgtm with a suggested change to the comments
https://github.com/llvm/llvm-project/pull/81181
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi
@@ -267,7 +264,6 @@ static void createRetBitCast(CallBase &CB, Type *RetTy,
CastInst **RetBitCast) {
/// Is replaced by the following:
///
/// cond_bb:
-/// %cond = icmp eq i32 ()* %ptr, @func
teresajohnson wrote:
I'm not sure these conditions should be
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81524
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81524
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/81523
>From 17ab8e592a01d4402e88de21c8f66742134b64f3 Mon Sep 17 00:00:00 2001
From: Amir Ayupov
Date: Mon, 12 Feb 2024 12:42:25 -0800
Subject: [PATCH] Fix build
Created using spr 1.3.4
---
bolt/lib/Rewrite/MachORewrit
https://github.com/aaupov closed https://github.com/llvm/llvm-project/pull/81529
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov closed https://github.com/llvm/llvm-project/pull/81530
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov closed https://github.com/llvm/llvm-project/pull/81531
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
teresajohnson wrote:
Can you pull out the part that does the refactoring into getCanonicalName and
adds comments etc into a separate nfc patch, so that this one is just about
adding vtables?
https://github.com/llvm/llvm-project/pull/81051
___
llvm-br
https://github.com/aaupov created
https://github.com/llvm/llvm-project/pull/81531
Co-authored-by: Rafael Auler
Test Plan: NFC
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
https://github.com/aaupov created
https://github.com/llvm/llvm-project/pull/81530
Test Plan:
NFC
Co-authored-by: Rafael Auler
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
https://github.com/aaupov created
https://github.com/llvm/llvm-project/pull/81529
Co-authored-by: Rafael Auler
Test Plan: NFC
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
https://github.com/minglotus-6 edited
https://github.com/llvm/llvm-project/pull/81378
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/minglotus-6 edited
https://github.com/llvm/llvm-project/pull/81378
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov converted_to_draft
https://github.com/llvm/llvm-project/pull/81524
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov converted_to_draft
https://github.com/llvm/llvm-project/pull/81523
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov created
https://github.com/llvm/llvm-project/pull/81524
Co-authored-by: Rafael Auler
Test Plan: NFC
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
https://github.com/aaupov created
https://github.com/llvm/llvm-project/pull/81523
Test Plan:
NFC
Co-authored-by: Rafael Auler
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
https://github.com/aaupov created
https://github.com/llvm/llvm-project/pull/81522
Co-authored-by: Rafael Auler
Test Plan: NFC
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
agozillon wrote:
I believe this is the top of the PR stack that should pass... however, it's my
first time using SPR so we'll see how it goes. If there's anything odd with the
PR (outside of the usual code review) please don't hesitate to mention it so I
can address it!
I believe I fragmented
https://github.com/jpienaar approved this pull request.
https://github.com/llvm/llvm-project/pull/81373
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
MaskRay wrote:
Ping:)
https://github.com/llvm/llvm-project/pull/77221
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
llvmbot wrote:
@llvm/pr-subscribers-flang-fir-hlfir
Author: None (agozillon)
Changes
This patch is one in a series of four patches that seeks to refactor
slightly and extend the current record type map support that was
put in place for Fortran's descriptor types to handle explicit
member m
llvmbot wrote:
@llvm/pr-subscribers-mlir-openmp
@llvm/pr-subscribers-mlir
Author: None (agozillon)
Changes
This patch seeks to refactor slightly and extend the current record type map
support that was put in place for Fortran's descriptor types to handle explicit
member mapping for record
llvmbot wrote:
@llvm/pr-subscribers-flang-openmp
@llvm/pr-subscribers-mlir
Author: None (agozillon)
Changes
This PR adds two new fields to omp.map_info, one BoolAttr and one I64ArrayAttr.
The BoolAttr is named partial_map, and is a flag that indicates if the record
type captured by
the m
https://github.com/agozillon created
https://github.com/llvm/llvm-project/pull/81511
This patch is one in a series of four patches that seeks to refactor
slightly and extend the current record type map support that was
put in place for Fortran's descriptor types to handle explicit
member mapping
https://github.com/agozillon created
https://github.com/llvm/llvm-project/pull/81510
This patch seeks to refactor slightly and extend the current record type map
support that was put in place for Fortran's descriptor types to handle explicit
member mapping for record types at a single level of d
https://github.com/agozillon created
https://github.com/llvm/llvm-project/pull/81509
This PR adds two new fields to omp.map_info, one BoolAttr and one I64ArrayAttr.
The BoolAttr is named partial_map, and is a flag that indicates if the record
type captured by
the map_info operation is a partia
llvmbot wrote:
@llvm/pr-subscribers-llvm-transforms
Author: Mingming Liu (minglotus-6)
Changes
* Given the code sequence
```
bb:
%vtable = load ptr, ptr %d, !prof !8
%vfn = getelementptr inbounds ptr, ptr %vtable, i64 1
%1 = load ptr, ptr %vfn
%call = tail call i3
https://github.com/minglotus-6 ready_for_review
https://github.com/llvm/llvm-project/pull/81378
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/topperc edited
https://github.com/llvm/llvm-project/pull/80124
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -445,6 +450,9 @@ void RISCVPassConfig::addIRPasses() {
if (EnableLoopDataPrefetch)
addPass(createLoopDataPrefetchPass());
+if (EnableSelectOpt && getOptLevel() == CodeGenOptLevel::Aggressive)
topperc wrote:
Shoudl this be after addIRPasses li
https://github.com/minglotus-6 edited
https://github.com/llvm/llvm-project/pull/81378
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/minglotus-6 updated
https://github.com/llvm/llvm-project/pull/81378
>From ac5dc1bf77b67cbf0aa5e2c8fb6a7ce0080fb501 Mon Sep 17 00:00:00 2001
From: mingmingl
Date: Sat, 10 Feb 2024 12:03:25 -0800
Subject: [PATCH 1/3] [CallPromotionUtils]Implement conditional indirect call
prom
https://github.com/minglotus-6 updated
https://github.com/llvm/llvm-project/pull/81378
>From ac5dc1bf77b67cbf0aa5e2c8fb6a7ce0080fb501 Mon Sep 17 00:00:00 2001
From: mingmingl
Date: Sat, 10 Feb 2024 12:03:25 -0800
Subject: [PATCH 1/2] [CallPromotionUtils]Implement conditional indirect call
prom
https://github.com/minglotus-6 edited
https://github.com/llvm/llvm-project/pull/81181
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
llvmbot wrote:
@llvm/pr-subscribers-llvm-transforms
Author: Mingming Liu (minglotus-6)
Changes
The parent patch is https://github.com/llvm/llvm-project/pull/81051
---
Full diff: https://github.com/llvm/llvm-project/pull/81181.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/Util
https://github.com/minglotus-6 ready_for_review
https://github.com/llvm/llvm-project/pull/81181
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ldionne wrote:
Approved.
https://github.com/llvm/llvm-project/pull/81410
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
dobbelaj-snps wrote:
> > Hmm. 10 changes + 1 new usage of setAAMetaData, But only 4 relevant changes
> > in tests.. That part of SROA seems to lack some testing ?
>
> Yes, will add the missing coverage, just wanted to make sure this makes sense
> in general beforehand
I think it makes sense.
fhahn wrote:
> lgtm
>
> Maybe rephrase the commit message to something like:
>
> ```
> [tbaa] Use !tbaa for first accessed field if it is an exact match in offset
> and size.
> ```
Updated, thanks! It would be great if you could take another look at
https://github.com/llvm/llvm-project/pul
https://github.com/fhahn edited https://github.com/llvm/llvm-project/pull/81313
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/DavidSpickett approved this pull request.
https://github.com/llvm/llvm-project/pull/81468
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
DavidSpickett wrote:
I've confirmed locally that this fixes the test as claimed, let's merge it.
https://github.com/llvm/llvm-project/pull/81468
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
llvmbot wrote:
@llvm/pr-subscribers-lld
Author: None (llvmbot)
Changes
Backport 0bf4ff29816c0eead99ba576a2df2e3c4d214b1f
Requested by: @mstorsjo
---
Full diff: https://github.com/llvm/llvm-project/pull/81468.diff
1 Files Affected:
- (modified) lld/test/COFF/timestamp.test (+19-10)
llvmbot wrote:
@DavidSpickett What do you think about merging this PR to the release branch?
https://github.com/llvm/llvm-project/pull/81468
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
https://github.com/huntergr-arm edited
https://github.com/llvm/llvm-project/pull/81135
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/llvmbot updated
https://github.com/llvm/llvm-project/pull/81468
>From 996b6d7194c6bc6dc4b9aa87bcca3525dafcaa03 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Storsj=C3=B6?=
Date: Mon, 12 Feb 2024 13:22:45 +0200
Subject: [PATCH] [LLD] [test] Avoid printing timestamps past I
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/81468
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/matthias-springer edited
https://github.com/llvm/llvm-project/pull/81462
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/matthias-springer updated
https://github.com/llvm/llvm-project/pull/81462
>From ba1a8085ab576b56a7d1780d9a3f01cb6769df65 Mon Sep 17 00:00:00 2001
From: Matthias Springer
Date: Mon, 12 Feb 2024 11:24:52 +
Subject: [PATCH] [mlir][Transforms][NFC] Simplify `ArgConverter` sta
llvmbot wrote:
@llvm/pr-subscribers-mlir-core
Author: Matthias Springer (matthias-springer)
Changes
* When converting a block signature, `ArgConverter` creates a new block with
the new signature and moves all operation from the old block to the new block.
The new block is temporarily ins
https://github.com/matthias-springer created
https://github.com/llvm/llvm-project/pull/81462
* When converting a block signature, `ArgConverter` creates a new block with
the new signature and moves all operation from the old block to the new block.
The new block is temporarily inserted into a
https://github.com/dobbelaj-snps approved this pull request.
lgtm
Maybe rephrase the commit message to something like:
[tbaa] Use !tbaa for first accessed field if it is an exact match in
offset and size.
https://github.com/llvm/llvm-project/pull/81313
__
1 - 100 of 112 matches
Mail list logo
