Skip to content

Conversation

@bbaltz505
Copy link
Contributor

No description provided.

@bbaltz505
Copy link
Contributor Author

@SidLeung please review. This PR passes BAT testing and compile tests

@SidLeung
Copy link
Contributor

@bbaltz505 @calvinatintel Looks good. Please merge. I didn't know the -lnsim flag is in application compile flag. I thought it was used for building the toolchain.

@bbaltz505
Copy link
Contributor Author

@SidLeung have you confirmed that this toolchain (http://mkj.ndg.intel.com/view/ARC%20Toolchain/job/ARC_Toolchain-Linux64-Sid/ws/arc-toolchain-linux64-arcem.tar.bz2) fixes the original floating point issue?

@SidLeung
Copy link
Contributor

SidLeung commented May 2, 2016

I am going to verify that today. Will keep you update.

@SidLeung
Copy link
Contributor

SidLeung commented May 2, 2016

The performance of the new toolchain is not good at all. Need to ping on the Synopsis team ASAP.

Signed-off-by: Brian Baltz <brian.a.baltz@intel.com>
@bbaltz505 bbaltz505 added this to the Bellatrix milestone May 10, 2016
@SidLeung
Copy link
Contributor

@bbaltz505 @calvinatintel Looks good. Please merge.

@bbaltz505 bbaltz505 merged commit ea7a804 into arduino:master May 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants