[windows] add PDB files to install target

[charles-zablit]

This is part of swiftlang/swift#88869.

[compnerd]

  install(FILES $<TARGET_PDB_FILE:${module}> 
    DESTINATION ${CMAKE_INSTALL_BINDIR}
    OPTIONAL)

[charles-zablit]

  install(FILES $<TARGET_PDB_FILE:${module}> 
    DESTINATION ${CMAKE_INSTALL_BINDIR}
    OPTIONAL)

Is there a specific reason for doing it this way instead of the other way around?

[compnerd]

  install(FILES $<TARGET_PDB_FILE:${module}> 
    DESTINATION ${CMAKE_INSTALL_BINDIR}
    OPTIONAL)

Is there a specific reason for doing it this way instead of the other way around?

Yes, this makes the code easier to read, this allows CMake to track the dependency properly, and it doesn't require you to know where the PDB is emitted in practice. This is the canonical way that CMake recommends.

[charles-zablit]

I tried this and it fails at configure time:

CMake Error:
  Error evaluating generator expression:

    $<TARGET_PDB_FILE:ArgumentParser>

  TARGET_PDB_FILE is not supported by the target linker.

It looks like TARGET_PDB_FILE is not available for Swift.