Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

benchmarks: Various fixes for compiling with Clang #225

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

benchmarks: Various fixes for compiling with Clang #225

wants to merge 3 commits into from

Commits on Dec 11, 2019

  1. benchmarks: Don't use GCC's C nested function extension 

    It's unnecessary and non-portable; Clang doesn't even implement that
part of GNU C.
    @jrtc27
    jrtc27 committed Dec 11, 2019
    Configuration menu
    Copy the full SHA
    50d75cb View commit details
    Browse the repository at this point in the history

  2. benchmarks: Pass -T to the linker 

    Clang's driver does not seem to forward this on, unlike GCC's, so
instead ensure the linker sees it by using -Wl.
    @jrtc27
    jrtc27 committed Dec 11, 2019
    Configuration menu
    Copy the full SHA
    65baa58 View commit details
    Browse the repository at this point in the history

  3. benchmarks: Fix init_tls on Clang 

    Clang only supports register variables when declared at global scope
(and only for non-allocatable registers, which is fine for tp), so move
thread_pointer. This does not make thread_pointer visible outside the
translation unit, it merely acts as a C alias for tp, so the semantics
remain unchanged.
    @jrtc27
    jrtc27 committed Dec 11, 2019
    Configuration menu
    Copy the full SHA
    20378c0 View commit details
    Browse the repository at this point in the history