Skip to content

Commit

Permalink
[Flang] fix initializer with empty string to fix aarch64 build (llvm#…
Browse files Browse the repository at this point in the history
…126918)

After tuneCPU was changed to std::string in
c8376a9 the flang builds broke, due to
a missing initializer.

If we want to add tuneCPU to the MLIRToLLVMPassPipelineConfig, we might
want to tackle that separately after the build is restored. This should
be no different than the previous behaviour.
  • Loading branch information
rscottmanley authored and flovent committed Feb 13, 2025
1 parent 9bd4c7e commit efd2bf0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions flang/lib/Optimizer/Passes/Pipelines.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -325,8 +325,8 @@ void createDefaultFIRCodeGenPassPipeline(mlir::PassManager &pm,

pm.addPass(fir::createFunctionAttr(
{framePointerKind, config.NoInfsFPMath, config.NoNaNsFPMath,
config.ApproxFuncFPMath, config.NoSignedZerosFPMath,
config.UnsafeFPMath}));
config.ApproxFuncFPMath, config.NoSignedZerosFPMath, config.UnsafeFPMath,
""}));

fir::addFIRToLLVMPass(pm, config);
}
Expand Down

0 comments on commit efd2bf0

Please sign in to comment.