Only use noasm with arm

TIBCOSoftware / mashling

Project Mashling

86 stars 15 forks source link

Only use noasm with arm #228

Closed pointlander closed 6 years ago

pointlander commented 6 years ago

noasm is not needed with amd64

jpark800 commented 6 years ago

@pointlander What is the background of this change?

pointlander commented 6 years ago

I applied the 'noasm' build tag to all platforms in order to fix issue #226, but the 'noasm' build tag is only needed for the 'arm64' architecture. Using the 'noasm' build tag for 'amd64' could negatively impact performance of the resulting binary. @jpark800