mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-08-23 02:29:18 +00:00
Simplify boolean expressions with true and false using clang-tidy
Patch by Richard (legalize@xmission.com) Differential Revision: http://reviews.llvm.org/D8521 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@232961 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
2f729838f1
commit
437c2863f0
@ -522,15 +522,15 @@ void NVPTXAsmPrinter::emitKernelFunctionDirectives(const Function &F,
|
||||
// If none of reqntid* is specified, don't output reqntid directive.
|
||||
unsigned reqntidx, reqntidy, reqntidz;
|
||||
bool specified = false;
|
||||
if (llvm::getReqNTIDx(F, reqntidx) == false)
|
||||
if (!llvm::getReqNTIDx(F, reqntidx))
|
||||
reqntidx = 1;
|
||||
else
|
||||
specified = true;
|
||||
if (llvm::getReqNTIDy(F, reqntidy) == false)
|
||||
if (!llvm::getReqNTIDy(F, reqntidy))
|
||||
reqntidy = 1;
|
||||
else
|
||||
specified = true;
|
||||
if (llvm::getReqNTIDz(F, reqntidz) == false)
|
||||
if (!llvm::getReqNTIDz(F, reqntidz))
|
||||
reqntidz = 1;
|
||||
else
|
||||
specified = true;
|
||||
@ -544,15 +544,15 @@ void NVPTXAsmPrinter::emitKernelFunctionDirectives(const Function &F,
|
||||
// If none of maxntid* is specified, don't output maxntid directive.
|
||||
unsigned maxntidx, maxntidy, maxntidz;
|
||||
specified = false;
|
||||
if (llvm::getMaxNTIDx(F, maxntidx) == false)
|
||||
if (!llvm::getMaxNTIDx(F, maxntidx))
|
||||
maxntidx = 1;
|
||||
else
|
||||
specified = true;
|
||||
if (llvm::getMaxNTIDy(F, maxntidy) == false)
|
||||
if (!llvm::getMaxNTIDy(F, maxntidy))
|
||||
maxntidy = 1;
|
||||
else
|
||||
specified = true;
|
||||
if (llvm::getMaxNTIDz(F, maxntidz) == false)
|
||||
if (!llvm::getMaxNTIDz(F, maxntidz))
|
||||
maxntidz = 1;
|
||||
else
|
||||
specified = true;
|
||||
@ -673,7 +673,7 @@ static bool usedInOneFunc(const User *U, Function const *&oneFunc) {
|
||||
}
|
||||
|
||||
for (const User *UU : U->users())
|
||||
if (usedInOneFunc(UU, oneFunc) == false)
|
||||
if (!usedInOneFunc(UU, oneFunc))
|
||||
return false;
|
||||
|
||||
return true;
|
||||
@ -687,7 +687,7 @@ static bool usedInOneFunc(const User *U, Function const *&oneFunc) {
|
||||
* 3. Is the global variable referenced only in one function?
|
||||
*/
|
||||
static bool canDemoteGlobalVar(const GlobalVariable *gv, Function const *&f) {
|
||||
if (gv->hasInternalLinkage() == false)
|
||||
if (!gv->hasInternalLinkage())
|
||||
return false;
|
||||
const PointerType *Pty = gv->getType();
|
||||
if (Pty->getAddressSpace() != llvm::ADDRESS_SPACE_SHARED)
|
||||
@ -696,7 +696,7 @@ static bool canDemoteGlobalVar(const GlobalVariable *gv, Function const *&f) {
|
||||
const Function *oneFunc = nullptr;
|
||||
|
||||
bool flag = usedInOneFunc(gv, oneFunc);
|
||||
if (flag == false)
|
||||
if (!flag)
|
||||
return false;
|
||||
if (!oneFunc)
|
||||
return false;
|
||||
@ -1472,7 +1472,7 @@ void NVPTXAsmPrinter::emitFunctionParamList(const Function *F, raw_ostream &O) {
|
||||
}
|
||||
}
|
||||
|
||||
if (PAL.hasAttribute(paramIndex + 1, Attribute::ByVal) == false) {
|
||||
if (!PAL.hasAttribute(paramIndex + 1, Attribute::ByVal)) {
|
||||
if (Ty->isAggregateType() || Ty->isVectorTy()) {
|
||||
// Just print .param .align <a> .b8 .param[size];
|
||||
// <a> = PAL.getparamalignment
|
||||
|
@ -78,10 +78,7 @@ bool NVPTXDAGToDAGISel::usePrecSqrtF32() const {
|
||||
return UsePrecSqrtF32;
|
||||
} else {
|
||||
// Otherwise, use sqrt.approx if fast math is enabled
|
||||
if (TM.Options.UnsafeFPMath)
|
||||
return false;
|
||||
else
|
||||
return true;
|
||||
return !TM.Options.UnsafeFPMath;
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -930,7 +930,7 @@ NVPTXTargetLowering::getPrototype(Type *retTy, const ArgListTy &Args,
|
||||
}
|
||||
first = false;
|
||||
|
||||
if (Outs[OIdx].Flags.isByVal() == false) {
|
||||
if (!Outs[OIdx].Flags.isByVal()) {
|
||||
if (Ty->isAggregateType() || Ty->isVectorTy()) {
|
||||
unsigned align = 0;
|
||||
const CallInst *CallI = cast<CallInst>(CS->getInstruction());
|
||||
@ -1075,7 +1075,7 @@ SDValue NVPTXTargetLowering::LowerCall(TargetLowering::CallLoweringInfo &CLI,
|
||||
EVT VT = Outs[OIdx].VT;
|
||||
Type *Ty = Args[i].Ty;
|
||||
|
||||
if (Outs[OIdx].Flags.isByVal() == false) {
|
||||
if (!Outs[OIdx].Flags.isByVal()) {
|
||||
if (Ty->isAggregateType()) {
|
||||
// aggregate
|
||||
SmallVector<EVT, 16> vtparts;
|
||||
@ -1459,7 +1459,7 @@ SDValue NVPTXTargetLowering::LowerCall(TargetLowering::CallLoweringInfo &CLI,
|
||||
ObjectVT) == NumElts &&
|
||||
"Vector was not scalarized");
|
||||
unsigned sz = EltVT.getSizeInBits();
|
||||
bool needTruncate = sz < 8 ? true : false;
|
||||
bool needTruncate = sz < 8;
|
||||
|
||||
if (NumElts == 1) {
|
||||
// Just a simple load
|
||||
@ -1577,7 +1577,7 @@ SDValue NVPTXTargetLowering::LowerCall(TargetLowering::CallLoweringInfo &CLI,
|
||||
for (unsigned i = 0, e = Ins.size(); i != e; ++i) {
|
||||
unsigned sz = VTs[i].getSizeInBits();
|
||||
unsigned AlignI = GreatestCommonDivisor64(RetAlign, Offsets[i]);
|
||||
bool needTruncate = sz < 8 ? true : false;
|
||||
bool needTruncate = sz < 8;
|
||||
if (VTs[i].isInteger() && (sz < 8))
|
||||
sz = 8;
|
||||
|
||||
@ -2116,7 +2116,7 @@ SDValue NVPTXTargetLowering::LowerFormalArguments(
|
||||
// to newly created nodes. The SDNodes for params have to
|
||||
// appear in the same order as their order of appearance
|
||||
// in the original function. "idx+1" holds that order.
|
||||
if (PAL.hasAttribute(i + 1, Attribute::ByVal) == false) {
|
||||
if (!PAL.hasAttribute(i + 1, Attribute::ByVal)) {
|
||||
if (Ty->isAggregateType()) {
|
||||
SmallVector<EVT, 16> vtparts;
|
||||
SmallVector<uint64_t, 16> offsets;
|
||||
|
@ -293,12 +293,9 @@ bool llvm::isKernelFunction(const Function &F) {
|
||||
unsigned x = 0;
|
||||
bool retval = llvm::findOneNVVMAnnotation(
|
||||
&F, llvm::PropertyAnnotationNames[llvm::PROPERTY_ISKERNEL_FUNCTION], x);
|
||||
if (retval == false) {
|
||||
if (!retval) {
|
||||
// There is no NVVM metadata, check the calling convention
|
||||
if (F.getCallingConv() == llvm::CallingConv::PTX_Kernel)
|
||||
return true;
|
||||
else
|
||||
return false;
|
||||
return F.getCallingConv() == llvm::CallingConv::PTX_Kernel;
|
||||
}
|
||||
return (x == 1);
|
||||
}
|
||||
@ -307,7 +304,7 @@ bool llvm::getAlign(const Function &F, unsigned index, unsigned &align) {
|
||||
std::vector<unsigned> Vs;
|
||||
bool retval = llvm::findAllNVVMAnnotation(
|
||||
&F, llvm::PropertyAnnotationNames[llvm::PROPERTY_ALIGN], Vs);
|
||||
if (retval == false)
|
||||
if (!retval)
|
||||
return false;
|
||||
for (int i = 0, e = Vs.size(); i < e; i++) {
|
||||
unsigned v = Vs[i];
|
||||
|
Loading…
Reference in New Issue
Block a user