Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2017-09-28T11:37:26Z

Those are setup by configure (for the most part), so your patch would only work in very limited cases.

For now, why wouldn't you just override those variables when you call make?

To fix this, the patch should use the proper configure host and target settings to specify a cross compiler. That setups two settings, one for the host (Verilator) compiler, and one for the target (Verilated) compiler etc. Then configure needs to run on each compiler separately to determine the appropriate flags, as assuming both are the same (flags/version) isn't correct for cross compilation. Alsosome make variables that are now common need host/target splits. It certainly would be appreciated if you'd like to make a patch to do this.

Original Redmine Comment
Author Name: Rob Stoddard
Original Date: 2017-09-28T17:30:10Z

I'm thinking you're thinking about ./configure for building Verilator itself, however, I'm talking about the verilated outcome from a design.

What I'm used to is doing a cross compile with the variable CROSS_COMPILE set, but in other cases it's not set... so if CROSS_COMPILE is not set, CXX becomes g++ which is the intended outcome for a non-cross build. In my case, there is a chance that CROSS_COMPILE is set to x86_64-w64-mingw32- which would make CXX become x86_64-w64-mingw32-g++ which would work for building Windows software on a Linux machine.

In no case do I want to fix Verilator to generating code for a specific platform.

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2017-09-28T22:23:55Z

Configure sets up both the compile options for the host, and for the target and assumes at present both are the same. You'll see the problem if you set your cross compiler to something really different (e.g. non GCC) as it will get passed the host's flags incorrectly (as again Verilator thinks host and target are the same).

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2018-08-25T14:44:50Z

Seems like this got stalled so closing due to age, if you end up with a general patch for cross compiling please reopen.