intel/llvm
1
0
Fork 0
mirror of https://github.com/intel/llvm.git synced 2026-01-20 10:58:11 +08:00
Code Issues Packages Projects Releases Wiki Activity

[flang] Remove extra space added with --dependent-lib option

Browse Source 
This patch fixes a bug with the --dependent-lib option where an
extra space is added to the directive causing linking to fail.
This commit is contained in:
David Truby
2023-11-20 16:50:03 +00:00
parent ddfed815c9
commit 5e36c64cb6
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
flang/lib/Frontend/FrontendActions.cpp
View File

@@ -260,7 +260,7 @@ static void addDepdendentLibs(mlir::ModuleOp &mlirModule,
auto builder = mlir::OpBuilder(mlirModule.getRegion());
for (const std::string &lib : libs) {
builder.create<mlir::LLVM::LinkerOptionsOp>(
mlirModule.getLoc(), builder.getStrArrayAttr({"/DEFAULTLIB:", lib}));
mlirModule.getLoc(), builder.getStrArrayAttr({"/DEFAULTLIB:" + lib}));
}
}

2
flang/test/Driver/dependent-lib.f90
View File

@@ -10,7 +10,7 @@
! REDEFINE: %{triple} = aarch64-apple-darwin
! RUN: not %{compile} | FileCheck %s --check-prefixes=CHECK-NOWIN
! CHECK: llvm.linker_options ["/DEFAULTLIB:", "libtest"]
! CHECK: llvm.linker_options ["/DEFAULTLIB:libtest"]
program test
end program test
! CHECK-NOWIN: --dependent-lib is only supported on Windows