This is an archive of the discontinued LLVM Phabricator instance.

[cmake] Quote args when generating /libpath option specifying runtime dir for link/lld-link
ClosedPublic

Authored by ormris on Mar 10 2023, 1:34 PM.

Details

Summary

When the runtime dir path includes whitespace (ex. when it's installed under "Program Files" on Windows), the lack of quotes causes the linker to interpret the path as one or more arguments. None of these path fragments exist, so the link fails.

Diff Detail

Event Timeline

ormris created this revision.Mar 10 2023, 1:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 10 2023, 1:34 PM
ormris requested review of this revision.Mar 10 2023, 1:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 10 2023, 1:34 PM
smeenai accepted this revision.Mar 10 2023, 1:58 PM

LGTM

This revision is now accepted and ready to land.Mar 10 2023, 1:58 PM
This revision was landed with ongoing or failed builds.Mar 10 2023, 2:13 PM
This revision was automatically updated to reflect the committed changes.