[PATCH] make -freg-struct-return visibly a negative alias of -fpcc-struct-return

2024-04-28 Thread Alexandre Oliva
fpeel-loops @@ -2596,7 +2596,7 @@ Common Var(flag_record_gcc_switches) Record gcc command line switches in the object file. freg-struct-return -Common Var(flag_pcc_struct_return,0) Optimization +Common NegativeAlias Alias(fpcc_struct_return) Optimization Return small aggregates in registers. fregmo

Re: [PATCH] adjust vectorization expectations for ppc costmodel 76b

2024-04-28 Thread Alexandre Oliva
n testing for -mcpu=power7 on ppc64le-linux-gnu: it does vectorize the loop with 13 iterations. We need 16 iterations, as in an earlier version of this test, for it to pass for -mcpu=power7, but then it doesn't pass for -mcpu=power6. It looks like we're going to have to adjust

Re: [PATCH] ppc: testsuite: pr79004 needs -mlong-double-128

2024-04-28 Thread Alexandre Oliva
k into massaging the patch into that when I get back, if you haven't rendered it obsolete by then ;-) Thanks, -- Alexandre Oliva, happy hackerhttps://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer More tolerance and less prejudice are key f

Re: [PATCH] ppc: testsuite: pr79004 needs -mlong-double-128

2024-04-29 Thread Alexandre Oliva
(buffer, "3") != 0; } -} [add_options_for_ppc_long_double_override_64bit ""]] +} [add_options_for_long_double_64bit ""]] } # Return the appropriate options to specify that long double uses the IEEE -- Alexandre Oliva, happy hackerhttps://FSFLA.org/blogs/lxo/ Fre

Re: [PATCH] ppc: testsuite: pr79004 needs -mlong-double-128

2024-04-29 Thread Alexandre Oliva
and leave them all alone, despite the interface oddity. -- Alexandre Oliva, happy hackerhttps://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer More tolerance and less prejudice are key for inclusion and diversity Excluding neuro-others for not behaving ""normal"" is *not* inclusive

Re: [PATCH] decay vect tests from run to link for pr95401

2024-04-29 Thread Alexandre Oliva
nd I can't post feedback there) and to https://ci.linaro.org/job/tcwg_gnu_embed_check_gcc--master-thumb_m7_hard_eabi-build/10/artifact/artifacts/00-sumfiles/ (where I could get useful information) I'm reverting the patch, and I'll see about some alternate approach that can accommodate th

[PATCH] [libstdc++-v3] [testsuite] improve future/*/poll.cc calibration

2024-08-30 Thread Alexandre Oliva
t_until(epoch) should not be terribly slow. - VERIFY( wait_until_sys_epoch < (ready * 100) ); - VERIFY( wait_until_steady_epoch < (ready * 100) ); + VERIFY( wait_until_sys_epoch < (ready * 200) ); + VERIFY( wait_until_steady_epoch < (ready * 200) ); #endif } -- Alexandre Oliva, happy ha

[PATCH v2] testsuite: introduce hostedlib effective target

2024-09-03 Thread Alexandre Oliva
On Nov 9, 2023, Mike Stump wrote: > On Nov 8, 2023, at 8:29 AM, Alexandre Oliva wrote: >> >> On Nov 5, 2023, Mike Stump wrote: >> >>> that, otherwise, I'll approve this version. >> >> FWIW, this version is not usable as is. Something went

[PATCH] testsuite: a few more hostedlib adjustments

2024-09-12 Thread Alexandre Oliva
On Sep 12, 2024, Mike Stump wrote: > On Sep 3, 2024, at 11:44 PM, Alexandre Oliva wrote: >> >> Here's an updated and refreshed version that gets trunk built with >> --disable-hosted-libstdcxx on x86_64-linux-gnu to not get any spurious >> fails during in-tre

Re: [PATCH] testsuite: a few more hostedlib adjustments

2024-09-14 Thread Alexandre Oliva
r existing ones will be able to catch them from now on. Presumably there are still arch-specific tests that will have to be covered as well, so it would be good if other platforms' automated testers were to cover them as well. -- Alexandre Oliva, happy hackerhttps://FSFLA.org

Re: [PATCH v4 00/20] resurrect automatic dependencies

2013-09-24 Thread Alexandre Oliva
7;m concerned, the patch is approved, but I suggest giving Paolo a couple more days to chime in, since he reviewed the previous version. Thanks! -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA

Re: [PATCH, libvtv] Fix configure/testsuite issues with libvtv

2013-09-24 Thread Alexandre Oliva
, but the ChangeLog entries have two different typos for re-instate. Ok with the typos fixed. Thanks! -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin America board memb

[PR54177] Deal with var_lowpart failure in function parameters

2012-10-02 Thread Alexandre Oliva
failure in function parameters. From: Alexandre Oliva for gcc/ChangeLog * var-tracking.c (vt_add_function_parameter): Bail if var_lowpart fails. --- gcc/var-tracking.c |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/gcc/var-tracking.c b/gcc/var-tracking.c ind

[PR53135] Use block4 form for large debug expressions

2012-10-02 Thread Alexandre Oliva
I'm keeping the testcase open because we still have an underlying problem and other improvements to make. Use block4 form for large debug expressions. From: Alexandre Oliva for gcc/ChangeLog PR debug/53135 * dwarf2out.c (value_format): Use block4 for dw_val_class_loc when needed.

Re: [PR54551] global dead debug pseudo tracking in fast-dce

2012-10-02 Thread Alexandre Oliva
On Sep 25, 2012, Jakub Jelinek wrote: > On Sun, Sep 23, 2012 at 07:59:37AM -0300, Alexandre Oliva wrote: >> This patch introduces a global mode of dead_debug tracking for use in >> fast DCE. If a debug use reaches the top of a basic block before >> finding its death p

Re: [PATCH] Improve debug info for partial inlining (PR debug/54519, take 2)

2012-10-03 Thread Alexandre Oliva
ave a pending (still unreviewed) patch that might address :-), so to speak, this problem. for gcc/ChangeLog from Alexandre Oliva PR debug/53671 PR debug/49888 * var-tracking.c (attrs_list_by_loc_eq): New. (track_stack_pointer): New. (dataflow_set_merge): Use it. (vt_initialize): Record the i

Re: [PATCH] Improve var-tracking memory disambiguation with frame pointer (PR debug/54796)

2012-10-11 Thread Alexandre Oliva
associate sp from its previous value and call > cselib_set_value_sp_based on a new VALUE created for sp. > * gcc.dg/guality/pr54796.c: New test. > +/* Test whether a value is preserved. */ > + > +bool > +cselib_sp_based_value_p (cselib_val *v) The function comment nee

