[llvm] r321008 - [Hexagon] Higher versions of HVX imply presence of lower versions

[Krzysztof Parzyszek via llvm-commits]

Author: kparzysz
Date: Mon Dec 18 10:51:57 2017
New Revision: 321008

URL: http://llvm.org/viewvc/llvm-project?rev=321008&view=rev
Log:
[Hexagon] Higher versions of HVX imply presence of lower versions

The code in Hexagon_MC::completeHVXFeatures wasn't setting all HVX-
related features correctly.

Modified:
    llvm/trunk/lib/Target/Hexagon/MCTargetDesc/HexagonMCTargetDesc.cpp

Modified: llvm/trunk/lib/Target/Hexagon/MCTargetDesc/HexagonMCTargetDesc.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/MCTargetDesc/HexagonMCTargetDesc.cpp?rev=321008&r1=321007&r2=321008&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Hexagon/MCTargetDesc/HexagonMCTargetDesc.cpp (original)
+++ llvm/trunk/lib/Target/Hexagon/MCTargetDesc/HexagonMCTargetDesc.cpp Mon Dec 18 10:51:57 2017
@@ -363,14 +363,14 @@ FeatureBitset Hexagon_MC::completeHVXFea
 
   // HasHvxVer is false, and UseHvx is true.
   switch (CpuArch) {
-    case ArchV60:
-      FB.set(ExtensionHVXV60);
-      break;
-    case ArchV62:
-      FB.set(ExtensionHVXV62);
-      break;
     case ArchV65:
       FB.set(ExtensionHVXV65);
+      LLVM_FALLTHROUGH;
+    case ArchV62:
+      FB.set(ExtensionHVXV62);
+      LLVM_FALLTHROUGH;
+    case ArchV60:
+      FB.set(ExtensionHVXV60);
       break;
   }
   return FB;