Remove duplicate wrapper linker commands in pico_mem_ops #3010

Merged

DeepPG merged 1 commit from bazel-fix-1 into develop 2026-06-17 16:11:52 -05:00

Conversation 4 Commits 1 Files changed 2 -4

DeepPG commented 2026-06-16 10:59:47 -05:00 (Migrated from github.com)

• Copy link
• Copy Markdown

Fix duplicated wrapper functions definitions

Fix duplicated wrapper functions definitions

lurch commented 2026-06-16 11:19:45 -05:00 (Migrated from github.com)

• Copy link
• Copy Markdown

Hmmm, looks like the same duplication is being done in https://github.com/raspberrypi/pico-sdk/blob/develop/src/rp2_common/pico_bit_ops/CMakeLists.txt too?

Hmmm, looks like the same duplication is being done in https://github.com/raspberrypi/pico-sdk/blob/develop/src/rp2_common/pico_bit_ops/CMakeLists.txt too?

DeepPG commented 2026-06-16 12:36:25 -05:00 (Migrated from github.com)

• Copy link
• Copy Markdown

Thank you for your observation, I fixed that too and sqaushed them, I hope that's ok.

Thank you for your observation, I fixed that too and sqaushed them, I hope that's ok.

👍 1

lurch commented 2026-06-16 17:23:04 -05:00 (Migrated from github.com)

• Copy link
• Copy Markdown

I'm kinda poking in the dark here, but looking at https://gcc.gnu.org/onlinedocs/gccint/Integer-library-routines.html and https://software-dl.ti.com/codegen/docs/tiarmclang/rel5_1_1_LTS/compiler_manual/c_cpp_language_implementation/intrinsics/index.html I wonder if the "correct fix" might have been to leave one of the __clzsi2 entries in the # gcc section, and remove the one in the # armclang section?

I'm kinda poking in the dark here, but looking at https://gcc.gnu.org/onlinedocs/gccint/Integer-library-routines.html and https://software-dl.ti.com/codegen/docs/tiarmclang/rel5_1_1_LTS/compiler_manual/c_cpp_language_implementation/intrinsics/index.html I wonder if the "correct fix" might have been to leave one of the `__clzsi2` entries in the `# gcc` section, and remove the one in the `# armclang` section?

kilograham (Migrated from github.com) approved these changes 2026-06-17 16:11:21 -05:00

Sign in to join this conversation.

Reviewers

No reviewers

kilograham

Labels

Clear labels   

bazel

  

bluetooth

  

board settings

  

boot_stage2

  

bug

Something isn't working

  

build

  

c++

  

can't reproduce

Can not reproduce the bug

  

clang

  

clib-interface

  

cmsis

  

code cleanliness

  

code size

  

compiler version issues

  

cyw43_driver

  

debug

  

discussion

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

dynamic_clock

  

elf2uf2

  

enhancement

New feature or request

  

errata

  

filesystem

  

flash layout

  

floating point

  

FreeRTOS

  

hardware_adc

  

hardware_claim

  

hardware_clocks

  

hardware_dma

  

hardware_exception

  

hardware_flash

  

hardware_gpio

  

hardware_i2c

  

hardware_irq

  

hardware_pio

  

hardware_pwm

  

hardware_rcp

  

hardware_rtc

  

hardware_spi

  

hardware_sync

  

hardware_uart

  

hardware_vreg

  

hardware_watchdog

  

hardware_xosc

  

help wanted

Extra attention is needed

  

host

  

IAR

  

installation

  

invalid

This doesn't seem right

  

linker scripts

  

lto

  

lwip

  

mbedtls

  

missing API

  

needs reporter detail

  

networking

  

newlib

  

newlib-multicore

  

non-c

  

non-cmake

  

non-gcc

  

openocd

  

orphan PR

  

OS integration

  

pico_async_context

  

pico_base

  

pico_binary_info

  

pico_flash

  

picolibc

  

pico_lwip

  

pico_malloc

  

pico_multicore

  

pico_printf

  

pico_runtime

  

pico_stdio

  

pico_stdio_usb

  

pico_sync

  

pico_time

  

pico_util

  

pioasm

  

port

issues or discussions of porting code/libraries to rp2040

  

question

Further information is requested

  

review

  

review2

  

rtos interop

Consideration for FreeRTOS co-existence

  

runtime init

  

static library

  

svd

  

test

  

tinyusb

  

vscode

  

wifi integration

  

wontfix

This will not be worked on

No labels

bazel

bluetooth

board settings

boot_stage2

bug

build

c++

can't reproduce

clang

clib-interface

cmsis

code cleanliness

code size

compiler version issues

cyw43_driver

debug

discussion

documentation

duplicate

dynamic_clock

elf2uf2

enhancement

errata

filesystem

flash layout

floating point

FreeRTOS

hardware_adc

hardware_claim

hardware_clocks

hardware_dma

hardware_exception

hardware_flash

hardware_gpio

hardware_i2c

hardware_irq

hardware_pio

hardware_pwm

hardware_rcp

hardware_rtc

hardware_spi

hardware_sync

hardware_uart

hardware_vreg

hardware_watchdog

hardware_xosc

help wanted

host

IAR

installation

invalid

linker scripts

lto

lwip

mbedtls

missing API

needs reporter detail

networking

newlib

newlib-multicore

non-c

non-cmake

non-gcc

openocd

orphan PR

OS integration

pico_async_context

pico_base

pico_binary_info

pico_flash

picolibc

pico_lwip

pico_malloc

pico_multicore

pico_printf

pico_runtime

pico_stdio

pico_stdio_usb

pico_sync

pico_time

pico_util

pioasm

port

question

review

review2

rtos interop

runtime init

static library

svd

test

tinyusb

vscode

wifi integration

wontfix

Milestone

Clear milestone

No items

No milestone

2.3.0

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

gh-raspberrypi/pico-sdk!3010

No description provided.