Page MenuHomePhabricator
Differential D96340 Diff 323224 llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp

Changeset View

Standalone View

View Options

llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp

Show First 20 Lines • Show All 377 Lines • ▼ Show 20 Lines : LLVMTargetMachine(T, computeDataLayout(TT), TT, getGPUOrDefault(TT, CPU),
TLOF(createTLOF(getTargetTriple())) { TLOF(createTLOF(getTargetTriple())) {
initAsmInfo(); initAsmInfo();
if (TT.getArch() == Triple::amdgcn) { if (TT.getArch() == Triple::amdgcn) {
if (getMCSubtargetInfo()->checkFeatures("+wavefrontsize64")) if (getMCSubtargetInfo()->checkFeatures("+wavefrontsize64"))
MRI.reset(llvm::createGCNMCRegisterInfo(AMDGPUDwarfFlavour::Wave64)); MRI.reset(llvm::createGCNMCRegisterInfo(AMDGPUDwarfFlavour::Wave64));
else if (getMCSubtargetInfo()->checkFeatures("+wavefrontsize32")) else if (getMCSubtargetInfo()->checkFeatures("+wavefrontsize32"))
MRI.reset(llvm::createGCNMCRegisterInfo(AMDGPUDwarfFlavour::Wave32)); MRI.reset(llvm::createGCNMCRegisterInfo(AMDGPUDwarfFlavour::Wave32));
} }
// Set -fixed-function-abi to true if not provided..
if (TT.getOS() == Triple::AMDHSA &&
EnableAMDGPUFixedFunctionABIOpt.getNumOccurrences() == 0)
arsenmUnsubmitted
Done
ReplyInline Actions

This isn't just changing the default, this is breaking the flag. You should check if the option wasn't specified

arsenm: This isn't just changing the default, this is breaking the flag. You should check if the option…
EnableFixedFunctionABI = true;
} }
bool AMDGPUTargetMachine::EnableLateStructurizeCFG = false; bool AMDGPUTargetMachine::EnableLateStructurizeCFG = false;
bool AMDGPUTargetMachine::EnableFunctionCalls = false; bool AMDGPUTargetMachine::EnableFunctionCalls = false;
bool AMDGPUTargetMachine::EnableFixedFunctionABI = false; bool AMDGPUTargetMachine::EnableFixedFunctionABI = false;
AMDGPUTargetMachine::~AMDGPUTargetMachine() = default; AMDGPUTargetMachine::~AMDGPUTargetMachine() = default;
▲ Show 20 Lines • Show All 979 Lines • Show Last 20 Lines