summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKrzysztof Parzyszek <kparzysz@codeaurora.org>2017-05-02 17:58:52 +0000
committerKrzysztof Parzyszek <kparzysz@codeaurora.org>2017-05-02 17:58:52 +0000
commit57f5046b4af5893a8d598f9d9043ce8c47b58bcb (patch)
treea742fb58bab0bd17288d361ca18042fc3b2e21ce
parentc15f8d2a082173006f183b41d28eb228978f1d07 (diff)
downloadbcm5719-llvm-57f5046b4af5893a8d598f9d9043ce8c47b58bcb.tar.gz
bcm5719-llvm-57f5046b4af5893a8d598f9d9043ce8c47b58bcb.zip
[Hexagon] Remove unneeded code from HexagonShuffler
Patch by Colin LeMahieu. llvm-svn: 301947
-rw-r--r--llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.cpp16
-rw-r--r--llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.h4
2 files changed, 9 insertions, 11 deletions
diff --git a/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.cpp b/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.cpp
index eb303464555..4350c199e26 100644
--- a/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.cpp
+++ b/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.cpp
@@ -273,8 +273,8 @@ bool HexagonShuffler::check() {
unsigned memory = 0, loads = 0, load0 = 0, stores = 0, store0 = 0, store1 = 0;
// Number of HVX loads, HVX stores.
unsigned CVIloads = 0, CVIstores = 0;
- // Number of duplex insns, solo insns.
- unsigned duplex = 0, solo = 0;
+ // Number of duplex insns
+ unsigned duplex = 0;
// Number of insns restricting other insns in the packet to A and X types,
// which is neither A or X types.
unsigned onlyAX = 0, neitherAnorX = 0;
@@ -295,12 +295,10 @@ bool HexagonShuffler::check() {
for (iterator ISJ = begin(); ISJ != end(); ++ISJ) {
MCInst const &ID = ISJ->getDesc();
- if (HexagonMCInstrInfo::isSolo(MCII, ID))
- solo++;
- else if (HexagonMCInstrInfo::isSoloAX(MCII, ID))
- onlyAX++;
+ if (HexagonMCInstrInfo::isSoloAX(MCII, ID))
+ ++onlyAX;
else if (HexagonMCInstrInfo::isSoloAin1(MCII, ID))
- onlyAin1++;
+ ++onlyAin1;
neitherAnorX += countNeitherAnorX(MCII, ID);
if (HexagonMCInstrInfo::prefersSlot3(MCII, ID)) {
++pSlot3Cnt;
@@ -409,8 +407,8 @@ bool HexagonShuffler::check() {
}
// Check if the packet is legal.
- if ((load0 > 1 || store0 > 1 || CVIloads > 1 || CVIstores > 1) ||
- (duplex > 1 || (duplex && memory)) || (solo && size() > 1) ||
+ if ((load0 > 1 || store0 > 1) ||
+ (duplex > 1 || (duplex && memory)) ||
(onlyAX && neitherAnorX > 1) || (onlyAX && xtypeFloat)) {
reportError(llvm::Twine("invalid instruction packet"));
return false;
diff --git a/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.h b/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.h
index bd31c7be4c0..10a959008f4 100644
--- a/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.h
+++ b/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonShuffler.h
@@ -105,8 +105,8 @@ class HexagonInstr {
public:
HexagonInstr(HexagonCVIResource::TypeUnitsAndLanes *T,
MCInstrInfo const &MCII, MCInst const *id,
- MCInst const *Extender, unsigned s, bool x = false)
- : ID(id), Extender(Extender), Core(s), CVI(T, MCII, s, id) {}
+ MCInst const *Extender, unsigned s)
+ : ID(id), Extender(Extender), Core(s), CVI(T, MCII, s, id) {};
MCInst const &getDesc() const { return *ID; };
OpenPOWER on IntegriCloud