Skip to content
GitLab
    • Explore Projects Groups Snippets
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 7 years ago
  • 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)

Activity

  • Loading
  • Loading
  • Loading
Please register or sign in to reply
  1. 19 Sep, 2017 1 commit
    • Derek Bruening's avatar
      Fix 32-bit asm reg clobber · 97f9c3d2
      Derek Bruening authored 7 years ago
      97f9c3d2
  2. 18 Sep, 2017 1 commit
    • Derek Bruening's avatar
      Merge branch 'master' into i2632-clang-travis · 4c7393f9
      Derek Bruening authored 7 years ago
      4c7393f9
  3. 16 Sep, 2017 1 commit
    • Derek Bruening's avatar
      Merge branch 'master' into i2632-clang-travis · 28c09ade
      Derek Bruening authored 7 years ago
      28c09ade
  4. 10 Sep, 2017 1 commit
    • Derek Bruening's avatar
      i#2632 clang: fix Travis clang build · fb5be330
      Derek Bruening authored 7 years ago
      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
      fb5be330
0 Assignees
None
Assign to
0 Reviewers
None
Request review from
Labels
0
None
0
None
    Assign labels
  • Manage project labels

Milestone
No milestone
None
None
Time tracking
No estimate or time spent
Lock merge request
Unlocked
2
2 participants
Administrator
Derek Bruening
Reference: DynamoRIO/dynamorio!2633
Source branch: i2632-clang-travis

Menu

Explore Projects Groups Snippets