This is an archive of the discontinued LLVM Phabricator instance.

Differential D36174

[AArch64] Fix a typo in isExtFreeImpl()
ClosedPublic

Authored by haicheng on Aug 1 2017, 1:37 PM.

Details

Reviewers
mcrosier
qcolombet
t.p.northover
Commits
rL309748: [AArch64] Fix a typo in isExtFreeImpl()
Summary

next => not

Diff Detail

Repository
rL LLVM

Revision Contents

PathSize
lib/
Target/
AArch64/
2 lines

Diff 109195

lib/Target/AArch64/AArch64ISelLowering.cpp

  • This file is larger than 256 KB, so syntax highlighting is disabled by default.
Show First 20 Lines • Show All 7,415 Lines • ▼ Show 20 Lines return (VT1.isSimple() && !VT1.isVector() && VT1.isInteger() &&
VT2.isSimple() && !VT2.isVector() && VT2.isInteger() && VT2.isSimple() && !VT2.isVector() && VT2.isInteger() &&
VT1.getSizeInBits() <= 32); VT1.getSizeInBits() <= 32);
} }
bool AArch64TargetLowering::isExtFreeImpl(const Instruction *Ext) const { bool AArch64TargetLowering::isExtFreeImpl(const Instruction *Ext) const {
if (isa<FPExtInst>(Ext)) if (isa<FPExtInst>(Ext))
return false; return false;
// Vector types are next free. // Vector types are not free.
if (Ext->getType()->isVectorTy()) if (Ext->getType()->isVectorTy())
return false; return false;
for (const Use &U : Ext->uses()) { for (const Use &U : Ext->uses()) {
// The extension is free if we can fold it with a left shift in an // The extension is free if we can fold it with a left shift in an
// addressing mode or an arithmetic operation: add, sub, and cmp. // addressing mode or an arithmetic operation: add, sub, and cmp.
// Is there a shift? // Is there a shift?
▲ Show 20 Lines • Show All 3,402 Lines • Show Last 20 Lines