[riot-commits] [RIOT-OS/RIOT] 0feebc: cpu/cortexm_common: Drop #7776's LTO workaround

Martine Lenders noreply at github.com
Wed Jul 15 11:19:47 CEST 2020


  Branch: refs/heads/master
  Home:   https://github.com/RIOT-OS/RIOT
  Commit: 0feebcb0940827664a94c4d51afb1c0611cc93b5
      https://github.com/RIOT-OS/RIOT/commit/0feebcb0940827664a94c4d51afb1c0611cc93b5
  Author: Marian Buschsieweke <marian.buschsieweke at ovgu.de>
  Date:   2020-07-15 (Wed, 15 Jul 2020)

  Changed paths:
    M cpu/cortexm_common/Makefile

  Log Message:
  -----------
  cpu/cortexm_common: Drop #7776's LTO workaround

The workaround from #7776 is no longer needed with recent toolchains, e.g. such
as the toolchain in the riot/riotbuild docker image.


  Commit: 91a294aa45d48f1369b6a2b77b85a019125a05cd
      https://github.com/RIOT-OS/RIOT/commit/91a294aa45d48f1369b6a2b77b85a019125a05cd
  Author: Marian Buschsieweke <marian.buschsieweke at ovgu.de>
  Date:   2020-07-15 (Wed, 15 Jul 2020)

  Changed paths:
    M cpu/cortexm_common/Makefile
    M cpu/cortexm_common/thread_arch.c

  Log Message:
  -----------
  cpu/cortexm_common: Drop LTO workaround for Cortex M thread_arch.c

The `ldr    r1, =sched_active_thread` instruction couldn't be assembled with
LTO, as the no immediate offset could be found to construct the address of
`sched_active_thread`. This commit instructs the assembler to generate a
literate pool which can be used to construct the address. While this issue
was only triggered during LTO, it theoretically could also pop up without LTO
due to unrelated changes. Thus, it is a good idea to create the literate pool
even without LTO enabled.


  Commit: 7cff1b915d2b48caa377bee16f00d197240df415
      https://github.com/RIOT-OS/RIOT/commit/7cff1b915d2b48caa377bee16f00d197240df415
  Author: Martine Lenders <m.lenders at fu-berlin.de>
  Date:   2020-07-15 (Wed, 15 Jul 2020)

  Changed paths:
    M cpu/cortexm_common/Makefile
    M cpu/cortexm_common/thread_arch.c

  Log Message:
  -----------
  Merge pull request #14422 from maribu/cortexm_thread_arch

cpu/cortexm_common: Fixes for thread_arch.c and dropped workaround


Compare: https://github.com/RIOT-OS/RIOT/compare/ba329daeee85...7cff1b915d2b


More information about the commits mailing list