Opened 5 years ago

Closed 5 years ago

#10711 closed defect (fixed)

Wrong linker option in makefile.vc

Reported by: rk Owned by:
Priority: low Milestone:
Component: build Version: stable-latest
Keywords: Cc:
Blocked By: Blocking:
Patch: no

Description

If you build wxWidgets using nmake and specifying TARGET_CPU=AMD64 to build the 64-bit version the variable LINK_TARGET_CPU in makefile.vc will be set to /MACHINE:AMD64. But according to MSDN the only allowed values for the /MACHINE option are:

ARM|EBC|IA64|MIPS|MIPS16|MIPSFPU|MIPSFPU16|SH4|THUMB|X64|X86

The correct value for LINK_TARGET_CPU should therefor be /MACHINE:X64 instead of /MACHINE:AMD64.

I don't know what this wrong value currently does. I only caught it by accident while looking into another problem.

Change History (3)

comment:1 Changed 5 years ago by mweth

Ok thanks for letting us know, we'll look at it. AMD64 was the documented value that had to be used originally, though it's not suprising that they changed it.

comment:2 Changed 5 years ago by vadz

  • Priority changed from normal to low
  • Status changed from new to confirmed

AMD64 still works but the official name is indeed X64 and is also accepted by both VC8 and VC9 so let's use it instead.

comment:3 Changed 5 years ago by VZ

  • Resolution set to fixed
  • Status changed from confirmed to closed

(In [62825]) Use X64 machine type for MSVC linker instead of old AMD64.

The officially supported machine type name is X64 now, even if AMD64 is still
supported.

Closes #10711.

Note: See TracTickets for help on using tickets.