[PATCH] D116157: [AArch64] Adding "armv8.8-a" memcpy/memset support.

[Tomas Matheson via Phabricator via llvm-commits]

tmatheson added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrFormats.td:11455
+  let DecoderMethod = "DecodeCPYMemOpInstruction";
+  let hasNoSchedulingInfo = 1;
+}
----------------
dmgreen wrote:
> Should this be hasNoSchedulingInfo? Can we remove this or include something very basic?
> 
> Should something here be marking them MayLoad and MayStore? They may already be hasSideEffects, as nothing marks them otherwise and they do not have codegen patterns, which is more strict. But it's probably good to be explicit.
> Should this be hasNoSchedulingInfo? Can we remove this or include something very basic?

I'm not sure why this is set, @simon_tatham might remember. My guess would be that the alternative is adding Sched<[]> but that doesn't seem to work on the multiclass - CodeGenSchedModels::checkCompleteness() does not recognise the final defs as having the Sched subclass.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116157/new/

https://reviews.llvm.org/D116157