diff options
| author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2016-03-02 23:00:21 +0000 |
|---|---|---|
| committer | Matt Arsenault <Matthew.Arsenault@amd.com> | 2016-03-02 23:00:21 +0000 |
| commit | 8226fc482999342e899db0d046ad044ba2ada136 (patch) | |
| tree | 4eef69b7247160e6ff76eb98a6c84b85e6767028 /llvm/lib/Target/AMDGPU/R600Packetizer.cpp | |
| parent | 283a215c7a8e185ebdd247ff13d3b79db31070f1 (diff) | |
| download | bcm5719-llvm-8226fc482999342e899db0d046ad044ba2ada136.tar.gz bcm5719-llvm-8226fc482999342e899db0d046ad044ba2ada136.zip | |
AMDGPU: Simplify boolean conditional return statements
Patch by Richard Thomson
llvm-svn: 262536
Diffstat (limited to 'llvm/lib/Target/AMDGPU/R600Packetizer.cpp')
| -rw-r--r-- | llvm/lib/Target/AMDGPU/R600Packetizer.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/llvm/lib/Target/AMDGPU/R600Packetizer.cpp b/llvm/lib/Target/AMDGPU/R600Packetizer.cpp index 5d8ce418678..964c65cd952 100644 --- a/llvm/lib/Target/AMDGPU/R600Packetizer.cpp +++ b/llvm/lib/Target/AMDGPU/R600Packetizer.cpp @@ -178,9 +178,7 @@ public: return true; // XXX: This can be removed once the packetizer properly handles all the // LDS instruction group restrictions. - if (TII->isLDSInstr(MI.getOpcode())) - return true; - return false; + return TII->isLDSInstr(MI.getOpcode()); } // isLegalToPacketizeTogether - Is it legal to packetize SUI and SUJ @@ -214,10 +212,8 @@ public: TII->definesAddressRegister(MIJ); bool ARUse = TII->usesAddressRegister(MII) || TII->usesAddressRegister(MIJ); - if (ARDef && ARUse) - return false; - return true; + return !ARDef || !ARUse; } // isLegalToPruneDependencies - Is it legal to prune dependece between SUI |