[PR54693] loss of debug info in jump threading and loop ivopts

2012-10-25 Thread Alexandre Oliva
chset had already been cooking for a while, I figured I'd stop at this point and post the patchset with this caveat. Ok to install? Stabilize i386 sched calls and args WRT debug insns From: Alexandre Oliva for gcc/ChangeLog PR debug/54693 * config/i386/i386.c (

Re: [PATCH] Improve expansion into DEBUG_IMPLICIT_PTR (PR debug/54970)

2012-10-26 Thread Alexandre Oliva
expr changes are ok. I guess with this you got the whole patch approved. -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin America board member Free Software Evangelist Red Hat Brazil Compiler Engineer

Re: [PR54693] loss of debug info in jump threading and loop ivopts

2012-10-29 Thread Alexandre Oliva
On Oct 26, 2012, Jakub Jelinek wrote: > On Fri, Oct 26, 2012 at 04:30:41AM -0200, Alexandre Oliva wrote: >> From: Alexandre Oliva Fixed .git/config and the patches to use my @redhat.com address for these patches. >> +++ b/gcc/testsuite/gcc.dg/guality/pr54693.c > The #incl

Re: [PATCH] Improve valtrack handling during fast DCE (PR debug/54953)

2012-10-29 Thread Alexandre Oliva
_WITH_REG_FORCE. > * dce.c (word_dce_process_block, dce_process_block): Pass > DEBUG_TEMP_AFTER_WITH_REG_FORCE if insn is needed and therefore > not going to be eliminated. This is ok, thanks. -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the chang

Re: dead_debug_local_finish may be used uninit warning

2012-10-30 Thread Alexandre Oliva
sting the patch, and I'll check it as obvious when it's done. Avoid uninitialized warning when globalizing dead debug uses From: Alexandre Oliva for gcc/ChangeLog PR debug/54551 PR debug/54693 * valtrack.c (dead_debug_promote_uses): Assert-check that global used bit was clear and

Re: [PR54693] loss of debug info in jump threading and loop ivopts

2012-10-30 Thread Alexandre Oliva
me, from the first loop. More like a left-over from before I split the loop into two, but yeah, I'll take that out. > BTW, I'd also think that the first loop should set count = 2 if > the debug stmt already has a non-trivial expression Good! Will do. -- Alexandre Oliva, freedom f

Re: [PR54693] loss of debug info in jump threading and loop ivopts

2012-10-30 Thread Alexandre Oliva
On Oct 30, 2012, Alexandre Oliva wrote: >> BTW, I'd also think that the first loop should set count = 2 if >> the debug stmt already has a non-trivial expression > Good! Will do. Here's the revised patch. Bootstrap completed, test nearly finished. Ok to install i

