Needs to treat RET as a branch when we add dmb fences
authorPeizhao Ou <peizhaoo@uci.edu>
Tue, 3 Apr 2018 23:04:13 +0000 (16:04 -0700)
committerPeizhao Ou <peizhaoo@uci.edu>
Tue, 3 Apr 2018 23:04:13 +0000 (16:04 -0700)
lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp

index b783b15d988045b5373693fc5ee253ac73165999..cae2105bcf648064fa57b3f8b00f0407d2dba43f 100644 (file)
@@ -1600,8 +1600,6 @@ static bool isLoad(MachineInstr* MI) {
 static bool isBranch(MachineInstr* MI) {
   switch (MI->getOpcode()) {
     default: { return false; }
-    case AArch64::B:
-    case AArch64::BR:
     case AArch64::Bcc:
     case AArch64::CBZW:
     case AArch64::CBZX:
@@ -1611,11 +1609,11 @@ static bool isBranch(MachineInstr* MI) {
     case AArch64::TBZX:
     case AArch64::TBNZW:
     case AArch64::TBNZX:
-    case AArch64::BLR:
+    case AArch64::B:
+    case AArch64::BR:
     case AArch64::BL:
-//    case AArch64::BX:
-//    case AArch64::BLX:
-    {
+    case AArch64::BLR:
+    case AArch64::RET: {
       return true;
     }
   }