> From: Kumar, Venkataramanan <[email protected]> > Sent: Sunday, July 12, 2026 8:13 PM > > From: Haochen Jiang <[email protected]> > > Sent: Thursday, July 2, 2026 12:19 PM > > To: [email protected] > > > > There are several insts that will be reused from legacy AMX, coming from > AMX- > > TILE and AMX-AVX512. > > > > Since legacy AMX and ACE should not be used together, we choose not to > use > > the same intrin name for ACE and legacy AMX, while the builtins could be > > shared. > > > > For AMX-TILE related insts, we add "_ace_" in the middle of the intrin name > to > > explicitly mention the intrin is used by ACE. > > Ok with adding "_ace_". Note the ACE specification document mentions > intrinsic names without "_ace_". > Should that be changed, hoping other compiler like LLVM adhere to same > naming convention?
Forget to reply this topic. Yes, that should be changed. And not hoping, we need to make sure LLVM is the same as us. When we are implementing, we have synced with our LLVM folks on those names and we have reached the agreement. The only thing we need to do is to tell AMD LLVM folks to align on all the intrin names. I suppose in the review process, our folks should have mentioned this. If not, we need to remind them. Thx, Haochen
