================ @@ -0,0 +1,236 @@ +//===- X86SuppressAPXForReloc.cpp - Suppress APX features for relocations -===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception +// +//===----------------------------------------------------------------------===// +/// \file +/// +/// This pass is added to suppress APX features for relocations. It's used to +/// keep backward compatibility with old version of linker having no APX +/// support. It can be removed after APX support is included in the default +/// linker on OS. +/// +//===----------------------------------------------------------------------===// + +#include "MCTargetDesc/X86BaseInfo.h" +#include "MCTargetDesc/X86MCTargetDesc.h" +#include "X86.h" +#include "X86InstrInfo.h" +#include "X86RegisterInfo.h" +#include "X86Subtarget.h" + +#include "llvm/CodeGen/MachineFunctionPass.h" +#include "llvm/CodeGen/MachineInstrBuilder.h" +#include "llvm/CodeGen/MachineOperand.h" +#include "llvm/CodeGen/MachineRegisterInfo.h" +#include "llvm/CodeGen/Passes.h" +#include "llvm/CodeGen/TargetRegisterInfo.h" +#include "llvm/InitializePasses.h" +#include "llvm/Support/ErrorHandling.h" +#include "llvm/Target/TargetMachine.h" + +using namespace llvm; + +#define DEBUG_TYPE "x86-suppress-apx-for-relocation" + +cl::opt<bool> X86EnableAPXForRelocation( + "x86-enable-apx-for-relocation", + cl::desc("Enable APX features (EGPR, NDD and NF) for instructions with " + "relocations on x86-64 ELF"), + cl::init(false)); + +namespace { +class X86SuppressAPXForRelocationPass : public MachineFunctionPass { +public: + X86SuppressAPXForRelocationPass() : MachineFunctionPass(ID) {} + + StringRef getPassName() const override { + return "X86 Suppress APX features for relocation"; + } + + bool runOnMachineFunction(MachineFunction &MF) override; + + static char ID; +}; +} // namespace + +char X86SuppressAPXForRelocationPass::ID = 0; + +INITIALIZE_PASS_BEGIN(X86SuppressAPXForRelocationPass, DEBUG_TYPE, + "X86 Suppress APX features for relocation", false, false) +INITIALIZE_PASS_END(X86SuppressAPXForRelocationPass, DEBUG_TYPE, + "X86 Suppress APX features for relocation", false, false) + +FunctionPass *llvm::createX86SuppressAPXForRelocationPass() { + return new X86SuppressAPXForRelocationPass(); +} + +static void suppressEGPRRegClass(MachineFunction &MF, MachineInstr &MI, + unsigned int OpNum) { + MachineRegisterInfo *MRI = &MF.getRegInfo(); + Register Reg = MI.getOperand(OpNum).getReg(); + if (!Reg.isVirtual()) { + assert(!X86II::isApxExtendedReg(Reg) && "APX EGPR is used unexpectedly."); + return; + } + + const TargetRegisterClass *RC = MRI->getRegClass(Reg); + const TargetRegisterClass *NewRC = X86II::constrainRegClassToNonRex2(RC); + MRI->setRegClass(Reg, NewRC); +} + +static bool handleInstructionWithEGPR(MachineFunction &MF, + const X86Subtarget &ST) { + if (!ST.hasEGPR()) + return false; + + auto suppressEGPRInInstrWithReloc = [&](MachineInstr &MI, + ArrayRef<unsigned> OpNoArray) { + int MemOpNo = X86II::getMemoryOperandNo(MI.getDesc().TSFlags) + + X86II::getOperandBias(MI.getDesc()); + const MachineOperand &MO = MI.getOperand(X86::AddrDisp + MemOpNo); + if (MO.getTargetFlags() == X86II::MO_GOTTPOFF || + MO.getTargetFlags() == X86II::MO_GOTPCREL) { + LLVM_DEBUG(dbgs() << "Transform instruction with relocation type:\n " + << MI); + for (unsigned OpNo : OpNoArray) + suppressEGPRRegClass(MF, MI, OpNo); + LLVM_DEBUG(dbgs() << "to:\n " << MI << "\n"); + } + }; + + for (MachineBasicBlock &MBB : MF) { + for (MachineInstr &MI : MBB) { + unsigned Opcode = MI.getOpcode(); + switch (Opcode) { + // For GOTPC32_TLSDESC, it's emitted with physical register (EAX/RAX) in + // X86AsmPrinter::LowerTlsAddr, and there is no corresponding target + // flag for it, so we don't need to handle LEA64r with TLSDESC and EGPR + // in this pass (before emitting assembly). + case X86::TEST32mr: + case X86::TEST64mr: { + suppressEGPRInInstrWithReloc(MI, {5}); ---------------- phoebewang wrote:
Do we only need to suppress the register operand? There's no problem with base/index register in the memory operand? https://github.com/llvm/llvm-project/pull/136660 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
