Skip to content

Commit 862d346

Browse files
authored
opt: Fix bad merge of llvm#167996 (llvm#168110)
After the base branch was moved to main, this somehow ended up adding a second definition of RTLCI, instead of modifying the existing one. Also fix other build error with gcc bots.
1 parent 7e7ea9c commit 862d346

File tree

6 files changed

+13
-20
lines changed

6 files changed

+13
-20
lines changed

clang/lib/CodeGen/BackendUtil.cpp

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -483,31 +483,31 @@ static bool initTargetOptions(const CompilerInstance &CI,
483483

484484
switch (CodeGenOpts.getVecLib()) {
485485
case llvm::driver::VectorLibrary::NoLibrary:
486-
Options.VectorLibrary = llvm::VectorLibrary::NoLibrary;
486+
Options.VecLib = llvm::VectorLibrary::NoLibrary;
487487
break;
488488
case llvm::driver::VectorLibrary::Accelerate:
489-
Options.VectorLibrary = llvm::VectorLibrary::Accelerate;
489+
Options.VecLib = llvm::VectorLibrary::Accelerate;
490490
break;
491491
case llvm::driver::VectorLibrary::Darwin_libsystem_m:
492-
Options.VectorLibrary = llvm::VectorLibrary::DarwinLibSystemM;
492+
Options.VecLib = llvm::VectorLibrary::DarwinLibSystemM;
493493
break;
494494
case llvm::driver::VectorLibrary::LIBMVEC:
495-
Options.VectorLibrary = llvm::VectorLibrary::LIBMVEC;
495+
Options.VecLib = llvm::VectorLibrary::LIBMVEC;
496496
break;
497497
case llvm::driver::VectorLibrary::MASSV:
498-
Options.VectorLibrary = llvm::VectorLibrary::MASSV;
498+
Options.VecLib = llvm::VectorLibrary::MASSV;
499499
break;
500500
case llvm::driver::VectorLibrary::SVML:
501-
Options.VectorLibrary = llvm::VectorLibrary::SVML;
501+
Options.VecLib = llvm::VectorLibrary::SVML;
502502
break;
503503
case llvm::driver::VectorLibrary::SLEEF:
504-
Options.VectorLibrary = llvm::VectorLibrary::SLEEFGNUABI;
504+
Options.VecLib = llvm::VectorLibrary::SLEEFGNUABI;
505505
break;
506506
case llvm::driver::VectorLibrary::ArmPL:
507-
Options.VectorLibrary = llvm::VectorLibrary::ArmPL;
507+
Options.VecLib = llvm::VectorLibrary::ArmPL;
508508
break;
509509
case llvm::driver::VectorLibrary::AMDLIBM:
510-
Options.VectorLibrary = llvm::VectorLibrary::AMDLIBM;
510+
Options.VecLib = llvm::VectorLibrary::AMDLIBM;
511511
break;
512512
}
513513

llvm/include/llvm/Target/TargetOptions.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -411,7 +411,7 @@ class TargetOptions {
411411
DebuggerKind DebuggerTuning = DebuggerKind::Default;
412412

413413
/// Vector math library to use.
414-
VectorLibrary VectorLibrary = VectorLibrary::NoLibrary;
414+
VectorLibrary VecLib = VectorLibrary::NoLibrary;
415415

416416
private:
417417
/// Flushing mode to assume in default FP environment.

llvm/lib/CodeGen/CommandFlags.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -632,7 +632,7 @@ codegen::InitTargetOptionsFromCodeGenFlags(const Triple &TheTriple) {
632632
Options.EnableTLSDESC =
633633
getExplicitEnableTLSDESC().value_or(TheTriple.hasDefaultTLSDESC());
634634
Options.ExceptionModel = getExceptionModel();
635-
Options.VectorLibrary = getVectorLibrary();
635+
Options.VecLib = getVectorLibrary();
636636
Options.EmitStackSizeSection = getEnableStackSizeSection();
637637
Options.EnableMachineFunctionSplitter = getEnableMachineFunctionSplitter();
638638
Options.EnableStaticDataPartitioning = getEnableStaticDataPartitioning();

llvm/lib/CodeGen/TargetLoweringBase.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -753,8 +753,7 @@ TargetLoweringBase::TargetLoweringBase(const TargetMachine &tm)
753753
: TM(tm),
754754
RuntimeLibcallInfo(TM.getTargetTriple(), TM.Options.ExceptionModel,
755755
TM.Options.FloatABIType, TM.Options.EABIVersion,
756-
TM.Options.MCOptions.getABIName(),
757-
TM.Options.VectorLibrary),
756+
TM.Options.MCOptions.getABIName(), TM.Options.VecLib),
758757
Libcalls(RuntimeLibcallInfo) {
759758
initActions();
760759

llvm/tools/llc/llc.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -696,8 +696,7 @@ static int compileModule(char **argv, LLVMContext &Context,
696696
}
697697

698698
// Add an appropriate TargetLibraryInfo pass for the module's triple.
699-
TargetLibraryInfoImpl TLII(M->getTargetTriple(),
700-
Target->Options.VectorLibrary);
699+
TargetLibraryInfoImpl TLII(M->getTargetTriple(), Target->Options.VecLib);
701700

702701
// The -disable-simplify-libcalls flag actually disables all builtin optzns.
703702
if (DisableSimplifyLibCalls)

llvm/tools/opt/optdriver.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -680,11 +680,6 @@ optMain(int argc, char **argv,
680680
"", // FIXME: Get ABI name from MCOptions
681681
VecLib);
682682

683-
// FIXME: Get ABI name from MCOptions
684-
RTLIB::RuntimeLibcallsInfo RTLCI(ModuleTriple, codegen::getExceptionModel(),
685-
codegen::getFloatABIForCalls(),
686-
codegen::getEABIVersion());
687-
688683
// The -disable-simplify-libcalls flag actually disables all builtin optzns.
689684
if (DisableSimplifyLibCalls)
690685
TLII.disableAllFunctions();

0 commit comments

Comments
 (0)