dmgreen added inline comments.

================
Comment at: clang/lib/Headers/arm_acle.h:734
+/* Memory Operations Intrinsics */
+#if __ARM_FEATURE_MOPS && __ARM_FEATURE_MEMORY_TAGGING
+#define __arm_mops_memset_tag(tagged_address, value, size)                     
\
----------------
tyb0807 wrote:
> SjoerdMeijer wrote:
> > Why does this also need MTE? I think the ACLE specifies this intrinsic to 
> > be available when __ARM_FEATURE_MOPS is defined?
> Yes you are right, thanks for spotting this.
Hmm. These map to SETMG, and those instructions require MTE. It wouldn't make 
sense to have an intrinsic that cannot be emitted to a valid instruction. I 
think the spec might be wrong, to be honest.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117753

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to