Hi,
I hope I've got some good news:
On Thu, Jan 14, 2016 at 05:18:56PM -0800, Ian Lance Taylor wrote:
> Jakub Jelinek writes:
>
> > On Wed, Jan 13, 2016 at 06:39:33PM +0100, Martin Jambor wrote:
> >> the following patch adds a BRIG (binary representation of
Hi,
On Fri, Feb 26, 2016 at 06:51:34PM +0100, Jakub Jelinek wrote:
> On Fri, Feb 26, 2016 at 06:18:13PM +0100, Martin Jambor wrote:
> > > I'm a proponent of enabling as many useful warnings by default, or if not
> > > by default, then with -Wall. -Whsa is enable
; On Thu, 10 Dec 2015 18:51:48 +0100, Martin Jambor wrote:
> > > On Mon, Dec 07, 2015 at 12:46:45PM +0100, Jakub Jelinek wrote:
> > > > On Mon, Dec 07, 2015 at 12:17:58PM +0100, Martin Jambor wrote:
> > > > > [...] There are no failing
> > > >
c-5 branch together with the PR 69666 fix a few days
afterwards. OK?
Thanks,
Martin
2016-02-26 Martin Jambor
PR middle-end/69920
* tree-sra.c (sra_modify_assign): Do not remove loads of
uninitialized aggregates to SSA_NAMEs.
testsuite/
* gcc.dg/torture/pr69932.c
incorporate all feedback, I would of course like to commit
this to trunk. Needless to say, the patch has been tested and works.
Thanks,
Martin
2016-02-22 Martin Jambor
Martin Liska
* testsuite/lib/libgomp.exp (check_hsa_offloading_available): New.
* testsuite
e and generic way of doing that. However, all
testcases for non-shared memory devices were written with disallowed
fallback in mind and so this soulution also gives the desired result.
Tested both with and without HSA (enabled or present). OK for trunk?
Thanks,
Martin
2016-02-12 Martin J
ism
way of executing stuff. Hopefully we'll be able to revert many of the
hunks below when we get that working.
The patch has been tested both with and without HSA enabled. OK for
trunk?
Thanks,
Martin
2016-02-12 Martin Jambor
* testsuite/libgomp.c++/examples-4/target_data-5.C:
which we could do it
earlier, I can detect it and examine the issue. The explicit -Wno-hsa
options are added by the patch below, which has been tested both with
and without HSA. OK for trunk?
Thanks,
Martin
2016-02-12 Martin Jambor
testsuite/
* c-c++-common/gomp/clauses-1.c: Add -Wno
device with shared memory (this includes the
host). Keith included a C++ test to illustrate the use.
I have tested this thoroughly, both with and without HSA (enabled or
present). OK for trunk?
Thanks,
Martin
2016-02-10 Keith McDaniel
Martin Jambor
* testsuite/lib
gridification happens.
Tested, both with and without HSA enabled. OK for trunk?
Thanks,
Martin
2016-02-10 Martin Jambor
* target-supports.exp (check_effective_target_offload_hsa): New.
* c-c++-common/gomp/gridify-1.c: New test.
* gfortran.dg/gomp/gridify-1.f90: Likewise
).
The patch below fulfills that requirement. I have bootstrapped and
tested it and will commit it shortly.
Thanks,
Martin
2016-02-18 Martin Jambor
PR hsa/69568
* hsa.h (hsa_type_packed_p): Declare.
* hsa.c (hsa_type_packed_p): New function.
* hsa-gen.c
build libgomp plugin for it, but we should not ICE
and the new code below is the correct way of doing things anyway. So
I will commit it shortly, it has been included in a
bootstrap/testsuite run.
Thanks,
Martin
2016-02-18 Martin Jambor
pr hsa/69674
* hsa-gen.c
hanks,
Martin
2016-02-17 Martin Jambor
* hsa.h (is_a_helper): New overload for hsa_op_immed for
hsa_op_with_type operands.
(hsa_unsigned_type_for_type): Declare.
* hsa.c (hsa_unsigned_type_for_type): New function.
* hsa-gen.c (gen_hsa_binary_operation)
n the compiler, because otherwise there is testcase in the
libgomp testsuite that complains about the finalizer errors in the
output of the execute test. Bootstrapped and tested, I'll commit it
to trunk shortly.
Thanks,
Martin
2016-02-17 Martin Jambor
*
e the body even though the explicit loop is
missing, but that will also mean that the HSA grid size has to shrink
appropriately.
Bootstrapped and tested on x86_64-linux, with and without HSA
enabled. OK for trunk?
Thanks,
Martin
2016-02-17 Martin Jambor
* omp-
On Wed, Feb 24, 2016 at 02:59:11PM +0100, Martin Liska wrote:
> gcc/ChangeLog:
>
> 2016-02-24 Martin Liska
>
> * hsa-gen.c (generate_hsa): Replace ENABLE_CHECKING macro
> with flag_checking.
> * hsa-regalloc.c (linear_scan_regalloc): Likewise.
OK, thanks,
Martin
On Tue, Feb 23, 2016 at 06:45:08AM -0800, H.J. Lu wrote:
> On Fri, Feb 19, 2016 at 8:21 AM, Martin Jambor wrote:
> > Hi,
> >
> > in PR 69666, SRA attempts to turn a load from an aggregate that is
> > uninitialized into a load from a default definition SSA name (whi
.
Bootstrapped and tested on x86_64, OK for trunk and later for the 5
branch?
Thanks,
Martin
2016-02-19 Martin Jambor
PR tree-optimization/69666
* tree-sra.c (sra_modify_assign): Do not attempt to create
defaut_def replacements for unscalarizable regions.
testsuite
On Wed, Feb 17, 2016 at 03:00:00PM +0100, Richard Biener wrote:
> On Wed, Feb 17, 2016 at 2:51 PM, Marek Polacek wrote:
> > On Wed, Feb 17, 2016 at 02:45:36PM +0100, Richard Biener wrote:
> >> OTOH I have in my local trees a more convenient form (attached).
> >>
> >> (gdb) call debug_dot_cfg (cfun
are small because as I was restructuring
the code, I was moving it in this direction for some time already.
Committed to the branch (a few days ago actually, sorry for that).
Thanks,
Martin
2016-01-26 Martin Jambor
gcc/
* Makefile.in (BUILTINS_DEF): Add hsa-builtins.def
@@
News
+ Heterogeneous Systems Architecture support
+ [2016-01-27]
+ http://www.hsafoundation.com/";> Heterogeneous Systems
+ Architecture 1.0 https://gcc.gnu.org/gcc-6/changes.html#hsa";>
+ support was added to GCC, contributed by Martin Jambor, Martin Liška
+ and Michael M
it trunk even at
this stage, even though of course nothing in HSA is a regression.
Thanks,
Martin
2016-01-29 Martin Jambor
* hsa-gen.c (get_memory_order_name): Mask with MEMMODEL_BASE_MASK.
Use short lowercase names.
(get_memory_order): Mask with MEMMODEL_BASE_MASK
Hi,
On Fri, Dec 18, 2015 at 08:41:41AM -0500, David Edelsohn wrote:
> I am pleased to announce that the GCC Steering Committee has
> appointed Martin Jambor as HSA maintainer.
>
> Please join me in congratulating Martin on his new role.
> Martin, please update your
ous Systems Architecture support in GCC
+ [2016-01-27]
+ http://www.hsafoundation.com/";> Heterogeneous Systems
+ Architecture 1.0 https://gcc.gnu.org/gcc-6/changes.html#hsa";>
+ support was added to GCC. Contributed by Martin Jambor, Martin Liška
+ and Michael Matz f
On Fri, Jan 22, 2016 at 02:18:38PM +0100, Bernd Schmidt wrote:
> On 01/22/2016 09:36 AM, Jakub Jelinek wrote:
> >
> >I think it is a bad idea to go against what the user wrote. Warning that
> >some code might not be efficient? Perhaps (if properly guarded with some
> >warning option one can turn
unscalarized data). Bootstrapped and tested on
x86_64-linux. OK for trunk? And the gcc-5 branch?
Thanks,
Martin
2016-01-26 Martin Jambor
PR tree-optimization/69355
* tree-sra.c (analyze_access_subtree): Correct hole detection when
total_scalarization fails
Hi,
On Wed, Jan 13, 2016 at 06:39:25PM +0100, Martin Jambor wrote:
> Hi,
>
> this is hopefully the last big re-post of the HSA patches...
I have committed the combined patch as revision 232549 after
bootstrapping and testing all languages on x86_64-linux and i686-linux
and verifying
Hi,
On Sat, Jan 16, 2016 at 09:58:51AM +0100, Jakub Jelinek wrote:
> On Sat, Jan 16, 2016 at 12:49:12AM +0100, Martin Jambor wrote:
> > bootstrapping on i686-linux revealed the need for the following simple
> > patch. I've run into two types of compilation errors on
>
Hi,
On Sat, Jan 16, 2016 at 12:43:07PM +0100, Jakub Jelinek wrote:
> On Fri, Jan 15, 2016 at 06:23:05PM +0100, Martin Jambor wrote:
> > BRIG_KIND_OPERAND_REGISTER = 0x300a,
> > BRIG_KIND_OPERAND_STRING = 0x300b,
> > BRIG_KIND_OPERAND_WAVESIZE = 0x3009c,
> > BRI
ickly by making most of the contents of hsa-*.c
files compiled only conditionally (and leave potential hsa support on
non-linux platforms for later), but I will not have time to do the
change and test it properly until Monday.
But that will hopefully really be it,
Martin
2016-01-16 Martin J
On Fri, Jan 15, 2016 at 01:03:35PM +0100, Jakub Jelinek wrote:
> On Fri, Jan 15, 2016 at 11:37:32AM +0100, Jakub Jelinek wrote:
> > On Fri, Jan 15, 2016 at 11:14:33AM +0100, Martin Jambor wrote:
> > > > Martin, could you ask the HSA Foundation or AMD or whoever if there is
Hi,
On Fri, Jan 15, 2016 at 04:01:49PM +0100, Jakub Jelinek wrote:
> On Fri, Jan 15, 2016 at 03:53:23PM +0100, Martin Jambor wrote:
> > @@ -317,7 +319,7 @@ public:
> > bool
> > pass_ipa_hsa::gate (function *)
> > {
> > - return hsa_gen_requeste
Hi,
On Thu, Jan 14, 2016 at 03:41:34PM +0100, Jakub Jelinek wrote:
> On Wed, Jan 13, 2016 at 06:39:35PM +0100, Martin Jambor wrote:
> > +for (phi = hbb->m_first_phi;
> > +phi;
> > +phi = phi->m_next ? as_a (phi->m_next): NULL)
>
> Space before :
s.
Let me just quickly answer two comments as well:
On Thu, Jan 14, 2016 at 03:05:33PM +0100, Jakub Jelinek wrote:
> On Wed, Jan 13, 2016 at 06:39:34PM +0100, Martin Jambor wrote:
>
...
> > +#define HSA_WARN_MEMORY_ROUTINE "OpenMP device memory library routines
> > have "
On Thu, Jan 14, 2016 at 01:58:58PM +0100, Jakub Jelinek wrote:
> Otherwise LGTM.
>
> Jakub
Thanks Jakub, I have committed the following patch from Martin Liska
that addresses your comments.
Martin
2016-01-15 Martin Liska
* ipa-hsa.c (process_hsa_functions): Fixed coding style.
Thanks Jakub and Alex,
I have committed the following to the branch to address your comments:
2016-01-15 Martin Jambor
* gimple.h: Fixed comment of gimple_statement_omp_single_layout
* omp-low.c (get_target_argument_value): Fixed spelling in its
comment
Hi,
On Thu, Jan 14, 2016 at 05:18:56PM -0800, Ian Lance Taylor wrote:
> Jakub Jelinek writes:
>
> > On Wed, Jan 13, 2016 at 06:39:33PM +0100, Martin Jambor wrote:
> >> the following patch adds a BRIG (binary representation of HSAIL)
> >> representation description.
all of which has been posted in
https://gcc.gnu.org/ml/gcc-patches/2015-12/msg01884.html.
Thanks,
Martin
2016-01-13 Martin Jambor
gcc/
* builtin-types.def (BT_FN_VOID_UINT_PTR_INT_PTR): New.
(BT_FN_VOID_INT_OMPFN_SIZE_PTR_PTR_PTR_UINT_PTR_INT_INT): Removed
jections against HSA going forward,
including this file.
Thanks,
Martin
2015-12-04 Martin Jambor
* hsa-brig-format.h: New file.
diff --git a/gcc/hsa-brig-format.h b/gcc/hsa-brig-format.h
new file mode 100644
index 000..6e2fe75
--- /dev/null
+++ b/gcc/hsa-brig-format.h
@@ -0,0 +1,12
2016-01-13 Michael Matz
Martin Jambor
* hsa-regalloc.c: New file.
diff --git a/gcc/hsa-regalloc.c b/gcc/hsa-regalloc.c
new file mode 100644
index 000..5a42beb
--- /dev/null
+++ b/gcc/hsa-regalloc.c
@@ -0,0 +1,719 @@
+/* HSAIL IL Register allocation and out-of-SSA
tree-inline.h as I think that such a cleanup should be done
separately, but the structure could probably use some field-re
ordering to remove padding.
I hope I have grasped it correctly and that the patch is OK for trunk.
Thanks,
Martin
2016-01-13 Martin Jambor
* tree-inl
-12/msg00716.html with a number
of modifications requested by Jakub.
Thanks,
Martin
2016-01-13 Martin Jambor
Martin Liska
* plugin/plugin-hsa.c: New file.
diff --git a/libgomp/plugin/plugin-hsa.c b/libgomp/plugin/plugin-hsa.c
new file mode 100644
index 000..d888493
e same LTO partition so
that they can b registered together to libgomp.
Thanks,
Martin
2016-01-13 Martin Liska
Martin Jambor
* ipa-hsa.c: New file.
* lto-section-in.c (lto_section_name): Add hsa section name.
* lto-streamer.h (lto_section_type): Add h
communications with Jakub, I have the feeling he is fine
with the changes. But perhaps he or someone else would like to have
one more look.
Thanks,
Martin
2016-01-13 Martin Jambor
include/
* gomp-constants.h (GOMP_DEVICE_HSA): New macro.
(GOMP_VERSION_HSA): Likewise
Hi,
this is hopefully the last big re-post of the HSA patches. We have
incorporated all the feedback and found and fixed a couple more bugs.
The complete patch-set bootstraps and tests fine on an x86_64-linux,
when you do not enable HSA, there are a few expected warnings when HSA
is enabled which
Hi,
the pass manager changes required for HSA have already been committed
to trunk so all that remains are these additions to the pass pipeline.
This bit has already been approved by Richi in
https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00996.html
Thanks,
Martin
2016-01-13 Martin Jambor
/gcc-patches/2015-12/msg01284.html).
thanks,
Martin
2016-01-13 Martin Jambor
* Makefile.in (OBJS): Add new source files.
(GTFILES): Add hsa.c.
* common.opt (disable_hsa): New variable.
(-Whsa): New warning.
* config.in (ENABLE_HSA): New
Hi,
On Tue, Jan 12, 2016 at 02:38:15PM +0100, Jakub Jelinek wrote:
> On Tue, Jan 12, 2016 at 02:29:06PM +0100, Martin Jambor wrote:
> > GOMP_kernel_launch_attributes should not be there (it is a
> > reminiscence from before the device-specific target arguments) and
> > should
On Tue, Jan 12, 2016 at 06:36:21PM +0100, Martin Jambor wrote:
> > remap_decl (old_var, id);
> > }
> > - phase 2 - do the full remap_decls, but during that arrange that
> > remap_decl for non-zero id->remapping_type_depth if (!n) just returns
> > decl
Hi,
On Mon, Jan 11, 2016 at 05:38:47PM +0100, Jakub Jelinek wrote:
> On Mon, Jan 11, 2016 at 09:41:31AM +0100, Richard Biener wrote:
> > Hum. Can't you check id->remapping_type_depth?
For some reason, last week I reached the conclusion that no. But I
must have done something wrong because I hav
Hi,
On Fri, Dec 11, 2015 at 07:05:29PM +0100, Jakub Jelinek wrote:
> On Thu, Dec 10, 2015 at 06:52:23PM +0100, Martin Jambor wrote:
> > > > --- a/libgomp/task.c
> > > > +++ b/libgomp/task.c
> > > > @@ -581,6 +581,7 @@ GOMP_PL
2016-01-08 Martin Jambor
PR ipa/69044
* ipa-cp.c (estimate_local_effects): Do not clone for removal of
useless parameters if we cannot change function signature.
testsuite/
* gcc.target/i386/chkp-pr69044.c: New test.
diff --git a/gcc/ipa-cp.c b/gcc/ipa-cp.c
index
passed bootstrap and testing on i686-linux, I have also
included it in a bootstrap and testing that is underway on
x86_64-linux. OK if it passes there as well?
Thanks,
Martin
[PR ipa/66616] Copy can_change_signature flag to artificial thunks
2016-01-07 Martin Jambor
PR ipa/66616
found any.
If nobody has a better idea, is the following good for trunk? (I am
about to commit it to the hsa branch.) It has passed bootstrap and
testing on x86_64-linux.
Thanks,
Martin
2016-01-06 Martin Jambor
* tree-inline.h (copy_body_data): New field
decl_creation_prev
something
for another patch.
- "griddim" was changed to "_griddim_" at three places
- I fixed formatting in all the suggested ways.
Thanks,
Martin
2015-12-16 Martin Jambor
gcc/
* builtin-types.def: Removed a blank line.
* gimple-low.c (lower_stmt)
Hi,
On Thu, Dec 17, 2015 at 12:02:11PM +0300, Yury Gribov wrote:
> Another intransitive comparison in reload_pseudo_compare_func. Buggy
> scenario:
> 1) A and B are ints of equal presion so we return 0
> 2) C is REAL and thus can compare differently to A and B
>
> Cc-ing Martin who's the original
vector in that method (which is only used by IPA-CP at this
one spot and IPA-SRA so it should be fine). Bootstrapped and tested
on x86_64-linux. OK for trunk? And perhaps for the gcc-5 branch too?
Thanks,
Martin
2015-12-14 Martin Jambor
PR ipa/68851
* cgraph.c
Hi,
On Fri, Dec 11, 2015 at 10:20:20PM +0100, Jan Hubicka wrote:
> Actually I added
> if (!ipa_is_param_used (info, i))
>
> continue;
>
> shortcut to gather_context_independent_v
Hi,
PR 66616 happens because in find_more_scalar_values_for_callers_subset
we do not do the same thunk checks like we do in
propagate_constants_accross_call. I am in the process of
bootstrapping and testing the following patch to fix it. OK if it
passes?
Thanks,
Martin
2015-12-11 Martin
On Thu, Dec 10, 2015 at 05:56:26PM +0100, Jan Hubicka wrote:
> > Is this really necessary, is it not enough to remove the assignment to
> > ret below? If the parameter is not used, devirtualization time bonus,
> > which you then rely on estimate_local_effects, should be zero for it.
> >
> > It is
kes a big difference for debugging).
I have tossed this into a bootstrap and test run on an x86_64-linux
and found no issues. I believe the patch is quite obvious and so will
go ahead and commit it to trunk.
Thanks,
Martin
Add asssert and testcase for PR 68064
2015-12-09 Martin J
Hi,
thanks for the feedback. I have incorporated most of it into the
branch (the diff is below) but also have a few questions.
On Wed, Dec 09, 2015 at 12:35:36PM +0100, Jakub Jelinek wrote:
> On Mon, Dec 07, 2015 at 12:19:57PM +0100, Martin Jambor wrote:
> > +/* Flag set when the s
Hi,
On Mon, Dec 07, 2015 at 12:46:45PM +0100, Jakub Jelinek wrote:
> On Mon, Dec 07, 2015 at 12:17:58PM +0100, Martin Jambor wrote:
> > Because I have not been able to come up with any solution to failing
> > libgomp/testsuite/libgomp.c++/target-2.C, I have disabled use of
> >
Hi,
On Mon, Dec 07, 2015 at 12:19:08PM +0100, Martin Jambor wrote:
> Hi,
>
> this patch contains changes to the configuration mechanism and offload
> bits, so that users can build compilers with HSA support.
when writing up how to build an HSA-enabled GCC for the wiki page, and
Hi,
On Tue, Dec 08, 2015 at 10:43:15PM +, Richard Sandiford wrote:
> [Sorry for the low-quality review, was just reading out of interest...]
>
> Martin Jambor writes:
> > +If you configure GCC with HSA offloading but do not have the HSA
> > +run-time library installed i
Hi,
thanks for looking into this, I only have one question:
On Thu, Dec 10, 2015 at 08:30:37AM +0100, Jan Hubicka wrote:
> Martin,
> while looking into the ipa-cp dumps for bzip and Firefox I noticed few issues.
> First of all, ipcp_cloning_candidate_p calls
> optimize_function_for_speed_p (DECL
2015-12-04 Michael Matz
Martin Jambor
* hsa-regalloc.c: New file.
diff --git a/gcc/hsa-regalloc.c b/gcc/hsa-regalloc.c
new file mode 100644
index 000..9db4c1d
--- /dev/null
+++ b/gcc/hsa-regalloc.c
@@ -0,0 +1,719 @@
+/* HSAIL IL Register allocation and out-of-SSA
eering committee for
permission to add this file to GCC (and update it as HSA standard
evolves). Please let me know if there is something more I need to do
in this regard.
Thanks,
Martin
2015-12-04 Martin Jambor
* hsa-brig-format.h: New file.
diff --git a/gcc/hsa-brig-format.h b/gcc/h
gether and remain in the same LTO partition so
that they can b registered together to libgomp.
Thanks,
Martin
2015-12-04 Martin Liska
Martin Jambor
* ipa-hsa.c: New file.
* lto-section-in.c (lto_section_name): Add hsa section name.
* lto-streamer.h (lto_se
Hi,
the pass manager changes required for HSA have already been committed
to trunk so all that remains are these additions to the pass pipeline.
Thanks,
Martin
2015-12-04 Martin Jambor
Martin Liska
* passes.def: Schedule pass_ipa_hsa and pass_gen_hsail
nd yet
only re-implement the bare necessary minimum of functionality that is
already there. Any feedback is of course still very welcome.
Thanks,
Martin
2015-12-04 Martin Jambor
* builtin-types.def (BT_FN_VOID_UINT_PTR_INT_PTR): New.
(BT_FN_VOID_INT_OMPFN_SIZE_PTR_P
Subject: Make copy_gimple_seq_and_replace_locals copy seqs in omp clauses
Hi,
this is https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00477.html with
the early return requested by Jakub. Please refer to that previous
email for explanation why it is necessary.
Thanks,
2015-12-03 Martin Jambor
device-specific parameter and support
for asynchronous execution.
Any feedback will be greatly appreciated,
Martin
2015-12-04 Martin Jambor
Martin Liska
* plugin/plugin-hsa.c: New file.
diff --git a/libgomp/plugin/plugin-hsa.c b/libgomp/plugin/plugin-hsa.c
new file mode
2015-12-04 Martin Jambor
Martin Liska
include/
* gomp-constants.h (GOMP_DEVICE_HSA): New macro.
(GOMP_VERSION_HSA): Likewise.
(GOMP_TARGET_ARG_DEVICE_MASK): Likewise.
(GOMP_TARGET_ARG_DEVICE_ALL): Likewise.
(GOMP_TARGET_ARG_SUBSEQUENT_PARAM
make it a parameter rather than a
switch to emphasize the fact it will go away and to possibly allow us
select different levels of verbosity of the stores in the future).
Any feedback is very appreciated,
Martin
2015-12-04 Martin Jambor
gcc/
* Makefile.in (OBJS): Add new so
Hi,
I'm sorry it took me more than a month to come up with another round
of patches aiming at merging the HSA branch into the trunk. Keeping
up-to date with the latest changes in the OpenMP 4.5 area was
strenuous and we have discovered and fixed a few bugs as I intensified
my testing efforts.
Wh
e_seq_copy,
walk_gimple_seq and other to be technically incorrect, that is what I
have done in the patch below, which fixes my last ICEs and which I
have already committed to the branch.
Any feedback is of course very much appreciated,
Martin
2015-12-03 Martin Jambor
* tre
Hi,
I have found that adding the following checking assert very useful
when debugging omp lowering issues, so I have added it to the hsa
branch. I hope that nobody will mind, but it of course is not an
essential thing to have if someone does.
Thanks,
Martin
2015-12-03 Martin Jambor
. Fixed with this patch, where
pop_gimplify_context does the right thing like at other places in
omp-low.c. Committed to the branch.
Thanks,
Martin
2015-11-30 Martin Jambor
* omp-low.c (attempt_target_gridification): Use gimplify_expr.
---
gcc/omp-low.c | 27
Hi,
when looking at the attempt_target_gridification function I realized I
forgot to to replace some of the early code with proper gimple
statement access function calls. This patch addresses that.
Committed to the branch.
Thanks,
Martin
2015-11-30 Martin Jambor
* omp-low.c
grid size and group size along that dimension
(hence the new clause type has two parameters).
Committed to the branch, I will be preparing a new diff against the
trunk shortly.
Thanks,
Martin
2015-11-30 Martin Jambor
* gimple.c (gimple_omp_target_init_dimensions): Removed
Hi,
the patch below makes libgomp/testsuite/libgomp.c/target-28.c pass on
HSA, where it previously did not like the two static variables with
the same name. Committed to the branch.
Thanks,
Martin
2015-11-25 Martin Jambor
* hsa.c (hsa_get_declaration_name): Return ASM name for
Hi,
when looking at why target-12.c and target-24.c in
libgomp/testsuite/libgomp.c/, I found two other places in libgomp's
target.c where shared-memory devices ought to be treated like the
host. Committed to the branch.
Thanks,
Martin
2015-11-25 Martin Jambor
libgomp/
* tar
On Mon, Nov 23, 2015 at 03:16:42PM +0100, Jakub Jelinek wrote:
> On Mon, Nov 23, 2015 at 03:12:05PM +0100, Martin Jambor wrote:
> > +/* Thread routine to run a kernel asynchronously. */
> > +
> > +static void *
> > +run_kernel_asynchronously (void *thread_arg)
> >
welcome,
Martin
2015-11-23 Martin Jambor
gcc/
* builtin-types.def (BT_FN_VOID_PTR_INT_PTR): Removed.
(BT_FN_VOID_UINT_PTR_INT_PTR): New.
* fortran/types.def (BT_FN_VOID_PTR_INT_PTR): Removed.
(BT_FN_VOID_UINT_PTR_INT_PTR): New.
* hsa-brig.c: Include
part of this one, then there
are 8 bits for an ID of the argument and then up to 16 bits for a
value.
Thanks,
Martin
2015-11-20 Martin Jambor
gcc/
* omp-low.c (get_target_argument_identifier_1): New function.
(get_target_argument_identifier): Likewise
ew places where
libgomp has to treat shared memory devices like it treats host, and so
I added that to the patch too.
The hunk in gomp_create_target_task should have been in the previous
merge from trunk but I forgot to add it then.
Any feedback welcome,
Martin
2015-11-23 Martin Jambor
libgom
Hi,
I have applied the following patch tot he HSA branch to fix some
rather unexpected type requirements of the HSA finalizer we discovered
when running the target-3[34].c libgomp tests.
Thanks,
Martin
2015-11-23 Martin Jambor
* hsa-gen.c (gen_hsa_binary_operation): Make immediate
Hi,
On Mon, Nov 23, 2015 at 12:00:25AM +0100, Jan Hubicka wrote:
> Hi,
> here is updated patch which I finally comitted today. It addresses all the
> comments
> and also fixes one nasty bug that really cost me a lot of time to understand.
>
> + /* LTO type merging does not make any diffe
Hi,
this hsa patch is analogous to the for-trunk RFC I have sent a while
ago and implements passing HSA-specific grid sizes through a
device-specific argument. Committed to the branch.
Thanks,
Martin
2015-11-13 Martin Jambor
include/
* gomp-constants.h
ll be
an identifier and the second one the value itself.
As suggested by Jakub, the first two elements will be common NUM_TEAMS
and THREAD_LIMIT from the teams construct, if present.
Any comments welcome,
Thanks,
Martin
2015-11-13 Martin Jambor
include/
* gomp-consta
On Thu, Nov 12, 2015 at 12:19:50PM +0100, Jakub Jelinek wrote:
> On Mon, Nov 09, 2015 at 05:58:56PM +0100, Martin Jambor wrote:
> > > But I don't see any way to disable it on the command line? (no switch?)
> >
> > No, the switch is -foffload, which has missing docume
Hi
I have just Merged trunk revision 230248 into the hsa branch. I will
prepare a new submission for inclusion to trunk tomorrow.
Thanks,
Martin
On Fri, Nov 06, 2015 at 09:38:21AM +0100, Richard Biener wrote:
> On Thu, 5 Nov 2015, Martin Jambor wrote:
>
> > Hi,
> >
> > in the previous email I wrote we need to "change behavior" of a few
> > optimization passes. One was the flattening of GPU fu
Hi,
On Fri, Nov 06, 2015 at 09:42:25AM +0100, Richard Biener wrote:
> On Thu, 5 Nov 2015, Martin Jambor wrote:
>
> > Hi,
> >
> > the following small part of the merge deals with new options. It adds
> > four independent things:
> >
> > 1) flag_dis
Hi,
On Thu, Nov 05, 2015 at 10:47:44PM +, Joseph Myers wrote:
> This new file should have the standard libgomp copyright / license notice.
>
Oops, thanks for pointing this out. I am about to commit the
following remedy to the branch.
Thanks,
Martin
2015-11-09 Martin
On Thu, Nov 05, 2015 at 10:47:15PM +, Joseph Myers wrote:
> On Thu, 5 Nov 2015, Martin Jambor wrote:
>
> > libgomp plugin to be built. Because the plugin needs to use HSA
> > run-time library, I have introduced options --with-hsa-runtime (and
> > more precise --wit
Hi,
On Thu, Nov 05, 2015 at 10:57:33PM +0100, Martin Jambor wrote:
>
...
>
> For convenience of anybody reviewing the code, I'm attaching a very
> simple testcase with selection of dumps that illustrate the whole
> process.
>
My apologies, I have forgotten to att
2015-11-05 Michael Matz
Martin Jambor
* hsa-regalloc.c: New file.
diff --git a/gcc/hsa-regalloc.c b/gcc/hsa-regalloc.c
new file mode 100644
index 000..3919258
--- /dev/null
+++ b/gcc/hsa-regalloc.c
@@ -0,0 +1,711 @@
+/* HSAIL IL Register allocation and out-of-SSA
tandards. I hope that is reasonable.
Thanks,
Martin
2015-11-05 Martin Jambor
* hsa-brig-format.h: New file.
diff --git a/gcc/hsa-brig-format.h b/gcc/hsa-brig-format.h
new file mode 100644
index 000..f099fc6
--- /dev/null
+++ b/gcc/hsa-brig-format.h
@@ -0,0 +1,1283 @@
+/* HSAIL an
901 - 1000 of 1636 matches
Mail list logo