This patch restores the GCC default layout of HA, SA and DA mode.
The original fixed point support tried to be binary compatible with fixed-point
support already provided by some non-FSF ports, but that turned out to result
in wrong code in some situations.
Reason is that these ports adjusted the
Some more skips for tests that will fail with 16-bit int etc.
Ok for trunk?
Johann
PR testsuite/52641
* gcc.dg/unroll_5.c: Add dg-require-effective-target int32plus.
* gcc.dg/pr55150-1.c: Same.
* gcc.dg/torture/pr54894.c: Add size32plus.
Index: gcc.dg/unroll_5.c
=
This patchlet fixes missing STAMP command line tool that is needed if gnat is
build for target avr.
The patch is untested and as proposed in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55243#c5
Ok for trunk and 4.7?
Johann
PR ada/55243
* Makefile.in (TOOLS_FLAGS_TO_PASS_CROSS)
Better attach the patch...
Georg-Johann Lay wrote:
> This patchlet fixes missing STAMP command line tool that is needed if gnat is
> build for target avr.
>
> The patch is untested and as proposed in
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55243#c5
>
> Ok for tr
Here are some more fixes for 16-bit int and similar.
Ok for trunk?
Johann
* gcc.dg/c1x-align-4.c: Skip avr.
* gcc.dg/54455.c: Require scheduling.
* gcc.dg/pr44024.c: Skip avr in final scan.
PR testsuite/52641
* gcc.c-torture/execute/20120919-1.x: New file
Hans-Peter Nilsson wrote:
> On Fri, 23 Nov 2012, Georg-Johann Lay wrote:
>> Here are some more fixes for 16-bit int and similar.
>
>> * gcc.c-torture/execute/20120919-1.x: New file (int32plus).
>
> No, you should be able to use dg-directives in the main file these d
Ping #1 for:
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01914.html
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01915.html
Johann
PR ada/55243
* Makefile.in (TOOLS_FLAGS_TO_PASS_CROSS): Set STAMP.
Ondrej Bilka schrieb:
http://kam.mff.cuni.cz/~ondra/gcc_misspell.patch
This is wrong:
@@ -10834,7 +10834,7 @@ avr_convert_to_type (tree type, tree expr)
XOP[2] # Bytes to copy
Return TRUE if the expansion is accomplished.
- Return FALSE if the operand compination is not supported
The problem with this PR is that -size is used as addend in
avr_prologue_setup_frame instead of truncating it to the mode of the operation.
Fix as obvious.
Ok for trunk and backport?
Johann
PR target/57844
* config/avr/avr.c (avr_prologue_setup_frame): Trunk -size to mode
Several devices are duplicate in avr-mcus.def.
This patch removes the duplicate versions and adds a sanity check to
gen-avr-mmcu-texi.c.
Ok to apply?
Johann
PR target/57506
* config/avr/avr-mcus.def (atmega16hva, atmega16hva2, atmega16hvb)
(atmega16m1, atmega16u4, atmega
This change has been proposed so that the sanity checking of ISR
(signal,interrupt) functions tests the asm function name if available, not the
C name.
Okay to apply?
Johann
gcc/
PR target/57631
* config/avr/avr.c (avr_set_current_function): Sanity-check signal
name seen
Currently, the fixed-point rounding does not work correctly in the overflow
case. This is because of misreading section 2.1.7.2 of TR 18037.
Rounding builtins expand to saturated addition and AND so that the instruction
sequence is
add value1
if not overflow goto 0
load max value
0:
and value2
Hi,
running the cilk-plus.exp tests I get ~200 FAILs because the tests are not
written for 16-bit int or size_t platforms.
As a quick tentative fix, the cilk-plus tests are skipped on such platforms.
Common problems are:
- internal compiler error: in build_int_cst_wide, at tree.c:1214
- warnin
Here are some more skips of tests that won't work on int16 targets.
In most cases constants too big, bitfields too long or arrays too large are
used. For gcc.dg/torture/pr56488.c there are some notes at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56488#c4
that explain for why this test fails if
Joern Rennecke wrote:
> xfail for avr*-*-*
Hi,
the target machine is avr-*-* because, say, avr32 (a 32-bit MCU not officially
supported) is a quite different architecture than avr.
Similar for your other testsuite patches with avr*-*-*
Johann
http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=201121
Applied this obvious improvement of the sequence to load one byte from address
space memx (provided no ELPM is need).
The old sequence was
SBRC msb
LD reg, Z ; read from RAM
SBRS msp
LPM reg, Z ; read from flash
and the new one i
http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=201123
Applied this skip for int16 platforms that will obviously fail there.
Johann
PR testsuite/52641
* gcc.c-torture/execute/pr57124.x: Skip int16 platforms.
* gcc.c-torture/execute/pr53366-1.x: New: Skip int16 platf
http://gcc.gnu.org/r196332
This is obvious fix for problem with empty macro parameter.
Instead of an empty parameter, 'n' is used.
Johann
PR target/56445
* config/avr/avr.c (avr_init_builtins): Use 'n' instead of empty
macro parameters with: FX_FTYPE_FX, FX_FTYPE_FX_INT,
Joey Ye schrieb:
Ping
Subject: RE: [PATCH] Fix PR50293 - LTO plugin with space in path
Does this patch also work with MS-Windows as host, i.e. with \ as path
separator?
+static char * convert_white_space(char * orig);
Please fix formatting in many places in this patch to follow the GNU
C
http://gcc.gnu.org/r196428
Fixed this test case that assumed int is always 32 bits at least.
PR testsuite/52641
PR tree-optimization/52631
* gcc.dg/tree-ssa/pr52631.c: Fix 16-bit int.
Johann
Index: gcc.dg/tree-ssa/pr52631.c
http://gcc.gnu.org/r196429
gcc.dg/pr55153.c needs targets with scheduling, thus dg-require it.
gcc.dg/pr56228.c will fail on AVR, thus skip it there:
* gcc.dg/pr55153.c: Add dg-require-effective-target scheduling.
* gcc.dg/pr56228.c : Skip.
--
Index: gcc.dg/pr55153.c
===
Jakub Jelinek wrote:
> On Mon, Mar 04, 2013 at 12:15:31PM +0100, Georg-Johann Lay wrote:
>> http://gcc.gnu.org/r196428
>>
>> Fixed this test case that assumed int is always 32 bits at least.
>>
>>PR testsuite/52641
>>PR tree-optimization/5263
This patch fixed the speed of 64-bit shifts and rotate.
These operations were implemented by bit-wise shifts and thus the speed is not
reasonable for such basic arithmetic.
The new implementation first shifts byte-wise and only the remaining mod 8 is
shifted bit-wise.
The new methods needs few m
This patch implements a new warning option -Waddr-space-convert warns about
conversions to a non-containing address space.
Address spaces are implemented in such a way that each address space is
contained in each other space so that casting is possible, e.g. in code like
char read_c (bool in_flas
http://gcc.gnu.org/r196604
This adds a missing space in output_operand_lossage.
Applied as obvious.
Johann
PR target/56591
* config/avr/avr.c (avr_print_operand): Add space after '%c' in
output_operand_lossage message.
Index: config/avr/avr.c
==
Joerg Wunsch wrote:
The attached patch adds the new ATmega*RFR* devices to AVR-GCC.
[...]
Supply the auto generated files, too. Cf. t-avr, avr-mcus.def etc.
Johann
Some instructions like INC, DEC, NEG currently set cc0 to set_zn which is not
the whole story because they also set the V flag. This leads to a wrong branch
instruction in the remainder as cc0 is used. Fix is to do same as clobber cc0.
For the matter of clarity, I added a new cc0 alternative
Hi, I just applied the following backport to 4.9:
http://gcc.gnu.org/ml/gcc-cvs/2014-05/msg00310.html
Thanks.
2014-04-16 Pitchumani Sivanupandi
Backport from 2014-04-16 trunk r209446
2014-04-16 Pitchumani Sivanupandi
* config/avr/avr-mcus.def: Correct typo for
http://gcc.gnu.org/ml/gcc-cvs/2014-05/msg00311.html
Hi, applied this obvious patch to trunk in order to fix build warnings like
these:
gcc/config/avr/avr-fixed.md:434:59: warning: narrowing conversion of ‘-1’ from
‘int’ to ‘const unsigned int’ inside { } is ill-formed in C++11 [-Wnarrowing]
This patch adds GCC Runtime Library Exception to files that go into libgcc
because libgcc2.c includes tm.h and libgcc_tm.h.
Most of these files contain much code, some used by libgcc, some not. Some
potential users of (lib)gcc have objections that missing RLE might "infect"
their target code.
Am 05/10/2014 02:51 AM, schrieb Ian Lance Taylor:
Georg-Johann Lay writes:
This patch adds GCC Runtime Library Exception to files that go into
libgcc because libgcc2.c includes tm.h and libgcc_tm.h.
Most of these files contain much code, some used by libgcc, some
not. Some potential users of
Am 04/18/2014 11:52 AM, schrieb Senthil Kumar Selvaraj:
On Sat, Apr 12, 2014 at 06:36:01PM +0200, Georg-Johann Lay wrote:
Senthil Kumar Selvaraj schrieb:
This patch modifies AVR target's ASM spec to pass -mlink-relax to the
assembler if -mrelax is passed to the compiler driver. Thi
Am 05/13/2014 02:48 PM, schrieb Senthil Kumar Selvaraj:
On Mon, May 12, 2014 at 01:19:37PM +0200, Georg-Johann Lay wrote:
Am 04/18/2014 11:52 AM, schrieb Senthil Kumar Selvaraj:
On Sat, Apr 12, 2014 at 06:36:01PM +0200, Georg-Johann Lay wrote:
Senthil Kumar Selvaraj schrieb:
[...]
2014
https://gcc.gnu.org/r210418
Applied this patch in order to fixed build warnings like these
gcc/reginfo.c:115:5: warning: narrowing conversion of ‘-1073741824’ from ‘int’
to ‘const unsigned int’ inside { } is ill-formed in C++11 [-Wnarrowing]
gcc/reginfo.c:115:5: warning: narrowing conversion of
Am 05/15/2014 09:55 AM, schrieb Senthil Kumar Selvaraj:
On Wed, May 14, 2014 at 12:56:54PM +0200, Rainer Orth wrote:
Georg-Johann Lay writes:
Or what about simply that, which works for me:
Index: config/avr/avr.h
Backportet the fix for PR60991 also to 4.7 so that the issue can be closed:
http://gcc.gnu.org/r210635
Johann
gcc/
2014-05-20 Senthil Kumar Selvaraj
Backport from mainline r210325
2014-05-12 Senthil Kumar Selvaraj
PR target/60991
* config/avr/avr.c (avr_
:
Georg-Johann Lay writes:
This patch adds GCC Runtime Library Exception to files that go into
libgcc because libgcc2.c includes tm.h and libgcc_tm.h.
Most of these files contain much code, some used by libgcc, some
not. Some potential users of (lib)gcc have objections that missing RLE
might
This adds a note to the user manual that code with label differences is not
supported. This is because adding an offset to a stub address as generated
with gs() will in general not yield the address of the label+offset.
This actually occurs only if gs() has something to do, e.g. on devices wit
This adds runtime library exceptions for some more files that go into libgcc.c
via tm.h, mostly due to v850.
Ok for trunk?
Johann
gcc/
PR libgcc/61152
* config/dbx.h (License): Add Runtime Library Exception.
* config/newlib-stdint.h (License): Same.
* config/rt
Am 12/18/2013 12:37 PM, schrieb Richard Biener:
On Tue, Dec 17, 2013 at 2:05 PM, Georg-Johann Lay wrote:
Am 12/05/2013 04:09 PM, schrieb Richard Biener:
On Thu, Dec 5, 2013 at 3:53 PM, Georg-Johann Lay wrote:
This is a fix of a wrong warning for a bas ISR name. The assumption was
that if
S, Pitchumani schrieb:
Attached patch changes the device name macro of two rev devices, from REV
to rev in the name.
Device name macro:
__AVR_ATmega16HVBREVB__
__AVR_ATmega32HVBREVB__
Change to:
__AVR_ATmega16HVBrevB__
__AVR_ATmega32HVBrevB__
'rev' in small letters emphasizes the revision of t
Senthil Kumar Selvaraj schrieb:
The below patch adds a new preprocessor define for the device name
(__AVR_DEVICE_NAME__) that was passed to the compiler.
While the device name macro (say __AVR_ATmega128__) can be used to
check for a specific device, there is no way right now for code
to get th
This fixes a problem because cc_plus and cc_minus are in the wrong places in
calls of avr_out_plus_1. This is important when avr_out_plus is called from
notice_update_cc. This means that cc_status might be determined incorrectly.
In the vast majority of cases this leads to performance regress
This is again a request for approval to fix PR59396.
Problem is that the assembler name might or might not be prefixed by '*'
depending on when TARGET_SET_CURRENT_FUNCTION is called.
The change is just to fix wrong warning because the current implementation of
TARGET_SET_CURRENT_FUNCTION /alw
Am 03/13/2014 04:41 PM, schrieb Senthil Kumar Selvaraj:
On Thu, Mar 13, 2014 at 02:24:06PM +0100, Georg-Johann Lay wrote:
Problem is that the assembler name might or might not be prefixed by '*'
depending on when TARGET_SET_CURRENT_FUNCTION is called.
The change is just to fix wro
Am 03/13/2014 07:36 PM, schrieb Denis Chertykov:
2014-03-13 21:41 GMT+04:00 Georg-Johann Lay:
Am 03/13/2014 04:41 PM, schrieb Senthil Kumar Selvaraj:
On Thu, Mar 13, 2014 at 02:24:06PM +0100, Georg-Johann Lay wrote:
Problem is that the assembler name might or might not be prefixed by
Senthil Kumar Selvaraj schrieb:
This patch modifies AVR target's ASM spec to pass -mlink-relax to the
assembler if -mrelax is passed to the compiler driver. This was already
being passed on to the linker, this patch merely makes the assembler
also aware of it.
The corresponding patch in binutils
This patch is a rewrite of t-avr so that STAMP is not needed.
Ada triggers a mistimed re-build of targets using STAMP which causes PR55243
because the respective places in gnattools ada/gcc-interface don't set STAMP.
With this patch, files that were formerly auto-generated must be generated by
ha
Currently, data in address space __memx is allocated to .progmem.data, i.e. the
same section as for __flash resp. progmem. However, there os no need that
__memx goes onto the same section and wastes precious memory in the first 64 Ki.
The patch puts __memx data into .progmemx.data so that a linke
This addendum adds --with-avrlibc to the (web) documentation.
It inserts the explanation after --with-newlib which seems to be a reasonable
place.
Ok to apply?
Johann
PR target/PR54461
* doc/install.texi (Cross-Compiler-Specific Options): Document
--with-avrlibc.
Index:
Weddington, Eric wrote:
>
> Georg-Johann Lay Sent
>> Document --with-avrlibc
>>
>> This addendum adds --with-avrlibc to the (web) documentation.
>>
>> It inserts the explanation after --with-newlib which seems to be a
>> reasonable place.
>>
&g
Weddington, Eric wrote:
> Georg-Johann Lay Sent
>> Document --with-avrlibc
>>
>>> This option is only supported for the AVR target and only for non-RTEMS
>>> configurations, which currently use newlib. The option is supported
>>> since version
>
&
This is a tentative patch that adds symbols that can be used as predicates in
the linker script.
Background is binutils PR14406: Data in address space __flash1 must be located
in such a way that the high byte (hh8) of the address is 0x1. This is needed
because avr-gcc sets RAMPZ to 0x1 before rea
Variables min_high and min_low are set but not used which cases build warnings,
fixed by this patch.
Build fine for i686-pc-linux-gnu and avr-unknown-none, the latter definitely
using this code.
Ok for trunk?
Johann
* fixed-bit.c (SATFRACT) : Only
declare / set min_low, min_hig
Georg-Johann Lay wrote:
> Variables min_high and min_low are set but not used which cases build
> warnings,
> fixed by this patch.
Better attach the patch...
> Build fine for i686-pc-linux-gnu and avr-unknown-none, the latter definitely
> using this code.
>
> Ok f
Added copyright notices (avr-lib.h, avr-modes.def):
http://gcc.gnu.org/viewcvs?view=revision&revision=195143
http://gcc.gnu.org/viewcvs?view=revision&revision=195145
Fix typos, remove trailing blanks, coding rules, ...
http://gcc.gnu.org/viewcvs?view=revision&revision=195151
This patch is a no-op change that uses typedefs starting with avr_ in order to
make clear that they are in the avr "namespace". Old types were: struct
base_arch_s, struct arch_info_s, struct mcu_type_s.
avr-cross builds fine.
Ok to apply?
* config/avr/avr-arch.h
(struct base_ar
This patch does:
1) With 8-bit int, __INT24_MAX__ and __UINT24_MAX__ need a long long suffix.
2) Defines like __FLASH that indicate if address space is available should
be like a predicate, i.e. 1 if the space is available.
Obvious and lightly tested.
Ok?
Johann
* config/avr/avr-c
Weddington, Eric wrote:
>>
>> This patch does:
>>
>> 1) With 8-bit int, __INT24_MAX__ and __UINT24_MAX__ need a long long
>> suffix.
>>
>> 2) Defines like __FLASH that indicate if address space is available should
>> be like a predicate, i.e. 1 if the space is available.
>
> I'm not seeing #2
This patch factors out stuff from builtins.def that is used again and again.
Strings like the "__builtin_avr_" common to all builtins is moved to the
implementation of DEF_BUILTIN.
Similar for ICODE and CODE_FOR_.
ID is no more needed, the needed enums are derived from NAME.
There is no function
http://gcc.gnu.org/ml/gcc-cvs-wwwdocs/2013/msg00015.html
http://gcc.gnu.org/gcc-4.8/changes.html#avr
Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v
retrieving revision 1.85
diff -u -p -r1.85 changes.h
This is tentative patch as discussed in
http://gcc.gnu.org/ml/gcc/2013-01/msg00187.html
fold-const.c gets 2 new function native_encode_fixed and
native_interpret_fixed. Code common with the integer case is factored out and
moved to the new constructor-like function double_int::from_buffer.
The
This adds the "Embedded C" bitsfx and fxbits functions to stdfix.h.
The original stdfix.h is installed as stdfix-gcc.h and included by the new,
avr-specific stdfix.h.
The implementation uses __builtin_memcpy to get a no-op implementation.
Ok for trunk?
Johann
PR target/54222
*
Denis Chertykov wrote:
> 2013/1/21 Georg-Johann Lay:
>> This adds the "Embedded C" bitsfx and fxbits functions to stdfix.h.
>>
>> The original stdfix.h is installed as stdfix-gcc.h and included by the new,
>> avr-specific stdfix.h.
>>
>> The im
Committed the following change:
http://gcc.gnu.org/r195424
* config/avr/avr.c (avr_out_fract): Make register numbers that
might be outside of source operand signed.
The problem occurs with fixed-point conversions to a wider mode and where the
input operand is in a small regno li
This patch adds the absolute value functions to stdfix.h.
DEF_BUILTIN gets one more argument LIBNAME that sets the library name. If no
folding is found, for some builtins it's more convenient to call libgcc support
directly instead of expanding to an insn.
gcc's folding is not very good, thus im
Hurugalawadi, Naveen wrote:
> Hi,
>
> Please consider this as a reminder to review the patch posted at
> following link:-
> http://gcc.gnu.org/ml/gcc-patches/2012-12/msg00765.html
What is this good for?
Why are you changing the semantics of an existing, global constraint?
Applied as obvious:
http://gcc.gnu.org/r195471
* doc/extend.texi (Example of asm with clobbered asm reg): Fix
missing ':' in asm example.
PR54814 causes spill fails because reload.c:find_valid_class_1 tests only one
hard register instead of all hard registers of regno:mode in rclass:
http://gcc.gnu.org/PR54814
The patch was originally worked out by Bernd Schmidt and fixed a problem
introduced in
http://gcc.gnu.org/r190252
The pa
Committed as obvious:
http://gcc.gnu.org/r195473
http://gcc.gnu.org/r195472
* doc/invoke.texi (AVR Built-in Macros): Document __XMEGA__.
Jeff Law schrieb:
On 01/25/2013 11:41 AM, Georg-Johann Lay wrote:
PR54814 causes spill fails because reload.c:find_valid_class_1 tests
only one
hard register instead of all hard registers of regno:mode in rclass:
http://gcc.gnu.org/PR54814
The patch was originally worked out by Bernd
Georg-Johann Lay schrieb:
Jeff Law schrieb:
On 01/25/2013 11:41 AM, Georg-Johann Lay wrote:
PR54814 causes spill fails because reload.c:find_valid_class_1 tests
only one
hard register instead of all hard registers of regno:mode in rclass:
http://gcc.gnu.org/PR54814
The patch was originally
Gerald Pfeifer wrote:
> On Thu, 24 Jan 2013, Georg-Johann Lay wrote:
>> Committed the following change:
>>
>> http://gcc.gnu.org/r195424
>>
>> * config/avr/avr.c (avr_out_fract): Make register numbers that
>> might be outside of source operand sig
Georg-Johann Lay wrote:
> Applied as obvious:
>
> http://gcc.gnu.org/r195471
>
>
> * doc/extend.texi (Example of asm with clobbered asm reg): Fix
> missing ':' in asm example.
>
The patch:
--- trunk/gcc/doc/extend.texi 2013/01/25 17:
This removes modules from libgcc that are already supported by avr-specific
fixed-point implementation and avoids duplicate functions like __mulsa3.
Ok for trunk?
Johann
libgcc/
* config/avr/t-avr (LIB2FUNCS_EXCLUDE): Add:
_mulQQ, _mulHQ, _mulHA, _mulSA,
_mulUQQ, _mul
Ping #1 for
http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01053.html
Release Manager approval is here:
http://gcc.gnu.org/ml/gcc/2013-01/msg00222.html
This is tentative patch as discussed in
http://gcc.gnu.org/ml/gcc/2013-01/msg00187.html
fold-const.c gets 2 new function native_encode_fixed
Richard Biener wrote:
> Georg-Johann Lay wrote:
>> This is tentative patch as discussed in
>>
>> http://gcc.gnu.org/ml/gcc/2013-01/msg00187.html
>>
>> fold-const.c gets 2 new function native_encode_fixed and
>> native_interpret_fixed. Code common with t
This patch solves a problem with VIEW_CONVERT_EXPR folding for fixed_cst and
that use fixed-value.c:fixed_from_double_int.
The patch sign/zero extends the double_int input according to the requested
fixed-point mode.
The patch bootstraps on x86-linux-gnu and passes testsuite on avr-unknown one.
This adds variable delays to __builtin_avr_delay_cycles.
Is this okay?
Johann
gcc/
PR target/56254
* config/avr/avr.c (avr_expand_builtin) :
Expand to delay_cycles for non-const delays.
* config/avr/avr.md (delay_cycles): New expander.
(*delay_cycles.libgc
Weddington, Eric wrote:
> From: Georg-Johann Lay
>> This adds variable delays to __builtin_avr_delay_cycles.
>>
>> Is this okay?
>
> What does this do? How does it work?
>
> Could you explain the statement in the documentation "if ticks is not a
>
Weddington, Eric wrote:
>> -Original Message- From: Georg-Johann Lay Sent: Friday, February
>> 08, 2013 9:00 AM To: Weddington, Eric Cc: gcc-patches@gcc.gnu.org; Denis
>> Chertykov; Joerg Wunsch Subject: Re: [Patch,avr] Implement PR56254
>>
>> Weddington,
This patch adds optimized support for (short) (un)signed _Sat _Accum
multiplication (HA, UHA, SA, USA).
The 32-bit multiply uses 64 = 32 * 32 widening multiplication. As a spin-off,
the widening mul is available as [u]mulsidi3 default pattern.
Okay for trunk?
Johann
gcc/
PR target/54
om 5 to 6.
* config/avr/t-multilib: Regenerate.
* config/avr/avr-tables.opt: Regenerate.
* doc/avr-mmcu.texi: Regenerate.
Georg-Johann Lay wrote:
> This patch adds more devices to avr-mcus.def and regenerates depending files.
>
> There is a new spec function that is t
And here 2/2 with the device -> arch mapping for gas.
Ok for trunk?
Johann
* config/avr/avr.h (device_to_arch): Rename to device_to_ld.
(avr_device_to_arch): Rename to avr_device_to_ld.
(avr_device_to_as): New prototype.
(EXTRA_SPEC_FUNCTIONS): Add device_to_as.
CCing Richard in case this needs RM approval (I don't think so because
the patch just changes ternary parts).
Ping #1 for:
http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00579.html
Georg-Johann Lay schrieb:
And here 2/2 with the device -> arch mapping for gas.
Ok for trunk?
CCing Richard in case this needs RM approval (I don't think so because
the patch just changes ternary parts).
Ping #1 for:
http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00577.html
http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00576.html
Georg-Johann Lay schrieb:
Better attach the
Pedro Alves schrieb:
On Friday 14 October 2011 18:19:00, Georg-Johann Lay wrote:
Paolo Bonzini schrieb:
On 10/14/2011 06:23 PM, Georg-Johann Lay wrote:
+@item -mjump-to-noreturn
+@opindex mjump-to-noreturn
+Use a jump instruction instead of a call instruction when calling a
+no-return
Paolo Bonzini schrieb:
-moptimize-noreturn-calls
Optimize noreturn calls. This might make debugging harder but
will save storing the return address when calling roreturn
functions.
Enabled at levels -O2, -O3, -Os.
But the "makes debugging harder" clause is true for almo
This is more code clean-up for insn attributes.
It removes "mcu_have_movw", "mcu_mega" and defines "enabled" and "isa"
attributes instead.
The "isa" attribute which triggers "enabled" is a replacement for AVR_HAVE_MOVW
assembler dialect. We don't actually support assembler dialects but different
paul_kon...@dell.com schrieb:
>> There is no real post morten debugging on AVR as there is nothing like
>> core dump.
>
> But if there is any kind of debugging at all, you might use the debugger to
> put a breakpoint in abort(), and if so, having that invoked via jmp means
> you don't know who cal
This patch do some tweaks to addhi3 like adding QI scratch register.
The original *addhi3 insn is still there and located prior to new
addhi3_clobber insn because addhi3 is special to reload (thanks Danis for this
note) so that there is a version with and a version without scratch register.
Patch
Denis Chertykov schrieb:
> 2011/10/18 Georg-Johann Lay :
>> This patch do some tweaks to addhi3 like adding QI scratch register.
>>
>> The original *addhi3 insn is still there and located prior to new
>> addhi3_clobber insn because addhi3 is special to reload (thanks Dan
Denis Chertykov schrieb:
> 2011/10/18 Georg-Johann Lay :
>> Denis Chertykov schrieb:
>>> 2011/10/18 Georg-Johann Lay :
>>>> This patch do some tweaks to addhi3 like adding QI scratch register.
>>>>
>>>> The original *addhi3 insn is still t
Denis Chertykov schrieb:
> 2011/10/18 Georg-Johann Lay :
>> Denis Chertykov schrieb:
>>> 2011/10/18 Georg-Johann Lay :
>>>> Denis Chertykov schrieb:
>>>>> 2011/10/18 Georg-Johann Lay :
>>>>>> This patch do some tweaks to addhi3 like
Georg-Johann Lay schrieb:
Denis Chertykov schrieb:
What difference in size of avr-libc ?
I have no tool for smart size analysis, so here is just a diff:
After rebuilding avr-libc with respective compiler version, did respectively:
$ find . -name 'lib[mc].a' -exec avr-size {}
Denis Chertykov schrieb:
>> Georg-Johann Lay schrieb:
>>> Denis Chertykov schrieb:
>>>
>>>> What difference in size of avr-libc ?
>>> I have no tool for smart size analysis, so here is just a diff:
>>>
>>> After rebuilding
Ulrich Weigand schrieb:
> Hello,
>
> Georg-Johann Lay has proposed a patch to add named address space support
> to the AVR target here:
> http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00471.html
>
> Since the target needs to make register allocation decisions for
>
This patch adds support to consistently use EIND.
The compiler never sets this SFR but uses it in table jumps and EIJMP/EICALL
instructions.
Custom startup code could set EIND to an other value than 0 and the compiler
should use EIND consistently given that EIND might not be zero.
EIND != 0 will
>> This patch adds support to consistently use EIND.
>>
>> The compiler never sets this SFR but uses it in table jumps and EIJMP/EICALL
>> instructions.
>>
>> Custom startup code could set EIND to an other value than 0 and the compiler
>> should use EIND consistently given that EIND might not be ze
Ulrich Weigand schrieb:
> Georg-Johann Lay wrote:
>> Ulrich Weigand schrieb:
>>> Hello,
>>>
>>> Georg-Johann Lay has proposed a patch to add named address space support
>>> to the AVR target here:
>>> http://gcc.gnu.org/ml/gcc-patches/2011-1
501 - 600 of 1303 matches
Mail list logo
