diff options
author | Nikita Popov <npopov@redhat.com> | 2025-08-13 10:51:43 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-08-13 10:51:43 +0200 |
commit | 240c454c4d04586967531c0a999de5547d776f47 (patch) | |
tree | 4af4106eabfbe07f945daf4d816b9d90fa6db226 /llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp | |
parent | b563b274b8a8b00dadb63e67d648421c110449ff (diff) | |
download | llvm-240c454c4d04586967531c0a999de5547d776f47.zip llvm-240c454c4d04586967531c0a999de5547d776f47.tar.gz llvm-240c454c4d04586967531c0a999de5547d776f47.tar.bz2 |
[CodeGen] Remove default ctors for InputArg and OutputArg (#153205)
These make it easy to forget to initialize some members, like the newly
added OrigTy. Force these to always go through the ctor instead.
Diffstat (limited to 'llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp')
-rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp | 30 |
1 files changed, 14 insertions, 16 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp index 6256ebc..a71b440 100644 --- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp @@ -11069,23 +11069,20 @@ TargetLowering::LowerCallTo(TargetLowering::CallLoweringInfo &CLI) const { unsigned NumRegs = getNumRegistersForCallingConv(CLI.RetTy->getContext(), CLI.CallConv, VT); for (unsigned i = 0; i != NumRegs; ++i) { - ISD::InputArg MyFlags; - MyFlags.Flags = Flags; - MyFlags.VT = RegisterVT; - MyFlags.ArgVT = VT; - MyFlags.Used = CLI.IsReturnValueUsed; + ISD::InputArg Ret(Flags, RegisterVT, VT, CLI.RetTy, + CLI.IsReturnValueUsed, ISD::InputArg::NoArgIndex, 0); if (CLI.RetTy->isPointerTy()) { - MyFlags.Flags.setPointer(); - MyFlags.Flags.setPointerAddrSpace( + Ret.Flags.setPointer(); + Ret.Flags.setPointerAddrSpace( cast<PointerType>(CLI.RetTy)->getAddressSpace()); } if (CLI.RetSExt) - MyFlags.Flags.setSExt(); + Ret.Flags.setSExt(); if (CLI.RetZExt) - MyFlags.Flags.setZExt(); + Ret.Flags.setZExt(); if (CLI.IsInReg) - MyFlags.Flags.setInReg(); - CLI.Ins.push_back(MyFlags); + Ret.Flags.setInReg(); + CLI.Ins.push_back(Ret); } } } @@ -11095,11 +11092,12 @@ TargetLowering::LowerCallTo(TargetLowering::CallLoweringInfo &CLI) const { if (supportSwiftError()) { for (const ArgListEntry &Arg : Args) { if (Arg.IsSwiftError) { - ISD::InputArg MyFlags; - MyFlags.VT = getPointerTy(DL); - MyFlags.ArgVT = EVT(getPointerTy(DL)); - MyFlags.Flags.setSwiftError(); - CLI.Ins.push_back(MyFlags); + ISD::ArgFlagsTy Flags; + Flags.setSwiftError(); + ISD::InputArg Ret(Flags, getPointerTy(DL), EVT(getPointerTy(DL)), + PointerType::getUnqual(CLI.RetTy->getContext()), + /*Used=*/true, ISD::InputArg::NoArgIndex, 0); + CLI.Ins.push_back(Ret); } } } |