environments (linux, mingwn, cygwin)
Comments ? does this is seems reasonable to push to trunk ?
Cheers
Christian
2012-12-21 Christian Bruel
* configure.ac: Set target_header_dir for with-build-sysroot.
* configure
sroot."
looks incorrect to me as we seem to have here a use of
--with-build-sysroot without --with-sysroot.
Not sure if it's clear, but I'm wondering why this restriction in the
documentation ? Could we amend it ?
Cheers
Christian
On 08/29/2013 10:36 AM, Christian Bruel wrote:
&g
On 08/30/2013 05:50 PM, Joseph S. Myers wrote:
> On Fri, 30 Aug 2013, Christian Bruel wrote:
>
>> So to cross build a target library |
>> --with-build-sysroot=|dir looks appropriate to specify the alternative
>> host root path.
>> but
>> --with-sysroot looks no
Christian
2013-09-13 Christian Bruel
PR target/58314
* config/sh/sh.md (mov_reg_reg): Allow memory for reload.
2013-09-13 Christian Bruel
PR target/58314
* gcc.target/sh/pr58314.c: New test.
Index: gcc/config/sh/sh.md
For 4.8 and 4.9
2013-09-13 Christian Bruel
PR target/58314
* config/sh/sh.md (mov_reg_reg): Allow memory reloads.
2013-09-13 Christian Bruel
PR target/58314
* gcc.target/sh/torture/pr58314.c: New test.
Index: gcc/config/sh/sh.md
Hi Richard,
On 09/16/2013 07:10 PM, Richard Sandiford wrote:
> Hi Christian,
>
> Christian Bruel writes:
>> @@ -6893,11 +6894,14 @@ label:
>> ;; reloading MAC subregs otherwise. For that probably special patterns
>> ;; would be required.
>> (defin
Hi Oleg,
On 09/18/2013 02:59 PM, Oleg Endo wrote:
> On Wed, 2013-09-18 at 09:55 +0200, Christian Bruel wrote:
>> Hi Richard,
>>
>> On 09/16/2013 07:10 PM, Richard Sandiford wrote:
>>> Hi Christian,
>>>
>>> Christian Bruel writes:
>>>> @
Hi Kaz, Oleg,
On 09/19/2013 01:15 AM, Kaz Kojima wrote:
> Christian Bruel wrote:
>> && (!can_create_pseudo_p () && REG_P (operands[0]) && REG_P (operands[1]))"
>>
>> is necessary ?
> It looks an another hack to allow the 2nd and 3rd alternat
large number of benchmarks (CSIBE, EEMBC,
Coremark, ...)
sh4-linux-elf survived a full Linux distribution rebuild
OK for trunk?
many thanks,
Christian
2013-09-19 Christian Bruel
PR target/58475
* config/sh/sh.md (movsf_ie): Allow fpul_operand.
* config/sh/predicate.md (arith_reg_operand
On 09/20/2013 01:07 AM, Kaz Kojima wrote:
> Christian Bruel wrote:
>> This patch fixes the aforementioned PR by refusing FPUL_REG to be an
>> acceptable reg for any arithmetic_operand on TARGET_SH4. (This was a
>> strange SH4 singularity with regards to the SH family).
&g
Hi Oleg,
+/*
+This pass tries to optimize for example this:
+ mov.l @(4,r4),r1
+ tst r1,r1
+ movtr1
+ tst r1,r1
+ bt/s.L5
+
+into something simpler:
+ mov.l @(4,r4),r1
+ tst r1,r1
+ bf/s.L5
+
+Such sequences can be identif
Hello,
This patch just reorganizes the SH code used for memory builtins into
its own file, in preparation of the RTL strcmp hoisting in the next part.
OK for trunk ?
Thanks
Christian
2013-10-17 Christian Bruel
* config.gcc (sh-*): Add sh-mem.o to extra_obj.
* gcc/config/sh/t-sh (sh
with -fno-builtin-strcmp.
No regressions on sh4 in big and little endian, and sh2 (sh3, and sh4a
are still running for big and little endian for sanity)
OK for trunk
Thanks
Christian
2013-10-17 Christian Bruel
* gcc/config/sh/sh-mem.c (sh4_expand_cmpstr): New function.
* gcc/config/sh/s
On 10/18/2013 12:53 AM, Oleg Endo wrote:
> Hi,
>
> On Thu, 2013-10-17 at 16:13 +0200, Christian Bruel wrote:
>> Hello,
>>
>> This patch just reorganizes the SH code used for memory builtins into
>> its own file, in preparation of the RTL strcmp hoisting in the
On 10/18/2013 01:05 AM, Oleg Endo wrote:
> I was wondering, in file sh-mem.c, the new function
> 'sh4_expand_cmpstr' ... why is it SH4-something? It's a bit confusing,
> since cmp/str has been around since ever (i.e. since SH1). Maybe just
> rename it to 'sh_expand_cmpstr' instead?
Just historica
Hi Oleg,
On 10/19/2013 11:30 AM, Oleg Endo wrote:
>
>
> I've attached two test cases, tested with
> make -k check-gcc RUNTESTFLAGS="sh.exp=strcmp* --target_board=sh-sim
> \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}"
>
> Could you please include them?
>
> Cheers,
> Oleg
>
>
than
No regressions for -m2 and -m4 for sh-elf.
OK for trunk ?
Many thanks,
Christian
2013-10-27 Christian Bruel
* gcc/config/sh/sh-mem.cc (sh_expand_cmpnstr): Moved here.
(sh_expand_cmpstr): Handle known align and schedule improvements.
* gcc/config/sh/sh-protos.h (sh_expand_cmpstrn): Decl
In the ChangeLog, the entry
* gcc/config/sh/sh-mem.cc (sh_expand_cmpnstr): Moved here.
is instead
* gcc/config/sh/sh-mem.cc (sh_expand_cmpnstr): New function.
Sorry for this,
Christian
thanks
Christian
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 204346)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2013-11-05 Christian Bruel
+
+ * gcc/config/sh/sh-mem.cc (sh_expand_cmpnstr, sh_expand_cmpstr
nicely combines sequences like !strncmp(*av,
"any", strlen(*av))
No regressions for sh-none-elf. OK for trunk ?
Many thanks
Christian
2013-11-05 Christian Bruel
* gcc/config/sh/sh-mem.cc (sh_expand_strlen): New function.
* gcc/config/sh/sh-protos.h (sh_expand_strlen): Declare.
*
some predicate tests (e.g arm_neonv2_ok) in the
testsuite was returning the wrong value, thus marking some test as
UNRESOLVED instead of PASS. See the reduced case of the issue attached
is the patch.
Regtested, a few new PASS for -mfpu=neon-fp-armv8
2016-02-17 Christian Bruel
* config/arm
for trunk ?
thanks
2014-01-09 Christian Bruel
* gcc/config/sh/sh-mem.cc (sh_expand_cmpnstr): Unroll small sizes and
optimized non constant lengths.
2014-01-09 Christian Bruel
* gcc.target/sh/cmpstrn.c: New case.
Index: gcc/config/sh/sh-mem
On 11/05/2013 02:12 PM, Kaz Kojima wrote:
> Christian Bruel wrote:
>> No regressions for sh-none-elf. OK for trunk ?
> OK.
>
> Regards,
> kaz
thanks, applied together with the cleanup referenced earlier and a
slight variable renaming (start_addr->curr_addr, e
On 11/06/2013 11:57 AM, Eric Botcazou wrote:
>> thanks, applied together with the cleanup referenced earlier and a
>> slight variable renaming (start_addr->curr_addr, end_addr->start_addr)
>> for readability as obvious,
> Minor nit: no gcc/ prefix in gcc/ChangeLog.
ok I'll didn't notice this., do
ning different modes might not be
mandatory (or even counter-productive) at this stage ?
Best Regards
Christian
On 10/08/2014 03:05 PM, Ramana Radhakrishnan wrote:
> Hi Christian,
>
> Thanks for looking at this. I will need to read the code in detail but
> this is a first top l
On 10/08/2014 06:56 PM, Ramana Radhakrishnan wrote:
> Hi Christian,
<< snipped agreed stuf >>
> 3) about inlining
>I dislike inlining different modes, From a conceptual use, a user
> might want to switch mode only when changing a function's hotness.
> Usually inlining a cold function into a h
On 10/09/2014 04:11 PM, Richard Earnshaw wrote:
> On 09/10/14 12:35, Christian Bruel wrote:
>> On 10/08/2014 06:56 PM, Ramana Radhakrishnan wrote:
>>> Hi Christian,
>> << snipped agreed stuf >>
>>> 3) about inlining
>>>I dislike i
re I can send the remaining ARM parts.
Tested manually for arm-none-eabi with gdb, unwinding and frame access
seem OK when mixing modes.
x86 bootstrapped and regressions tests are running.
Many thanks,
Christian
2014-09-23 Christian Bruel
* execute_dwarf2_frame (dw_frame_pointer_regnum):
Hello,
Any feedback for this ? I'd like to commit only when OK for Epiphany.
many thanks,
Christian
On 05/26/2014 05:32 PM, Christian Bruel wrote:
>> On 04/28/2014 10:08 AM, Christian Bruel wrote:
>>>>>> Hello,
>>>>>>
>>>>>> I
s a cost to the phi defs constants to
allow the whole test to be hoisted out of the loop.
Richard, does this seem reasonable and OK for 4.10 ?
bootstrapped/regtested for x86
many thanks
Christian
2014-06-03 Christian Bruel
PR tree-optimization/43934
* tree-ssa-loop-i
On 06/11/2014 06:17 AM, Joern Rennecke wrote:
>
>>> Joern, is this new target macro interface OK with you ?
> Yes, this interface should allow me to do switches between rounding
> and truncating
> floating-point modes with an add/subtract immediate.
>
> However, the implentation, as posted, doesn'
On 06/10/2014 04:03 PM, Joern Rennecke wrote:
> On 13 May 2014 22:41, Oleg Endo wrote:
>
>> Right. I was thinking to add FPSCR.SZ mode switching to SH, in order to
>> do float vector moves. For that SZ and PR need to be switched both at
>> the same time (only SH4A has both, fpchg and fschg). S
On 06/11/2014 02:00 PM, Christian Bruel wrote:
> On 06/11/2014 06:17 AM, Joern Rennecke wrote:
>>>> Joern, is this new target macro interface OK with you ?
>> Yes, this interface should allow me to do switches between rounding
>> and truncating
>> floating
Hello,
I still miss an approval for the middle-end part of
http://gcc.gnu.org/ml/gcc-patches/2014-06/msg01038.html
thanks,
Christian
On 01/22/2015 11:33 AM, Richard Biener wrote:
> On Thu, 22 Jan 2015, Christian Bruel wrote:
>
>> Hi Richard,
>>
>> I thought one of my current issue would be solved by this patch, but it is
>> not
>> : I have some inlining failures with the attribute targ
ny thanks,
Christian
2014-09-23 Christian Bruel
* config/arm/arm.h (arm_option_override): Reoganized and split.
(arm_option_params_internal); New function.
(arm_option_check_internal): New function.
(arm_option_override_internal): New function.
(restrict_default): New boolean.
(thumb_code,
Hi,
I'd like to re-ping the following patch for GCC 5.2. It fixes the
__attribute__ ((__optimize__ (...))) on x86. Testcase is in the patch.
thanks
Christian
On 04/13/2015 04:24 PM, Christian Bruel wrote:
> https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00243.html
>
> thanks,
>
> Christian
>
On 04/30/2015 09:43 AM, Ramana Radhakrishnan wrote:
> On Mon, Apr 20, 2015 at 9:35 AM, Christian Bruel
> wrote:
>> Hello Ramana
>>
>>>>
>>>
>>> Can you respin this now that we are in stage1 again ?
>>>
>>> Ramana
>>>
&
OK I've have a look,
thanks
Christian
On 04/30/2015 10:27 AM, Bin.Cheng wrote:
> On Mon, Apr 27, 2015 at 8:01 PM, Uros Bizjak wrote:
>> On Wed, Feb 4, 2015 at 2:21 PM, Christian Bruel
>> wrote:
>>> While trying to reduce the PR64835 case for ARM and x86, I n
> Hi Christian,
> I noticed case gcc.dg/ipa/iinline-attr.c failed on aarch64. The
> original patch is x86 specific, while the case is added as general
> one. Could you please have a look at this?
>
> FAIL: gcc.dg/ipa/iinline-attr.c scan-ipa-dump inline
> "hooray[^\\n]*inline copy in test"
>
t
thanks for the validation and the confirmation that iinline-attr.c is
now fixed on aarch64. I can now send the patch for submission request
(this one was just illustrative).
thanks
Christian
On 05/05/2015 11:10 AM, Yvan Roux wrote:
> Hi Christian,
>
> On 4 May 2015 at 11:29, Christ
This fixes PR target/66015 and a latent issue revealed by
gcc.dg/ipa/iinline-attr.c since
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01609.html
Regtested on aarch64-linux-gnu by Linaro.
OK for trunk ?
Many thanks
Christian
2015-05-05 Christian Bruel
PR target/66015
* config/aarch64
issue ?
many thanks
Christian
2015-05-05 Christian Bruel
PR target/66015
* config/ia64/ia64.c (ia64_option_override): Move align_loops,
and align_functions into ia64_override_options_after_change.
2015-05-05 Christian Bruel
PR target/66015
* gcc.target/ia64/iinline-attr-1.c: New
On 05/05/2015 02:42 PM, Marcus Shawcroft wrote:
> On 5 May 2015 at 12:07, Christian Bruel wrote:
>> This fixes PR target/66015 and a latent issue revealed by
>> gcc.dg/ipa/iinline-attr.c since
>> https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01609.html
>>
>>
> A general note, please reply to each of the patches with a rebased
> patch as a separate email. Further more all your patches appear to
> have dos line endings so they don't seem to apply cleanly. Please
> don't have spurious headers in your patch submission - it then makes
> it hard to , please
-09-23 Christian Bruel
* config/arm/arm.h (arm_option_override): Reoganized and split.
(arm_option_params_internal); New function.
(arm_option_check_internal): New function.
(arm_option_override_internal): New function.
(restrict_default): New boolean.
(thumb_code, thumb1_code): Remove
In preparation of the pragma target
reorganize ÂTARGET_CPU_CPP_BUILTINSÂ to redefine mode dependent macros
based on current thumb_p.
Thanks,
Christian
2014-09-23 Christian Bruel
* config/arm/arm-c.c (cpp_def_or_undef): New functions.
(arm_cpp_builtins): Likewise.
* config/arm/arm.h
Re-implement ARM_DECLARE_FUNCTION_NAME as a function. That will make
changed related to unified/divided and mode directives easier to insert.
Thanks
Christian
2014-09-23 Christian Bruel
* config/arm/arm-protos.h (arm_declare_function_name): Declare.
(is_called_in_ARM_mode): Remove
tps://sourceware.org/bugzilla/show_bug.cgi?id=17505)
Regtests included
Thanks
Christian
2014-09-23 Christian Bruel
* config/arm/arm.opt (THUMB, arm_restrict_it, inline_asm_unified): Save.
* config/arm/arm.h (arm_valid_target_attribute_tree): Declare.
(arm_reset_previous_fndecl, arm_change_mod
Implements the hooks for #pragma GCC target
A test included to check that macros were correctly defined/undefined on
pragma regions.
Thanks
Christian
2014-09-23 Christian Bruel
* config/arm/arm.h (REGISTER_TARGET_PRAGMAS):
Call arm_register_target_pragmas.
* config/arm/arm-protos.h
Implement the -mflip-thump option. Undocumented for internal testing
only. This option artificially inserts alternative attribute thumb/modes
on functions.
This close the patch set. Thanks for your review,
Christian
2014-09-23 Christian Bruel
* config/arm/arm.c (add_attribute
+ Sandra's doc review fixes.
tested with make doc pdf
thanks
Christian
On 05/06/2015 04:24 PM, Christian Bruel wrote:
> Implements and document the hooks to support target_attributes.
>
> The emission of blx is handled directly for armv5 to overcome a bug with
> the curr
Highlights
---
- Beta Breakpad delived
- Upstreaming (almost accepted)
Issues
---
- Current FSF trunk boostrap broken (delay validations)
Completed
--
- Fixed FSF attribute optimize inlining
- Fixed FSF bad handling of ipa-cp (inlining of functions with constant
propagat
please ignore
On 05/07/2015 10:43 AM, Christian Bruel wrote:
>
> Highlights
> ---
> - Beta Breakpad delived
> - Upstreaming (almost accepted)
>
> Issues
> ---
> - Current FSF trunk boostrap broken (delay validations)
>
> Completed
>
On 05/07/2015 10:49 AM, Ramana Radhakrishnan wrote:
>
>
> On 06/05/15 15:20, Christian Bruel wrote:
>> In preparation of the pragma target
>>
>> reorganize ÂTARGET_CPU_CPP_BUILTINSÂ to redefine mode dependent macros
>> based on current thumb_p.
>
> I
binunvnEzkM1k.bin
Description: PGP/MIME version identification
encrypted.asc
Description: OpenPGP encrypted message
OK with those changes.
Ramana
thanks, done
following up the thumb_code cleanup, here is a missing chunk for the
vxworks config.
arm-vxworks build checked. ok for trunk ?
thanks,
Christian
2015-05-11 Christian Bruel
* config/arm/arm-protos.h (thumb_code, thumb1_code): Remove
here the p1 patch committed at rev 222995
On 05/11/2015 11:56 AM, Ramana Radhakrishnan wrote:
On Mon, May 11, 2015 at 10:13 AM, Christian Bruel
wrote:
OK with those changes.
Ramana
thanks, done
following up the thumb_code cleanup, here is a missing chunk for the vxworks
config.
arm
Hello Ramana,
Here is the attribute revisited after your comments, so
- thumb1 is now supported
- -mflip-thump option added for testing.
- inlining is allowed between modes.
This set of patches was tested on rev#217709 as:
no regressions with:
arm-sim/
arm-sim/-march=armv7-a
arm
2014-09-23 Christian Bruel
* config/arm/arm.h (arm_option_override): Reoganized and split.
(arm_option_params_internal); New function.
(arm_option_check_internal): New function.
(arm_option_override_internal): New function.
(restrict_default): New boolean.
(thumb_code, thumb1_code): Remove
In preparation of the pragma target
reorganize ´TARGET_CPU_CPP_BUILTINS´ to redefine mode dependent macros
based on current thumb_p.
Thanks,
Christian
2014-09-23 Christian Bruel
* config/arm/arm-c.c (cpp_def_or_undef): New functions.
(arm_cpp_builtins): Likewise.
* config/arm/arm.h
Re-implement ARM_DECLARE_FUNCTION_NAME as a function. That will make
changed related to unified/divided and mode directives easier to insert.
Thanks
Christian
2014-09-23 Christian Bruel
* config/arm/arm-protos.h (arm_declare_function_name): Declare.
(is_called_in_ARM_mode): Remove
tps://sourceware.org/bugzilla/show_bug.cgi?id=17505)
Regtests included
Thanks
Christian
2014-09-23 Christian Bruel
* config/arm/arm.opt (mthumb): Save.
* config/arm/arm.h (arm_valid_target_attribute_tree): Declare.
(arm_reset_previous_fndecl, arm_change_mode_p): Likewise.
(SWITCHABLE_TARGET): Def
Implements the hooks for #pragma GCC target
A test included to check that macros were correctly defined/undefined on
pragma regions.
Thanks
Christian
2014-09-23 Christian Bruel
* config/arm/arm.h (REGISTER_TARGET_PRAGMAS):
Call arm_register_target_pragmas.
* config/arm/arm-protos.h
Implement the -mflip-thump option. Undocumented for internal testing
only. This option artificially inserts alternative attribute thumb/modes
on functions.
This close the patch set. Thanks for your review,
Christian
2014-09-23 Christian Bruel
* config/arm/arm.c (add_attribute
On 11/19/2014 03:18 PM, Ramana Radhakrishnan wrote:
On Wed, Nov 19, 2014 at 1:24 PM, Christian Bruel wrote:
I think I missed the stage3, Anyway would it be OK for stage1 when it
reopens ?
Since you submitted this well during stage1 and given that these
patches address comments from
Hi Ramana,
On 11/27/2014 11:36 AM, Ramana Radhakrishnan wrote:
On Wed, Nov 19, 2014 at 2:54 PM, Christian Bruel wrote:
On 11/19/2014 03:18 PM, Ramana Radhakrishnan wrote:
On Wed, Nov 19, 2014 at 1:24 PM, Christian Bruel
wrote:
I think I missed the stage3, Anyway would it be OK for
https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00243.html
thanks,
Christian
committed, thanks
sorry for the delay.
Christian
On 10/14/2014 08:25 PM, Richard Henderson wrote:
> On 10/14/2014 06:02 AM, Christian Bruel wrote:
>> 2014-09-23 Christian Bruel
>>
>> * execute_dwarf2_frame (dw_frame_pointer_regnum): Reinitialize for each
>> f
Hi Richard,
I thought one of my current issue would be solved by this patch, but it
is not : I have some inlining failures with the attribute target on ARM.
(e.g inline-3.c) where obvious early inline fails with because we fail
into the last can_inline_edge_p case:
opt_for_fn (callee->decl,
nd x86
bootstrapped, regtested with no new failures for x86_64-unknown-linux-gnu
Comments ? I'd like to candidate this for trunk when stage1 opens again.
Many Thanks
Christian
2015-02-06 Christian Bruel
PR target/64835
* config/i386/i386.c (ix86_default_align): N
pdate in the ARM back-end :-(
Comments, questions are welcome,
Many thanks
Christian
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 220436)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,11 @@
+2015-02-06 Christian Bruel
+
In order to fix the various conflicts that have happened since, please
find attached the re-based patches to trunk rev #220529 (respectively
from above p0.patch, p1.patch, p2,patch, p3.patch, p4,patch, p5,patch,
p6,patch).
oops, please don't review p0.patch here. This last one will be reviewe
hi,
the ARM bootstrap seems to fail for libgcc2.c on the thumb multilib for
libgcc2: muldi3 -mthumb -O2 -g
/tmp/ccYrycUw.s: Assembler messages:
/tmp/ccYrycUw.s:69: Error: MOV Rd, Rs with two low registers is not
permitted on this architecture -- `mov r6,r7'
preprocessed attached.
Thanks
sions-_0028ACLE_0029.html#ARM-C-Language-Extensions-_0028ACLE_0029)
already as a visionary description of the attribute. I might have missed
something but this time the doc comes earlier than the implementation
:-) without the historical background.
Best Regards
Christian
2014-12-14 Christian
Hello,
We should not enter the first iteration when length is 0. Testcase
attached. Difficult to reduce because register allocation generated
accidentally the correct return value.
testsuite OK
OK for 4.9 and trunk ?
Christian
2015-01-08 Christian Bruel
PR target/64507
* config/sh
Please use 'gen_cmpeqsi_t (len, const0_rtx)' for comparing a value
against zero instead of the bit test insn.
OK, also then OK to replace the other occurrences of the idiom for
coding consistency ? (not sure if I could commit this as obvious ?).
Cheers
Christian
2015-01-08
n't noticed any difference on x86.
This also solves a few "Invalid sum of incoming frequencies" messages
while dumping the CFG
Reg-tested on x85 and sh-superh-elf. Is it OK for the 4.7 and 4.8 branches ?
Thanks
Christian
2012-11-07 Christian Bruel
* tree-ssa-tail-merge.c (replace
> OK,
> is bb1 going to die? If not, probably bb1->count = 0 should be there, if so,
> then the bb1->frequency = 0 is redundant.
Agree, we do 'delete_basic_block (bb1)' and the frequency is not used in
between, so the setting to 0 seems unnecessary.
testing it:
Index: tree-ssa-tail-merge.c
===
Hi Kaz,
Now that the cross-jumping problem is fixed since rev #193350, I'd like
to remove this restriction and close PR/54546.
Checked with default sh-sim target_board and
--target_board=sh-sim/-freorder-blocks-and-partition.
Thanks
Christian
2012-11-09 Christian Bruel
* config/sh
, feedback from the respective arch maintainers
would be appreciated as I don't run the gdb testsuite on those targets.
Many thanks,
Christian
2013-04-26 Christian Bruel
* dwarf2out.c (multiple_reg_loc_descriptor): Use DBX_REGISTER_NUMBER
for spaning registers.
2013-04-26
Hello,
This patches set the correct operand mode for tstsi_t_zero_extract_eq,
to avoid reload generating a move between a constant and a void register.
Reg tested for sh-elf. No performance impact
OK for 4.7, 4.8 and trunk ?
Thanks
2013-04-26 Christian Bruel
PR target/57108
* sh.md
leaf_regs is not handled for the spanning
case.
Does that seem OK with the attached patch ?
Thanks
Christian
> -cary
2013-04-26 Christian Bruel
* dwarf2out.c (multiple_reg_loc_descriptor): Use DBX_REGISTER_NUMBER
for span
Yes, this looks good. OK for trunk, but please add a note about those
additional changes you made to the ChangeLog entry. Thanks!
Thanks, done with this entry:
2013-05-21 Christian Bruel
* dwarf2out.c (multiple_reg_loc_descriptor): Use dbx_reg_number for
spanning
, so the DBX conversion
information doesn't need to be duplicated.
Build and regtested with a arm-none-eabi newlib toolset configured with
--with-fpu=neon-vfpv4 --with-float=hard --with-arch=armv7-a
OK for trunk ?
Thanks
Christian
2013-05-22 Christian Bruel
PR debug/57351
* config/arm/
h64-unknown-elf (The original sh64-elf build failure assertion in
dwarf2cfi is fixed.)
- arm-none-eabi -with-fpu=neon-vfpv4
- powerpc-e500v2-linux-gnuspe
- x86_64-unknown-linux-gnu sanity build OK
Is dwarf-span-target-dbx.patch OK for trunk ?. More comments ?
Many Thanks,
Christian
Hello,
May I have a review from the DWARF, the ARM and RS6000 maintainers for
comments/approval ?
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg01613.html
Needed to fix the powerpc-spe bootstrap referenced in bugzilla #57389
Many Thanks
Christian
Christian Bruel wrote:
>> However I
needed, I think)
Tested for no regression for arm-none-eabi [,-with-arch=armv7-a]
OK for trunk ?
many thanks,
Christian
2014-09-23 Christian Bruel
PR target/52144
* config/arm/arm.opt (THUMB): Sqve target option.
* config/arm/arm-protos.h (arm_declare_function_name,
Hi Ramana,
Your patch https://gcc.gnu.org/ml/gcc-patches/2012-02/msg01492.html
seems to have not been applied for 4.10. Are there any stoppers or is it
an omission ?
Many Thanks
Christian
direct
access to the Linaro archives) ?
Thanks a lot,
Christian
On 10/01/2014 12:43 PM, Ramana Radhakrishnan wrote:
> On Wed, Oct 1, 2014 at 10:03 AM, Christian Bruel
> wrote:
>> Hi Ramana,
>>
>> Your patch https://gcc.gnu.org/ml/gcc-patches/2012-02/msg01492.html
>> seem
by allowing
the sh-elf build to complete.
Thanks,
Christian
2013-04-09 Christian Bruel
* config/sh/sh.md (barrier_align): Use next/prev_active_insn instead
of next/prev_real_insn.
Index: gcc/config/sh/sh.c
===
--- gcc/config
2013-04-18 Christian Bruel
PR target/56995
* gcc.target/sh/mfmovd.c: Add new function and check hard_float.
2013-04-18 Christian Bruel
PR target/56995
* config/sh/sh.h (enum reg_class): Remove DF_HI_REGS.
(REG_CLASS_NAMES): Idem.
(REG_CLASS_CONTENTS): Idem.
(REGCLASS_HAS_FP_REG): Idem.
Hello,
I'd like to ping the following patches
[Hookize mode-switching]
http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01003.html
[Add new hooks to support toggle and SH4A fpchg instruction]
http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01005.html
Many thanks
dual mode).
Patch2 only does very little restructuring but if is not interesting
enough for all targets, patch 1 should not be that intrusive.
For RTL middle end and (X86, SH, Epiphany) target reviewers,
Many thanks,
Christian
On 04/28/2014 10:08 AM, Christian Bruel wrote:
> Hello,
>
&g
Just saw the Jeff's approval for the RTL part. Sorry for the crossed answers
remains the target maintainers. Joern, Kaz ?
Many thanks.
Christian
On 05/12/2014 10:44 AM, Christian Bruel wrote:
> Hello,
>
> I'd still wish to ping for the following set of patches. Those change
On 04/28/2014 10:08 AM, Christian Bruel wrote:
>>>> Hello,
>>>>
>>>> I'd like to ping the following patches
>>>>
>>>> [Hookize mode-switching]
>>>> http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01003.html
>>>>
2014-05-16 Christian Bruel
PR target/61195
* config/sh/sh.c (movsf_ie): Unset fp_mode for fmov.
2014-05-16 Christian Bruel
PR target/61195
* gcc.target/sh/pr61195.c: New test.
Index: config/sh/sh.md
===
--- config/sh
> On 04/28/2014 10:08 AM, Christian Bruel wrote:
>>>>> Hello,
>>>>>
>>>>> I'd like to ping the following patches
>>>>>
>>>>> [Hookize mode-switching]
>>>>> http://gcc.gnu.org/ml/gcc-patches/2014-04
testsuites., OK for trunk ?
thanks,
Christian
2014-01-23 Christian Bruel
* config/sh/sh-mem.cc (sh_expand_cmpnstr): Fix remaining bytes after
words comparisons.
2014-01-23 Christian Bruel
* gcc.target/sh/torture/strncmp.c: New tests.
Index: gcc/config/sh/sh-mem.cc
Hello,
Considering the attached trivial case with the epilogue:
sub sp, fp, #12
ldmia sp, {fp, sp, lr}frame_related_p
the sub instruction should also be frame_related_fp. (a gcc_assert
triggers in dwarf2out_frame_debug_adjust_cfa)
This patch sets RTX_FRAME_RELATED_P on
1 - 100 of 163 matches
Mail list logo