[PR69315] enable finish_function to recurse for constexpr functions

2016-01-26 Thread Alexandre Oliva
ool operator==(Iter, Iter) { + return true; +} + +template constexpr bool operator!=(Iter a, Iter b) { + return a == b; +} -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin Ameri

Re: [PATCH, 69217]: [6 Regression] ICE at var-tracking.c:5038 Segmentation fault

2016-01-26 Thread Alexandre Oliva
On Jan 23, 2016, Iain Buclaw wrote: > PR rtl-optimization/69217 > * var-tracking.c (tracked_record_parameter_p): Don't segfault if there > are no TYPE_FIELDS set for the record type. This looks good to me, thanks. -- Alexandre Oliva, freedom fighterht

[PR69634] fix debug_insn-inconsistent REG_N_CALLS_CROSSED

2016-02-06 Thread Alexandre Oliva
_0[1] + v16u16_1[3] + v16u64_1[0] + v16u64_1[1]; +} -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin America board member Free Software Evangelist|Red Hat Brasil GNU Toolchain Engineer

Re: [PR69315] enable finish_function to recurse for constexpr functions

2016-02-10 Thread Alexandre Oliva
On Jan 26, 2016, Alexandre Oliva wrote: > We don't want finish_function to be called recursively from mark_used. > However, it's desirable and necessary to call itself recursively when > performing delayed folding, because that may have to instantiate and > evaluate conste

better debug info for C++ cdtors, aliases, thunks and other trampolines

