@@ -234,8 +244,15 @@ void SparcAsmPrinter::LowerGETPCXAndEmitMCInsts(const
MachineInstr *MI,
// add , %o7,
OutStreamer->emitLabel(StartLabel);
- MCOperand Callee = createPCXCallOP(EndLabel, OutContext);
- EmitCall(*OutStreamer, Callee, STI);
+ if
https://github.com/s-barannikov edited
https://github.com/llvm/llvm-project/pull/77196
___
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,59 @@
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc < %s -relocation-model=pic -mtriple=sparc | FileCheck
--check-prefix=SPARC %s
+; RUN: llc < %s -relocation-model=pic -mtriple=sparcv9 | FileCheck
--check-prefix=SPARC64
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/76904
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
Author: carlobertolli
Date: 2024-01-12T15:01:30-06:00
New Revision: 09a3a18b2d9a83c1f0399f60fe7673af40edcee4
URL:
https://github.com/llvm/llvm-project/commit/09a3a18b2d9a83c1f0399f60fe7673af40edcee4
DIFF:
https://github.com/llvm/llvm-project/commit/09a3a18b2d9a83c1f0399f60fe7673af40edcee4.diff
kparzysz wrote:
> > This makes it unnecessary to embed OpenMPSectionConstruct inside of
> > OpenMPSectionConstruct anymore.
>
> What do you mean by this? Is there a typo?
Yes, fixed. I meant embedding it inside of `OpenMPConstruct`. Thanks for the
catch.
> Would we be able to remove
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/77759
___
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, looks good.
https://github.com/llvm/llvm-project/pull/76903
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
boomanaiden154 wrote:
@legrosbuffle I saw you approved the two prerequisite PRs. Do you have any
additional comments on this one?
https://github.com/llvm/llvm-project/pull/76653
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
kiranchandramohan wrote:
> This makes it unnecessary to embed OpenMPSectionConstruct inside of
> OpenMPSectionConstruct anymore.
What do you mean by this? Is there a typo?
Would we be able to remove `OpenMPSectionConstruct` from `struct
OpenMPConstruct` after this change?
https://github.com/mordante updated
https://github.com/llvm/llvm-project/pull/76268
>From b3c567f4a4369b1d22f189f272a3fa86c1f0f401 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Fri, 22 Dec 2023 21:43:57 +0100
Subject: [PATCH] [libc++][modules] Increase clang-tidy version used.
As
@@ -2248,11 +2248,15 @@ static void createBodyOfOp(
if (clauses)
ClauseProcessor(converter, *clauses).processCopyin();
}
+
+ if (genNested)
+genNestedEvaluations(converter, eval);
}
static void genBodyOfTargetDataOp(
Fortran::lower::AbstractConverter
@@ -3358,14 +3396,17 @@ genOMP(Fortran::lower::AbstractConverter ,
.t);
// Currently only private/firstprivate clause is handled, and
// all privatization is done within `omp.section` operations.
+ symTable.pushScope();
kparzysz wrote:
https://github.com/skatrak edited
https://github.com/llvm/llvm-project/pull/77760
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -2248,11 +2248,15 @@ static void createBodyOfOp(
if (clauses)
ClauseProcessor(converter, *clauses).processCopyin();
}
+
+ if (genNested)
+genNestedEvaluations(converter, eval);
}
static void genBodyOfTargetDataOp(
Fortran::lower::AbstractConverter
@@ -2178,7 +2178,7 @@
createAndSetPrivatizedLoopVar(Fortran::lower::AbstractConverter ,
template
static void createBodyOfOp(
Op , Fortran::lower::AbstractConverter , mlir::Location ,
-Fortran::lower::pft::Evaluation ,
+Fortran::lower::pft::Evaluation , bool
@@ -2178,7 +2178,7 @@
createAndSetPrivatizedLoopVar(Fortran::lower::AbstractConverter ,
template
static void createBodyOfOp(
Op , Fortran::lower::AbstractConverter , mlir::Location ,
-Fortran::lower::pft::Evaluation ,
+Fortran::lower::pft::Evaluation , bool
https://github.com/skatrak edited
https://github.com/llvm/llvm-project/pull/77760
___
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/skatrak commented:
I have just a couple of small comments, thank you again for this work!
https://github.com/llvm/llvm-project/pull/77760
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://github.com/legrosbuffle approved this pull request.
https://github.com/llvm/llvm-project/pull/77900
___
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/skatrak approved this pull request.
Thanks, LGTM!
https://github.com/llvm/llvm-project/pull/77759
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
@@ -110,6 +110,32 @@ static void gatherFuncAndVarSyms(
}
}
+static Fortran::lower::pft::Evaluation *
+getEvalPastCollapse(Fortran::lower::pft::Evaluation , int collapseValue) {
skatrak wrote:
Nit: I tend to read this function name as meaning "return the
@@ -3655,8 +3661,7 @@ void Fortran::lower::genOpenMPDeclarativeConstruct(
Fortran::lower::pft::Evaluation ,
const Fortran::parser::OpenMPDeclarativeConstruct ) {
genOMP(converter, eval, omp);
skatrak wrote:
Nit: I guess the `symTable` argument
@@ -3358,14 +3396,17 @@ genOMP(Fortran::lower::AbstractConverter ,
.t);
// Currently only private/firstprivate clause is handled, and
// all privatization is done within `omp.section` operations.
+ symTable.pushScope();
skatrak wrote:
Why
https://github.com/skatrak commented:
Thanks Krzysztof! Just some small comments, but I like the approach.
https://github.com/llvm/llvm-project/pull/77758
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://github.com/skatrak edited
https://github.com/llvm/llvm-project/pull/77758
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -156,7 +156,9 @@ export namespace std {
using std::reinterpret_pointer_cast;
using std::static_pointer_cast;
+#ifndef _LIBCPP_HAS_NO_RTTI
using std::get_deleter;
+#endif // _LIBCPP_HAS_NO_RTTI
mordante wrote:
This tested here, but should land
https://github.com/mordante updated
https://github.com/llvm/llvm-project/pull/76246
>From eebe9b2fab5c5eef4776852270bf70af4626cfcb Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Wed, 20 Dec 2023 20:43:38 +0100
Subject: [PATCH] [libc++][modules] Adds module testing.
This adds a new module
https://github.com/boomanaiden154 edited
https://github.com/llvm/llvm-project/pull/76653
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -112,9 +116,11 @@ class Counter {
PerfEvent Event;
int FileDescriptor = -1;
bool IsDummyEvent;
+ std::vector ValidationEvents;
boomanaiden154 wrote:
Done. I've split some of the refactoring into separate patches to try and keep
the diffs a little
https://github.com/boomanaiden154 updated
https://github.com/llvm/llvm-project/pull/76653
>From 0196a8780f2d3a378f0fd3b86a6e6b6590ca02de Mon Sep 17 00:00:00 2001
From: Aiden Grossman
Date: Sat, 30 Dec 2023 15:37:56 -0800
Subject: [PATCH 01/15] [llvm-exegesis] Remove unused Counter::read method
https://github.com/boomanaiden154 edited
https://github.com/llvm/llvm-project/pull/76653
___
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-tools-llvm-exegesis
Author: Aiden Grossman (boomanaiden154)
Changes
This further sets things up for validation events. Having a separate
abstraction for a configured event that is setup as a counter allows for much
easier creation of more events in
https://github.com/boomanaiden154 created
https://github.com/llvm/llvm-project/pull/77900
This further sets things up for validation events. Having a separate
abstraction for a configured event that is setup as a counter allows for much
easier creation of more events in the future within a
Author: martinboehme
Date: 2024-01-12T09:53:53+01:00
New Revision: 44fbb43480c8779e8164a21dacb4d524d69b0e07
URL:
https://github.com/llvm/llvm-project/commit/44fbb43480c8779e8164a21dacb4d524d69b0e07
DIFF:
https://github.com/llvm/llvm-project/commit/44fbb43480c8779e8164a21dacb4d524d69b0e07.diff
https://github.com/boomanaiden154 updated
https://github.com/llvm/llvm-project/pull/76653
>From 0196a8780f2d3a378f0fd3b86a6e6b6590ca02de Mon Sep 17 00:00:00 2001
From: Aiden Grossman
Date: Sat, 30 Dec 2023 15:37:56 -0800
Subject: [PATCH 01/15] [llvm-exegesis] Remove unused Counter::read method
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 e3e47a15679e2df3f1b5f161164bc0fe82833583
ea146f2462c7155fc796b58bb7f7d92527e58422 --
https://github.com/boomanaiden154 updated
https://github.com/llvm/llvm-project/pull/76653
>From 0196a8780f2d3a378f0fd3b86a6e6b6590ca02de Mon Sep 17 00:00:00 2001
From: Aiden Grossman
Date: Sat, 30 Dec 2023 15:37:56 -0800
Subject: [PATCH 01/14] [llvm-exegesis] Remove unused Counter::read method
https://github.com/boomanaiden154 edited
https://github.com/llvm/llvm-project/pull/76653
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
39 matches
Mail list logo
