Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • D dblclockfft
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • 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
  • Dan Gisselquist
  • dblclockfft
  • Merge requests
  • !2

Fix compile errors for macOS 10.12

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/C-Elegans/fix_compile_errors_mac into master Oct 03, 2018
  • Overview 5
  • Commits 1
  • Pipelines 0
  • Changes 7

Created by: C-Elegans

On macOS Sierra, running 'make bench' gives compile errors in all of the verilator testbenches, complaining about an ambiguous call to dump (because there is no dump() defined for a uint64_t, or clang++ doesn't define vluint64_t to be the same as uint64_t). This patch fixes those compiler errors

Compiler error:

mpy_tb.cpp:113:13: error: call to member function 'dump' is ambiguous
                        m_trace->dump((uint64_t)(10ul*m_tickcount+5));
                        ~~~~~~~~~^~~~
/usr/local/Cellar/verilator/3.926/share/verilator/include/verilated_vcd_c.h:434:10: note: candidate function
    void dump (vluint64_t timeui) { m_sptrace.dump(timeui); }
         ^
/usr/local/Cellar/verilator/3.926/share/verilator/include/verilated_vcd_c.h:437:10: note: candidate function
    void dump (double timestamp) { dump(static_cast<vluint64_t>(timestamp)); }
         ^
/usr/local/Cellar/verilator/3.926/share/verilator/include/verilated_vcd_c.h:438:10: note: candidate function
    void dump (vluint32_t timestamp) { dump(static_cast<vluint64_t>(timestamp)); }
         ^
/usr/local/Cellar/verilator/3.926/share/verilator/include/verilated_vcd_c.h:439:10: note: candidate function
    void dump (int timestamp) { dump(static_cast<vluint64_t>(timestamp)); }

versions:

$ clang++ -v
    Apple LLVM version 9.0.0 (clang-900.0.38)
    Target: x86_64-apple-darwin16.7.0
    Thread model: posix

$ verilator --version
    Verilator 3.926 2018-08-22 rev UNKNOWN_REV
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/C-Elegans/fix_compile_errors_mac