> -----Original Message-----
> From: Prathamesh Kulkarni <prathame...@nvidia.com>
> Sent: 26 December 2024 22:36
> To: gcc-patches@gcc.gnu.org; ja...@redhat.com; Thomas Schwinge
> <tschwi...@baylibre.com>
> Subject: PR118200: note_simd_array_uses crashes in SIMT region with
> offloading to nvptx
>
> External email: Use caution opening links or attachments
>
>
> Hi,
> As mentioned in PR, note_simd_array_uses (and
> adjust_simduid_builtins), crash while processing GOMP internal
> functions (GOMP_SIMD_VF) in SIMT region.
> This happens because in SIMT region, first argument to GOMP internal
> function is lhs of call to IFN_GOMP_SIMT_ENTER, rather than default
> definition, which results in segfault while accessing SSA_NAME_VAR
> (gimple_call_arg (stmt, 0)), where arg is simduid.
>
> The attached patch simply skips processing of simduid if it's not
> default def (which'd indicate it's in SIMT region), which seems to fix
> the issue.
> I am assuming that'd be OK (instead of walking SSA use-def chain to
> obtain default def and corresponding simduid var) since SIMT region
> would be dead-code on host side ? On nvptx side, the builtins get
> folded in ompdevlow pass itself.
>
> Patch passes libgomp testing on AArch64/nvptx offloading, and
> bootstrap+test on aarch64-linux-gnu.
> Does it look OK ?
Hi,
ping: https://gcc.gnu.org/pipermail/gcc-patches/2024-December/672269.html
Thanks,
Prathamesh
>
> Signed-off-by: Prathamesh Kulkarni <prathame...@nvidia.com>
>
> Thanks,
> Prathamesh
>
>
