(original) (raw)

Seems attribute “\_\_inline\_\_” was missing in these intrinsics. Candidate patch https://reviews.llvm.org/D108509

Thanks

Pengfei

From: llvm-dev On Behalf Of Craig Topper via llvm-dev
Sent: Friday, August 20, 2021 11:32 PM
To: Umesh Kalappa
Cc: LLVM Developers Mailing List
Subject: Re: \[llvm-dev\] Intel tile unused warnings .

Aren’t the intrinsics usually marked inline? Would that prevent the warning?

On Fri, Aug 20, 2021 at 3:04 AM Umesh Kalappa via llvm-dev <llvm-dev@lists.llvm.org> wrote:

Hi Devs,

We think the following function definitions should be attributed with
"unused" to avoid the below warnings .

include/amxintrin.h:255:13: warning: unused function '\_\_tile\_loadd'
\[-Wunused-function\]
static void \_\_tile\_loadd(\_\_tile1024i \*dst, const void \*base,
^
/include/amxintrin.h:261:13: warning: unused function '\_\_tile\_dpbssd'
\[-Wunused-function\]
static void \_\_tile\_dpbssd(\_\_tile1024i \*dst, \_\_tile1024i src1,
^
include/amxintrin.h:268:13: warning: unused function '\_\_tile\_stored'
\[-Wunused-function\]
static void \_\_tile\_stored(void \*base, \_\_SIZE\_TYPE\_\_ stride, \_\_tile1024i src) {

include/amxintrin.h:273:13: warning: unused function '\_\_tile\_zero'
\[-Wunused-function\]

We can make the required changes and upstream the same .

Thank you
\~Umesh
\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_
LLVM Developers mailing list
llvm-dev@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev

\~Craig