Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • D dynamorio
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,467
    • Issues 1,467
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 44
    • Merge requests 44
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • DynamoRIO
  • dynamorio
  • Merge requests
  • !2633

i#2632 clang: fix Travis clang build

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Derek Bruening requested to merge i2632-clang-travis into master Sep 10, 2017
  • Overview 3
  • Commits 4
  • Pipelines 0
  • Changes 6

Fixes a clang warning in suite/tests/linux/signal_racesys.c.

Changes two inline asm direct jumps to indirect in core/unix/signal.c to work around clang text relocations that have suddently appeared in recent clang versions.

However, clang is still marking libdynamorio.so as TEXTREL, so I'm disabling the TEXTREL check on libdynamorio.so for 32-bit release clang. I also add a check refusing to build an official package with clang until this is fixed.

Issue: #2632 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: i2632-clang-travis