[llvm] r314123 - [ARM] Fix -Wdangling-else warning.
Arnold Schwaighofer via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 25 10:45:09 PDT 2017
Thanks!
> On Sep 25, 2017, at 10:35 AM, Benjamin Kramer via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>
> Author: d0k
> Date: Mon Sep 25 10:35:38 2017
> New Revision: 314123
>
> URL: http://llvm.org/viewvc/llvm-project?rev=314123&view=rev
> Log:
> [ARM] Fix -Wdangling-else warning.
>
> A ternary is clearer here. No functionality change.
>
> Modified:
> llvm/trunk/lib/Target/ARM/ARMBaseRegisterInfo.cpp
>
> Modified: llvm/trunk/lib/Target/ARM/ARMBaseRegisterInfo.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMBaseRegisterInfo.cpp?rev=314123&r1=314122&r2=314123&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/ARM/ARMBaseRegisterInfo.cpp (original)
> +++ llvm/trunk/lib/Target/ARM/ARMBaseRegisterInfo.cpp Mon Sep 25 10:35:38 2017
> @@ -94,10 +94,8 @@ ARMBaseRegisterInfo::getCalleeSavedRegs(
>
> if (STI.getTargetLowering()->supportSwiftError() &&
> F->getAttributes().hasAttrSomewhere(Attribute::SwiftError))
> - if (STI.isTargetDarwin())
> - return CSR_iOS_SwiftError_SaveList;
> - else
> - return CSR_AAPCS_SwiftError_SaveList;
> + return STI.isTargetDarwin() ? CSR_iOS_SwiftError_SaveList
> + : CSR_AAPCS_SwiftError_SaveList;
>
> if (STI.isTargetDarwin() && F->getCallingConv() == CallingConv::CXX_FAST_TLS)
> return MF->getInfo<ARMFunctionInfo>()->isSplitCSR()
> @@ -125,10 +123,8 @@ ARMBaseRegisterInfo::getCallPreservedMas
>
> if (STI.getTargetLowering()->supportSwiftError() &&
> MF.getFunction()->getAttributes().hasAttrSomewhere(Attribute::SwiftError))
> - if (STI.isTargetDarwin())
> - return CSR_iOS_SwiftError_RegMask;
> - else
> - return CSR_AAPCS_SwiftError_RegMask;
> + return STI.isTargetDarwin() ? CSR_iOS_SwiftError_RegMask
> + : CSR_AAPCS_SwiftError_RegMask;
>
> if (STI.isTargetDarwin() && CC == CallingConv::CXX_FAST_TLS)
> return CSR_iOS_CXX_TLS_RegMask;
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list