2016-02-19 Thread Alexandre Oliva
emble_alias (alias->decl, - DECL_ASSEMBLER_NAME (decl)); + { + do_assemble_alias (alias->decl, +DECL_ASSEMBLER_NAME (decl)); + if (!DECL_IGNORED_P (alias->decl) && !DECL_IGNORED_P (decl)) + (*debug_hooks->alia

Re: better debug info for C++ cdtors, aliases, thunks and other trampolines

2016-02-23 Thread Alexandre Oliva
is implemented as DIEs is an internal implementation detail, so why should assumptions made by the user side of the interface matter? -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/

Re: [PR69315] enable finish_function to recurse for constexpr functions

2016-02-24 Thread Alexandre Oliva
On Feb 10, 2016, Alexandre Oliva wrote: > On Jan 26, 2016, Alexandre Oliva wrote: >> We don't want finish_function to be called recursively from mark_used. >> However, it's desirable and necessary to call itself recursively when >> performing delayed fo

Re: [PR64164] drop copyrename, integrate into expand

2015-07-16 Thread Alexandre Oliva
On Jun 10, 2015, Richard Biener wrote: > On Wed, Jun 10, 2015 at 2:24 AM, Alexandre Oliva wrote: >> This caused the sparc regression reported by Eric in >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64164#c37 >> We need to match the mode of the rtl created for

Re: [PR64164] drop copyrename, integrate into expand

2015-07-16 Thread Alexandre Oliva
I'll post a followup as soon as I sort that out. -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin America board member Free Software Evangelist|Red Hat Brasil GNU Toolchain Engineer

Re: [PR64164] drop copyrename, integrate into expand

2015-07-18 Thread Alexandre Oliva
On Jul 16, 2015, Alexandre Oliva wrote: > So, I decided to run a ppc64le-linux-gnu bootstrap, just in case, and > there are issues with split complex parms that caused go and fortran > libs to fail the build. This incremental patch, along with the previously-posted patches, fix spli

Re: [PR64164] drop copyrename, integrate into expand

2015-07-22 Thread Alexandre Oliva
On Jul 21, 2015, Richard Biener wrote: > On Sat, Jul 18, 2015 at 9:37 AM, Alexandre Oliva wrote: >> On Jul 16, 2015, Alexandre Oliva wrote: >> + /* If we are assigning parameters for a function, rather >> +than for a call, propagate the RTL o

Re: [PR64164] drop copyrename, integrate into expand

2015-07-22 Thread Alexandre Oliva
On Jul 21, 2015, Richard Biener wrote: > On Sat, Jul 18, 2015 at 9:37 AM, Alexandre Oliva wrote: >> + if (cfun->gimple_df) > If the cfun->gimple_df check is to decide whether this is a call or a function > then no, this can't work reliably. What is this te

Re: [PR64164] drop copyrename, integrate into expand

2015-07-23 Thread Alexandre Oliva
h at the end): [PR64164] Drop copyrename, use coalescible partition as base when optimizing. From: Alexandre Oliva for gcc/ChangeLog PR rtl-optimization/64164 * Makefile.in (OBJS): Drop tree-ssa-copyrename.o. * tree-ssa-copyrename.c: Removed. * opts.c (defa

Re: PATCH: PR bootstrap/66978: [6 Regression] bootstrap failure with --with-multilib-list=m32,m64,mx32

2015-07-24 Thread Alexandre Oliva
Pmode if needed. > + if (GET_MODE (local) != Pmode) > + local = convert_to_mode (Pmode, local, > + TYPE_UNSIGNED (TREE_TYPE (parm))); > + -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to

Re: [PR64164] drop copyrename, integrate into expand

2015-07-24 Thread Alexandre Oliva
On Jul 23, 2015, Segher Boessenkool wrote: > On Thu, Jul 23, 2015 at 12:29:14PM -0300, Alexandre Oliva wrote: >> Yeah. Thanks, I've tested it with this change, and I'm now checking >> this in (full patch first; adjusted incremental patch at the end): > Unfortunatel

Re: [PR64164] drop copyrename, integrate into expand

2015-07-24 Thread Alexandre Oliva
On Jul 23, 2015, "H.J. Lu" wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66983 My test logs with the patch have: PASS: c-c++-common/dfp/func-vararg-dfp.c execution test so I think this one is not caused by the PR64164 patch. -- Alexandre Oliva, freedom fighterhtt

Re: [PR64164] drop copyrename, integrate into expand

2015-07-24 Thread Alexandre Oliva
orrupted, and I don't want to mess with it before running an fsck which will take me a while. Apologies for the breakage. I'll add ppc64-linux-gnu regression testing to the test list for this patchset. Thanks, -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must

Re: [PR64164] drop copyrename, integrate into expand

2015-07-24 Thread Alexandre Oliva
t and try to find out what the source of the difference in test results H.J. and I get could be. -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin America board member Free Softwa

Re: [PR64164] drop copyrename, integrate into expand

2015-07-24 Thread Alexandre Oliva
On Jul 24, 2015, "H.J. Lu" wrote: > On Fri, Jul 24, 2015 at 11:43 AM, Alexandre Oliva wrote: >> On Jul 23, 2015, "H.J. Lu" wrote: >> >>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66983 >> >> My test logs with the patch have: >&g

Re: PATCH: PR bootstrap/66978: [6 Regression] bootstrap failure with --with-multilib-list=m32,m64,mx32

2015-07-27 Thread Alexandre Oliva
On Jul 24, 2015, "H.J. Lu" wrote: > On Fri, Jul 24, 2015 at 10:53 AM, Alexandre Oliva wrote: >> On Jul 24, 2015, "H.J. Lu" wrote: >> >>> Static chain returned from get_rtl_for_parm_ssa_default_def may not >>> have Pmode. This patch co

Re: [PR64164] drop copyrename, integrate into expand

2015-07-27 Thread Alexandre Oliva
On Jul 24, 2015, David Edelsohn wrote: > On Fri, Jul 24, 2015 at 4:02 PM, Alexandre Oliva wrote: >> On Jul 23, 2015, David Edelsohn wrote: >> >>> I request that this patch be reverted (again). >> >> Might I kindly ask you to please do so for me.

Re: [PR64164] drop copyrename, integrate into expand

2015-07-29 Thread Alexandre Oliva
On Jul 23, 2015, Segher Boessenkool wrote: > On Thu, Jul 23, 2015 at 12:29:14PM -0300, Alexandre Oliva wrote: >> Yeah. Thanks, I've tested it with this change, and I'm now checking >> this in (full patch first; adjusted incremental patch at the end): > Unfortunatel

Re: [PR64164] drop copyrename, integrate into expand

2015-07-29 Thread Alexandre Oliva
all the -m32 -msse2 regressions) in the git branch aoliva/pr64164, but I'm going to investigate a few more issues affecting other targets before I start full regression tests all over the build farm ;-) -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you

Re: [PR64164] drop copyrename, integrate into expand

2015-08-03 Thread Alexandre Oliva
I don't intend to install it this week, even if approved, because I'm going to be away Aug 5-10, and I would like to be around should any further problems arise. So, ok to install when I return? [PR64164] Drop copyrename, use coalescible partition as base when optimizing. From: Alexandre Oli

Re: [PR64164] drop copyrename, integrate into expand

2015-08-04 Thread Alexandre Oliva
le but not exactly easy. As for codegen, I hope to have covered all cases now, but should we find out I haven't, I'll try the split and see what that gets us. Did you have any special cases in mind that it looks like I may have missed? Thanks, -- Alexandre Oliva, freedom fighterht

Re: [PR64164] drop copyrename, integrate into expand

2015-08-05 Thread Alexandre Oliva
oes what it used to, namely, copy the incoming hard reg to a pseudo, and then use the pseudo as the MEM address. > But I guess you have eyed code-gen changes a bit anyway. Yeah. Not much has changed in the before parm_birth area; expected changes have to do with the pseudo numbering. IIRC, a

Re: [PR64164] drop copyrename, integrate into expand

2015-09-23 Thread Alexandre Oliva
MEM, we'll just load them to (coalescible) pseudos and use the pseudos instead, instead of coalescing other vars that expected pseudos to the same MEM. Is this ok to install? revert to assign_parms assignments using default defs From: Alexandre Oliva Revert the fragile and complicated

Re: [PR64164] drop copyrename, integrate into expand

2015-10-07 Thread Alexandre Oliva
not sure using these additional information would make it better overall. Compiling with -fno-tree-dominator-opts skips uncprop so that we don't even try to coalesce other variables with , so we get the code expected by the testcase. But we obviously don't want to disable this optimiz

[PR67828] don't unswitch loops on undefined SSA values (was: Re: [PR64164] drop copyrename, integrate into expand)

2015-10-08 Thread Alexandre Oliva
d behavior. Regstrapped on x86_64-linux-gnu and i686-linux-gnu. Ok to install? [PR67828] don't unswitch on default defs of non-parms From: Alexandre Oliva for gcc/ChangeLog PR rtl-optimizatoin/67828 * tree-ssa-loop-unswitch.c: Include tree-ssa.h. (tree_may_unswitch_on): Do

[PR67766] reorder return value copying from PARALLELs and CONCATs (was: Re: [PR64164] drop copyrename, integrate into expand)

2015-10-08 Thread Alexandre Oliva
y, we should be fine with this simple change, that Uroš kindly tested with an alpha-linux-gnu regstrap. I tested it myself on x86_64-linux-gnu and i686-linux-gnu. Ok to install? [PR67766] reorder handling of parallels, concats and promoted values in return From: Alexandre Oliva for gcc/Chan

[PR67891] drop is_gimple_reg test from set_parm_rtl (was: [PR67766] reorder return value copying from PARALLELs and CONCATs)

2015-10-09 Thread Alexandre Oliva
On Oct 9, 2015, Alexandre Oliva wrote: > This fixes fallout from the PR64164 expander revamp. > Uroš kindly tested with an alpha-linux-gnu regstrap. The one regression he mentioned from that run was gcc.dg/pr43300.c. The vector parameter there is handled by the emit_block_move c

Re: [PR67891] drop is_gimple_reg test from set_parm_rtl

2015-10-10 Thread Alexandre Oliva
d there was agreement to go with it. https://gcc.gnu.org/ml/gcc-patches/2011-06/msg01746.html I'm leaving it alone, since I can't reasonably test on the platforms where the problems showed up. -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change y

Re: Do not use TYPE_CANONICAL in useless_type_conversion

2015-10-13 Thread Alexandre Oliva
a compatible mode? This patch was regstrapped on i686-linux-gnu and x86_64-linux-gnu. Ok to install? support BLKmode inputs for store_bit_field From: Alexandre Oliva Revision 228586 changed useless_type_conversion_p and added mode changes for MEM:BLKmode inputs in store_expr_with_bounds, but i

Re: Do not use TYPE_CANONICAL in useless_type_conversion

2015-10-13 Thread Alexandre Oliva
On Oct 13, 2015, Eric Botcazou wrote: > Note that this is PR middle-end/67912. Thanks. I added this piece of information to the ChangeLog entry, and checked the patch in. -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the wo

Re: [PR67891] drop is_gimple_reg test from set_parm_rtl

2015-10-13 Thread Alexandre Oliva
On Oct 12, 2015, Richard Biener wrote: > On Sat, Oct 10, 2015 at 3:16 PM, Alexandre Oliva wrote: >> On Oct 9, 2015, Richard Biener wrote: >> >>> Ok. Note that I think emit_block_move shouldn't mess with the addressable >>> flag. >> >> I h

Re: Do not use TYPE_CANONICAL in useless_type_conversion

2015-10-14 Thread Alexandre Oliva
hoever reported the ia64 problem that the patch fixed it too ;-) -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin America board member Free Software Evangelist|Red Hat Brasil GNU Toolchain Engineer

[PR68083] stop ifcombine from moving uninitialized uses before their guards

2015-10-30 Thread Alexandre Oliva
a_useless_type_conversion (tree); extern tree tree_ssa_strip_useless_type_conversions (tree); extern bool ssa_undefined_value_p (tree, bool = true); +extern bool gimple_uses_undefined_value_p (gimple *); extern void execute_update_addresses_taken (void); /* Given an edge_var_map V, return the PHI arg definition. */

Re: [PR68083] stop ifcombine from moving uninitialized uses before their guards

2015-10-30 Thread Alexandre Oliva
On Oct 30, 2015, Alexandre Oliva wrote: > Incidentally, bb_no_side_effects_p (inner_cond_bb) is called in all four > tests in tree_ssa_ifcombine_bb_1, for each outer_cond_bb that > tree_ssa_ifcombine_bb might choose. Is there any reason to not factor > it out to the test that checks

Re: [PR67891] drop is_gimple_reg test from set_parm_rtl

2015-11-02 Thread Alexandre Oliva
On Oct 14, 2015, Richard Biener wrote: > On Wed, Oct 14, 2015 at 5:25 AM, Alexandre Oliva wrote: >> On Oct 12, 2015, Richard Biener wrote: >> >>> On Sat, Oct 10, 2015 at 3:16 PM, Alexandre Oliva wrote: >>>> On Oct 9, 2015, Richard Biener wrote

Re: [PR67891] drop is_gimple_reg test from set_parm_rtl

2015-11-02 Thread Alexandre Oliva
On Nov 3, 2015, Jeff Law wrote: > On 11/02/2015 06:11 PM, Alexandre Oliva wrote: >> On Oct 14, 2015, Richard Biener wrote: >>> It looks ok to me but lacks a comment in mark_addressable_1 why we >>> do this queueing when currently expanding to RTL. >> >>&

Re: [PR64164] drop copyrename, integrate into expand

2015-11-04 Thread Alexandre Oliva
On Sep 23, 2015, Alexandre Oliva wrote: > @@ -2982,38 +2887,39 @@ assign_parm_setup_block (struct assign_parm_data_all > *all, [snip] > + if (GET_CODE (reg) != CONCAT) > +stack_parm = reg; > + else > +/* This will use or allocate a stack slo

Re: [PATCH] PR 68192 Export AIX TLS symbols

2015-11-05 Thread Alexandre Oliva
d GCC >> 4.9.x. > I think it's okay to wait for the patch to be upstream. *nod*. The reason we want it upstream first is that we don't want a regression when someone updates the libtool version in GCC, now or in the future. It's such a simple patch that it should be really

Re: [PR64164] drop copyrename, integrate into expand

2015-11-10 Thread Alexandre Oliva
un is running slower and probably won't be done before I call it a day, but I wanted to give you something before taking off for the day. Is this ok to install if ppc64-linux-gnu also regstraps successfully? [PR67753] adjust for padding when bypassing memory in assign_parm_setup_block Fro

Re: [PR64164] drop copyrename, integrate into expand

2015-11-11 Thread Alexandre Oliva
raight to shifting when we get there. I tried to document my reasoning in the comments, but maybe it was still too obscure? -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin Amer

Re: [PR64164] drop copyrename, integrate into expand

2015-11-16 Thread Alexandre Oliva
On Nov 13, 2015, Jeff Law wrote: > On 11/11/2015 11:10 AM, Alexandre Oliva wrote: >> On Nov 10, 2015, Jeff Law wrote: >> >>>> * function.c (assign_parm_setup_block): Right-shift >>>> upward-padded big-endian args when bypassing the stack slot. &

[PR67335] drop dummy zero from reverse VTA ops, fix infinite recursion

2015-11-26 Thread Alexandre Oliva
s known to be 0), as that - breaks a lot of routines during var-tracking. */ - ret = gen_rtx_fmt_ee (PLUS, GET_MODE (val), val, const0_rtx); break; default: gcc_unreachable (); -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change

Re: [PR67335] drop dummy zero from reverse VTA ops, fix infinite recursion

2015-11-26 Thread Alexandre Oliva
On Nov 26, 2015, Alexandre Oliva wrote: > for gcc/ChangeLog > PR debug/67355 > * var-tracking.c (reverse_op): Don't add dummy zero to reverse > ops that simplify back to the original value. > * alias.c (refs_newer_value_p): Cut off recursion for

Re: [PR67335] drop dummy zero from reverse VTA ops, fix infinite recursion

2015-11-27 Thread Alexandre Oliva
e_op of an earlier insn, between the debug insn and the second plus, and that equivalence cuts off the infinite recursion when checking they're equal for cselib. -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-10 Thread Alexandre Oliva
to use the most efficient insns to do the work. It's certainly good to allow machine-specific optimized code sequences, but it would certainly be desirable to have a machine-independent fallback. It doesn't seem exceedingly hard to loop over the registers and emit a (set (reg:M N) (const

Re: [PATCH] introduce attribute exalias

2020-08-14 Thread Alexandre Oliva
es, namely, attribute exalias("symbol_name") as a means to have symbol_name output as a same-linkage alias for functions, variables, and for C++ class types' RTTI symbols. Thanks in advance, On Aug 7, 2020, Alexandre Oliva wrote: > Since last week's patchlet, I've delay

Re: [PATCH] introduce attribute exalias

2020-08-14 Thread Alexandre Oliva
ed attribute that might be more mnemonic is "aka"; unfortunately, that's pretty much a synonym to alias, so it might be mistaken as such, rather than as a complementary feature, akin to the other end of a power extension cable: whereas alias does the job of a plug, *ex*alias provides a socket/*out*let. -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: [PATCH] introduce attribute exalias

2020-08-14 Thread Alexandre Oliva
t;aliased_to" or so. But I'm not seriously suggesting us to rename a long-available attribute while assigning uses thereof a different semantics, that would be preposterous. Since you don't seem to have liked 'aka' either, how about 'nickname', or 'nicknamed'? A more convenient name to refer to an entity is exactly what this is about, eh? -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: [PATCH] introduce attribute exalias

2020-08-15 Thread Alexandre Oliva
;t get exported. You could use it, and then another alias declaration referencing it, to get it exported, if you wish, but that was enough of a reason for me to stay away from the term "export". > *** right now Darwin fails silently (there doesn’t seem to be the usual error &g

Re: [PATCH] introduce attribute exalias

2020-08-15 Thread Alexandre Oliva
ou'd be bringing back the very variation across targets that this feature was designed to overcome. -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: [PATCH] Fix libstdc++ testsuite to handle VxWorks gthreads implementation

2020-08-21 Thread Alexandre Oliva
On Dec 20, 2019, Jonathan Wakely wrote: > On 10/12/19 15:58 +0100, Corentin Gay wrote: >> This patch was tested on x86_64-linux and is part of our nightly testing >> on all platforms, including VxWorks. > Was it tested on AIX? > I think dg-require-gthreads will prevent the tests running for the

Re: [PATCH] Fix libstdc++ testsuite to handle VxWorks gthreads implementation

2020-08-24 Thread Alexandre Oliva
default? Anyway, thanks for fixing the failure. That said, I confess I don't get how this test will (or should) ever be run in C++14 mode, being given an explicit -std=gnu++11. Is this really the right way to go about it? -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: [PATCH 1/5] Don't enable -gvariable-location-views by default for DWARF5.

2020-08-24 Thread Alexandre Oliva
tputting locviews is no use if there are no consumers for it, more so when they actively disturb existing standard-compliant consumers. -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-24 Thread Alexandre Oliva
o get them zeroed in the exception path, and requests a target hook to emit the insns to zero them in the regular return path. -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: [PATCH] introduce attribute exalias

2020-08-25 Thread Alexandre Oliva
rgument to be number 2: int attribute((alias("Y", 2))) X; reading as imperative "alias Y to X", whereas existing uses: int attribute((alias("Y"))) Z; read as imperative "alias Y Z", nonsense, but taken as "alias Z to Y", perhaps by analogy with how "give Y Z" is taken as "give Z to Y". /me jokingly suggests inhalias and exhalias ;-) This is enough bikeshedding for me ;-) -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: [PATCH] introduce attribute exalias

2020-08-25 Thread Alexandre Oliva
On Aug 15, 2020, Iain Sandoe wrote: > Alexandre Oliva wrote: >> I'm pretty sure setting multiple labels at the same address is used >> and relied on quite often. > That’s what’s currently disallowed (the assemblers all support .set). I understand you mean it's

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-26 Thread Alexandre Oliva
On Aug 25, 2020, Jeff Law wrote: > On Tue, 2020-08-25 at 02:16 -0300, Alexandre Oliva wrote: >> On Aug 24, 2020, Richard Biener wrote: >> >> > since the option is quite elaborate on what (sub-)set of regs is >> > supposed to be cleared I'm not sure an

Re: [PATCH 1/5] Don't enable -gvariable-location-views by default for DWARF5.

2020-08-26 Thread Alexandre Oliva
Hello, Mark, On Aug 25, 2020, Mark Wielaard wrote: > On Tue, 2020-08-25 at 01:05 -0300, Alexandre Oliva wrote: >> it would seem to >> make more sense to adopt and promote the proposed extension, >> implemented in =incompat5 in GCC, but it would need some >> implemen

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-08-28 Thread Alexandre Oliva
hy shouldn't that work for them as well? -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

[RFC] enable flags-unchanging asms, add_overflow/expand/combine woes

2020-09-01 Thread Alexandre Oliva
5807d..56e086a 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -21021,6 +21021,12 @@ ix86_md_asm_adjust (vec &outputs, vec &/*inputs*/, switch (con[0]) { + case 'C': + saw_asm_flag = true; + constraints[i] = "=X"; + o

Re: [RFC] enable flags-unchanging asms, add_overflow/expand/combine woes

2020-09-03 Thread Alexandre Oliva
On Sep 2, 2020, Segher Boessenkool wrote: > Hi! > On Tue, Sep 01, 2020 at 07:22:57PM -0300, Alexandre Oliva wrote: >> It's ugly, I know. > Yeah, it's bloody disgusting :-) :-) > (And aarch, but not the other targets with default clobbers). And arm. I didn't

Re: [RFC] enable flags-unchanging asms, add_overflow/expand/combine woes

2020-09-03 Thread Alexandre Oliva
[dropping port maintainers for combine-mostly subthread] On Sep 3, 2020, Segher Boessenkool wrote: > On Thu, Sep 03, 2020 at 07:03:53AM -0300, Alexandre Oliva wrote: >> On Sep 2, 2020, Segher Boessenkool wrote: >> >> we might succeed, but only if we had a patte

Re: [RFC] enable flags-unchanging asms, add_overflow/expand/combine woes

2020-09-03 Thread Alexandre Oliva
On Sep 3, 2020, Segher Boessenkool wrote: > On Thu, Sep 03, 2020 at 07:03:53AM -0300, Alexandre Oliva wrote: >> On Sep 2, 2020, Segher Boessenkool wrote: >> > (And aarch, but not the other targets with default clobbers). >> >> And arm. I didn't look very

Re: [RFC] enable flags-unchanging asms, add_overflow/expand/combine woes

2020-09-03 Thread Alexandre Oliva
On Sep 3, 2020, Segher Boessenkool wrote: > On Thu, Sep 03, 2020 at 02:07:24PM -0300, Alexandre Oliva wrote: >> On Sep 3, 2020, Segher Boessenkool wrote: >> > On Thu, Sep 03, 2020 at 07:03:53AM -0300, Alexandre Oliva wrote: >> >> On Sep 2, 2020, Segher Boessen

Re: [RFC] enable flags-unchanging asms, add_overflow/expand/combine woes

2020-09-03 Thread Alexandre Oliva
On Sep 3, 2020, Segher Boessenkool wrote: > On Thu, Sep 03, 2020 at 02:17:04PM -0300, Alexandre Oliva wrote: >> On Sep 3, 2020, Segher Boessenkool wrote: >> > Oh, I meant what was in your patch. From a grep, all of mn10300, nds32, >> > arm, cris, pdp11, rs6000,

Re: [RFC] enable flags-unchanging asms, add_overflow/expand/combine woes

2020-09-07 Thread Alexandre Oliva
e i3 parallel, which is happens to already be the most likely order for recog. In both cases, the likelihood follows from the fact that i1 and i2 were split, in this order, out of the original i2's parallel. -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: [PATCH] ppc64 check for incompatible setting of minimal-toc

2020-09-11 Thread Alexandre Oliva
variants thereof as the -mcmodel option got adopted by other systems. Thanks in advance, -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

Re: [PATCH] ppc64 check for incompatible setting of minimal-toc

2020-09-11 Thread Alexandre Oliva
it fine, but doesn't. I guess this means we withdraw the patch. Thanks again! -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer

[fixinc] Allow CONFIG_SHELL to override build-time shell in mkheaders

2020-02-20 Thread Alexandre Oliva
MACRO_LIST="${itoolsdatadir}/macro_list" \ - @SHELL@ ./fixinc.sh ${subincdir} \ + $shell ./fixinc.sh ${subincdir} \ ${isysroot}${SYSTEM_HEADER_DIR} ${OTHER_FIXINCLUDES_DIRS} rm -f ${subincdir}/syslimits.h if [ -f ${subincdir}/limits

work-around system header namespace pollution

2020-02-25 Thread Alexandre Oliva
; -- Alexandre Oliva, freedom fighterhe/himhttps://FSFLA.org/blogs/lxo/ Free Software Evangelist Stallman was right, but he's left :( GNU Toolchain Engineer The darkest places in hell are reserved for those who maintain their neutrality in times of moral crisis. -- Dante Alighieri

define NO_DOT_IN_LABEL only in vxworks6

2020-02-25 Thread Alexandre Oliva
e does. */ #define TARGET_HAVE_CTORS_DTORS false -- Alexandre Oliva, freedom fighterhe/himhttps://FSFLA.org/blogs/lxo/ Free Software Evangelist Stallman was right, but he's left :( GNU Toolchain Engineer The darkest places in hell are reserved for those who maintain their neu

collect2.exe errors not pruned

2020-02-27 Thread Alexandre Oliva
uot;(^|\n)collect: re(compiling|linking)\[^\n\]*" $text "" text regsub -all "(^|\n)Please submit.*instructions\[^\n\]*" $text "" text regsub -all "(^|\n)\[0-9\]\[0-9\]* errors\." $text "" text -- Alexandre Oliva, freedom fighterhe/him

<    1   2   3   4   5   6   7   8   9   10   >