SjoerdMeijer 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) \ ---------------- dmgreen wrote: > 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. Ah yeah, thanks Dave. These are the tag setting variants, I didn't look careful enough, then probably got confused about the ACLE which I agree must be wrong. Would be good to double check that first. 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