Commit 3c627b37 authored by Milad Fa's avatar Milad Fa Committed by V8 LUCI CQ

PPC/s390: [compiler] Teach InstructionScheduler about protected memory accesses

Port e301d71f

Original Commit Message:

    Because these instructions can trap, we don't want them to be reordered
    as freely as unprotected accesses.

    As part of this, make explicit which opcodes support a MemoryAccessMode.

R=neis@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com
BUG=
LOG=N

Change-Id: I122a53a67c9d2b3b99c5c25395064b61969483a4
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3181539Reviewed-by: 's avatarGeorg Neis <neis@chromium.org>
Reviewed-by: 's avatarJunliang Yan <junyan@redhat.com>
Commit-Queue: Milad Fa <mfarazma@redhat.com>
Cr-Commit-Position: refs/heads/main@{#77046}
parent ec4a3124
...@@ -11,7 +11,12 @@ namespace compiler { ...@@ -11,7 +11,12 @@ namespace compiler {
// PPC-specific opcodes that specify which assembly sequence to emit. // PPC-specific opcodes that specify which assembly sequence to emit.
// Most opcodes specify a single instruction. // Most opcodes specify a single instruction.
// Opcodes that support a MemoryAccessMode.
#define TARGET_ARCH_OPCODE_WITH_MEMORY_ACCESS_MODE_LIST(V) // None.
#define TARGET_ARCH_OPCODE_LIST(V) \ #define TARGET_ARCH_OPCODE_LIST(V) \
TARGET_ARCH_OPCODE_WITH_MEMORY_ACCESS_MODE_LIST(V) \
V(PPC_Peek) \ V(PPC_Peek) \
V(PPC_Sync) \ V(PPC_Sync) \
V(PPC_And) \ V(PPC_And) \
......
...@@ -11,7 +11,12 @@ namespace compiler { ...@@ -11,7 +11,12 @@ namespace compiler {
// S390-specific opcodes that specify which assembly sequence to emit. // S390-specific opcodes that specify which assembly sequence to emit.
// Most opcodes specify a single instruction. // Most opcodes specify a single instruction.
// Opcodes that support a MemoryAccessMode.
#define TARGET_ARCH_OPCODE_WITH_MEMORY_ACCESS_MODE_LIST(V) // None.
#define TARGET_ARCH_OPCODE_LIST(V) \ #define TARGET_ARCH_OPCODE_LIST(V) \
TARGET_ARCH_OPCODE_WITH_MEMORY_ACCESS_MODE_LIST(V) \
V(S390_Peek) \ V(S390_Peek) \
V(S390_Abs32) \ V(S390_Abs32) \
V(S390_Abs64) \ V(S390_Abs64) \
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment