diff options
author | Vedant Kumar <vsk@apple.com> | 2018-07-06 20:17:42 +0000 |
---|---|---|
committer | Vedant Kumar <vsk@apple.com> | 2018-07-06 20:17:42 +0000 |
commit | b3091da3af7911bc0c6f59d44d215a84c9625213 (patch) | |
tree | bd2eebfa82d3168bbfaf75575a80b604815d31d0 /llvm/lib/CodeGen/CodeGenPrepare.cpp | |
parent | 2bd02db943adcb56e9fa90f316dac6036eae18c8 (diff) | |
download | llvm-b3091da3af7911bc0c6f59d44d215a84c9625213.zip llvm-b3091da3af7911bc0c6f59d44d215a84c9625213.tar.gz llvm-b3091da3af7911bc0c6f59d44d215a84c9625213.tar.bz2 |
Use Type::isIntOrPtrTy where possible, NFC
It's a bit neater to write T.isIntOrPtrTy() over `T.isIntegerTy() ||
T.isPointerTy()`.
I used Python's re.sub with this regex to update users:
r'([\w.\->()]+)isIntegerTy\(\)\s*\|\|\s*\1isPointerTy\(\)'
llvm-svn: 336462
Diffstat (limited to 'llvm/lib/CodeGen/CodeGenPrepare.cpp')
-rw-r--r-- | llvm/lib/CodeGen/CodeGenPrepare.cpp | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/llvm/lib/CodeGen/CodeGenPrepare.cpp b/llvm/lib/CodeGen/CodeGenPrepare.cpp index dd9e2b5..3c95539 100644 --- a/llvm/lib/CodeGen/CodeGenPrepare.cpp +++ b/llvm/lib/CodeGen/CodeGenPrepare.cpp @@ -3235,7 +3235,7 @@ static bool MightBeFoldableInst(Instruction *I) { // Don't touch identity bitcasts. if (I->getType() == I->getOperand(0)->getType()) return false; - return I->getType()->isPointerTy() || I->getType()->isIntegerTy(); + return I->getType()->isIntOrPtrTy(); case Instruction::PtrToInt: // PtrToInt is always a noop, as we know that the int type is pointer sized. return true; @@ -3723,8 +3723,7 @@ bool AddressingModeMatcher::matchOperationAddr(User *AddrInst, unsigned Opcode, case Instruction::BitCast: // BitCast is always a noop, and we can handle it as long as it is // int->int or pointer->pointer (we don't want int<->fp or something). - if ((AddrInst->getOperand(0)->getType()->isPointerTy() || - AddrInst->getOperand(0)->getType()->isIntegerTy()) && + if (AddrInst->getOperand(0)->getType()->isIntOrPtrTy() && // Don't touch identity bitcasts. These were probably put here by LSR, // and we don't want to mess around with them. Assume it knows what it // is doing. @@ -5349,8 +5348,7 @@ bool CodeGenPrepare::optimizeExtUses(Instruction *I) { // x = phi x1', x2' // y = and x, 0xff bool CodeGenPrepare::optimizeLoadExt(LoadInst *Load) { - if (!Load->isSimple() || - !(Load->getType()->isIntegerTy() || Load->getType()->isPointerTy())) + if (!Load->isSimple() || !Load->getType()->isIntOrPtrTy()) return false; // Skip loads we've already transformed. |