https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102138
--- Comment #10 from Drea Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Drea Pinski from comment #9)
> (In reply to Drea Pinski from comment #8)
> > Created attachment 64839 [details]
> > Work in progress
> >
> > This is my version of the patch; it is still a works in progress. It does
> > not handle the case where a edge is no-longer excutable which we could use
> > some fake value for it; maybe 0.
> >
> >
> > It also can be slightly more relaxed when accepting a ssa name. Right now
> > the definition of the ssa name needs not to be in the same bb and the use
> > stmt needs to be in the same bb.
> >
> > I wonder if we could also relax the single use case. One other thing is we
> > do this even for all phis args are the same which we don't need to handle as
> > we are already going to prop that anyways.
>
>
> Looks like I am running into a go front-end issue.
>
> So the phi was:
> SR.2529_103 = PHI
> <&imt..
> interface_4Info_bfunc_8_9_8io_1fs_0FileInfo_3error_9_cIsDir_bfunc_8_9_8bool_9
> _cModTime_bfunc_8_9_8time_0Time_9_cMode_bfunc_8_9_8io_1fs_0FileMode_9_cName_b
> func_8_9_8string_9_cSize_bfunc_8_9_8int64_9_cSys_bfunc_8_9_8interface_4_5_9_c
> Type_bfunc_8_9_8io_1fs_0FileMode_9_5..archive_1zip.headerFileInfo(16),
> &pimt..
> interface_4Info_bfunc_8_9_8io_1fs_0FileInfo_3error_9_cIsDir_bfunc_8_9_8bool_9
> _cModTime_bfunc_8_9_8time_0Time_9_cMode_bfunc_8_9_8io_1fs_0FileMode_9_cName_b
> func_8_9_8string_9_cSize_bfunc_8_9_8int64_9_cSys_bfunc_8_9_8interface_4_5_9_c
> Type_bfunc_8_9_8io_1fs_0FileMode_9_5..archive_1zip.fileListEntry(17)>
>
> The use stmt is:
> _37 = SR.2529_103->__type_descriptor
>
> So combing those, and we get a read from a const memory and we get back:
> &archive/zip.headerFileInfo..d
>
> BUT the ADDRESSABLE bit was not set:
> /home/apinski/src/upstream-gcc/gcc/libgo/go/archive/zip/reader.go: In
> function ‘archive/zip.openDir.ReadDir’:
> /home/apinski/src/upstream-gcc/gcc/libgo/go/archive/zip/reader.go:828:1:
> error: address taken, but ADDRESSABLE bit not set
> 828 | func (d *openDir) ReadDir(count int) ([]fs.DirEntry, error) {
> | ^
> PHI argument
> &archive/zip.headerFileInfo..d;
> for PHI node
>
> Or I am supposed to do something special here? Or is this a real go
> front-end bug? Where the address is in the constructor of the const but
> TREE_ADDRESSABLE was not set correctly?
Yes it does look like a go front-end issue, very similar to PR 100537.
