A64: Fix assertion in MacroAssembler::CallApiFunctionAndReturn.

src/a64/macro-assembler-a64.cc

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 1322 matching lines @@
       ExternalReference::handle_scope_next_address(isolate());
   const int kNextOffset = 0;
   const int kLimitOffset = AddressOffset(
       ExternalReference::handle_scope_limit_address(isolate()),
       next_address);
   const int kLevelOffset = AddressOffset(
       ExternalReference::handle_scope_level_address(isolate()),
       next_address);
 
   ASSERT(function_address.Is(x3));
-  ASSERT(!AreAliased(function_address, thunk_last_arg, x1, x2));
+  ASSERT(thunk_last_arg.is(x1) || thunk_last_arg.is(x2));

[ulan, 2014/02/10 16:38:51]

I took this assert from the ARM port, which also uses r1 as a scratch.

   // TODO(all): Why do we care about aliasing x2? (This function uses x1 as a
   // scratch regiser.)
 
   // TODO(all): Why isn't thunk_last_arg used?
   USE(thunk_last_arg);
 
   Label profiler_disabled;
   Label end_profiler_check;
   bool* is_profiling_flag = isolate()->cpu_profiler()->is_profiling_address();
   STATIC_ASSERT(sizeof(*is_profiling_flag) == 1);
@@ skipping 3451 matching lines @@
   }
 }
 
 
 #undef __
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_TARGET_ARCH_A64