Bump the C stack pointer when building Dart frames to maintain the ARM64 ABI requirements without g…

runtime/vm/constants_arm64.h

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #ifndef VM_CONSTANTS_ARM64_H_
 #define VM_CONSTANTS_ARM64_H_
 
 #include "platform/assert.h"
 
 namespace dart {
@@ skipping 978 matching lines @@
     //  Add/sub (extended) when not setting the flags.
     //  Logical (immediate) when not setting the flags.
     // Otherwise, R31 is the ZR.
     if (IsAddSubImmOp() || (IsAddSubShiftExtOp() && IsExtend())) {
       if (HasS()) {
         return R31IsZR;
       } else {
         return R31IsSP;
       }
     }
-    // TODO(zra): Handle for logical immediate operations.
+    if (IsLogicalImmOp()) {
+      const int op = Bits(29, 2);
+      const bool set_flags = op == 3;
+      if (set_flags) {
+        return R31IsZR;
+      } else {
+        return R31IsSP;
+      }
+    }
     return R31IsZR;
   }
 
   // Indicate whether Rn can be CSP or ZR. This does not check that the
   // instruction actually has an Rn field.
   R31Type RnMode() const {
     // The following instructions use CSP as Rn:
     //  All loads and stores.
     //  Add/sub (immediate).
     //  Add/sub (extended).
@@ skipping 80 matching lines @@
   static Instr* At(uword pc) { return reinterpret_cast<Instr*>(pc); }
 
  private:
   DISALLOW_ALLOCATION();
   DISALLOW_IMPLICIT_CONSTRUCTORS(Instr);
 };
 
 }  // namespace dart
 
 #endif  // VM_CONSTANTS_ARM64_H_