[Bug target/34210] ffs builtin calls undefined __ffshi2

[Bug target/34210] ffs builtin calls undefined __ffshi2

[jackie.rosen at hushmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34210

Jackie Rosen <jackie.rosen at hushmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jackie.rosen at hushmail dot com

--- Comment #9 from Jackie Rosen <jackie.rosen at hushmail dot com> ---
*** Bug 260998 has been marked as a duplicate of this bug. ***
Seen from the domain http://volichat.com
Page where seen: http://volichat.com/adult-chat-rooms
Marked for reference. Resolved as fixed @bugzilla.

[Bug c/34210] New: ffs builtin calls undefined __ffshi2

[sjackman at gmail dot com]

ffs((uint8_t)x) and ffs((uint16_t)x) generate a call to __ffshi2, but this
function does not exist in libgcc.a, giving the error
undefined reference to `__ffshi2'.

Test case:

int main(int argc)
{
        return ffs(argc);
}


-- 
           Summary: ffs builtin calls undefined __ffshi2
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: sjackman at gmail dot com
GCC target triplet: avr


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34210

[Bug target/34210] ffs builtin calls undefined __ffshi2

[rask at gcc dot gnu dot org]

------- Comment #3 from rask at gcc dot gnu dot org  2007-12-21 22:20 -------
You want something like this in libgcc/config/avr/t-avr to get the 16-bit
versions:

# Extra 16-bit integer functions.
intfuncs16 = _absvXX2 _addvXX3 _subvXX3 _mulvXX3 _negvXX2 _ffsXX2 _clzXX2 \
             _ctzXX2 _popcountXX2
hiintfuncs16 = $(subst XX,hi,$(intfuncs16))
siintfuncs16 = $(subst XX,si,$(intfuncs16))

iter-items := $(hiintfuncs16)
iter-labels := $(siintfuncs16)
iter-sizes := $(patsubst %,2,$(siintfuncs16)) $(patsubst %,2,$(hiintfuncs16))

include $(srcdir)/empty.mk $(patsubst
%,$(srcdir)/siditi-object.mk,$(iter-items))

libgcc-objects += $(patsubst %,%$(objext),$(hiintfuncs16))
ifeq ($(enable_shared),yes)
libgcc-s-objects += $(patsubst %,%_s$(objext),$(hiintfuncs16))
endif


-- 

rask at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|c                           |target


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34210

[Bug target/34210] ffs builtin calls undefined __ffshi2

[pmarques at grupopie dot com]

------- Comment #4 from pmarques at grupopie dot com  2008-01-20 03:41 -------

gcc.c-torture/execute/ffs-1.c and gcc.c-torture/execute/ffs-2.c also fail with
this message. The test case gcc.c-torture/execute/builtin-bitops-1.c shows some
more similar errors:

builtin-bitops-1.c:(.text+0x6b4): undefined reference to `__popcounthi2'
builtin-bitops-1.c:(.text+0x6ee): undefined reference to `__parityhi2'
builtin-bitops-1.c:(.text+0x183e): undefined reference to `__clzhi2'

I tried to add the suggested libgcc/config/avr/t-avr. It didn't work, and I
don't know enough of gcc internals to understand what is going on.


-- 

pmarques at grupopie dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pmarques at grupopie dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34210

[Bug target/34210] ffs builtin calls undefined __ffshi2

[hutchinsonandy at aim dot com]

------- Comment #5 from hutchinsonandy at aim dot com  2008-03-23 00:24 -------
Patch posted:

http://gcc.gnu.org/ml/gcc-patches/2008-03/msg01341.html


-- 

hutchinsonandy at aim dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hutchinsonandy at aim dot
                   |                            |com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34210

[Bug target/34210] ffs builtin calls undefined __ffshi2

[eric dot weddington at atmel dot com]

------- Comment #6 from eric dot weddington at atmel dot com  2008-03-23 03:04 -------
*** Bug 35508 has been marked as a duplicate of this bug. ***


-- 

eric dot weddington at atmel dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dmixm at marine dot febras
                   |                            |dot ru


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34210

[Bug target/34210] ffs builtin calls undefined __ffshi2

[hutchinsonandy at gcc dot gnu dot org]

------- Comment #7 from hutchinsonandy at gcc dot gnu dot org  2008-04-07 23:16 -------
Subject: Bug 34210

Author: hutchinsonandy
Date: Mon Apr  7 23:15:35 2008
New Revision: 133993

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=133993
Log:
PR target/34210 PR target/35508 * config.host (avr-*-*): Add avr cpu_type and
avr tmake_file. * config/t-avr: New file. Build 16bit libgcc functions.

Added:
    trunk/libgcc/config/avr/
    trunk/libgcc/config/avr/t-avr
Modified:
    trunk/libgcc/ChangeLog
    trunk/libgcc/config.host


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34210

[Bug target/34210] ffs builtin calls undefined __ffshi2

[eric dot weddington at atmel dot com]

------- Comment #8 from eric dot weddington at atmel dot com  2008-04-08 15:14 -------
Commit from Andy fixes the bug.


-- 

eric dot weddington at atmel dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34210