[PATCH] D52483: [AArch64][v8.5A] Add speculation barriers SSBB and PSSBB

[Oliver Stannard via Phabricator via llvm-commits]

olista01 added inline comments.


================
Comment at: lib/Target/AArch64/AArch64InstrInfo.td:648-649
 def : InstAlias<"isb", (ISB 0xf)>;
+def : InstAlias<"ssbb", (DSB 0)>;
+def : InstAlias<"pssbb", (DSB 4)>;
 
----------------
t.p.northover wrote:
> I think these should have `Predicates` set since not all CPUs support them.
This is deliberate, these instructions have been retroactively added to v8.0-A onwards. This works because they are aliases for "DSB #0" and "DSB #4", which older specs say have to behave like "DSB SY", which has stronger semantics than SSBB and PSSBB. This means that existing cores already implement these, though in a more costly way than v8.5-A allows.


Repository:
  rL LLVM

https://reviews.llvm.org/D52483