Re: [Patch,avr] Implement PR56254

2013-02-08 Thread Georg-Johann Lay
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,

RE: [Patch,avr] Implement PR56254

2013-02-08 Thread Weddington, Eric
> -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, Eric wrote: > > > &

Re: [Patch,avr] Implement PR56254

2013-02-08 Thread Georg-Johann Lay
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 > compile-time constant, the delay might be overe

RE: [Patch,avr] Implement PR56254

2013-02-08 Thread Weddington, Eric
> -Original Message- > From: Georg-Johann Lay > Sent: Friday, February 08, 2013 5:53 AM > To: gcc-patches@gcc.gnu.org > Cc: Denis Chertykov; Weddington, Eric > Subject: [Patch,avr] Implement PR56254 > > This adds variable delays to __builtin_avr_delay_c

[Patch,avr] Implement PR56254

2013-02-08 Thread Georg-Johann Lay
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