https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94629

--- Comment #23 from Andrew Stubbs <ams at gcc dot gnu.org> ---
(In reply to Jakub Jelinek from comment #12)
> (In reply to Andrew Stubbs from comment #11)
> > (In reply to Jakub Jelinek from comment #10)
> > > or if instead we should drop the "status = " for the cases where nothing
> > > checks it. Andrew?
> > 
> > I think checking the status is probably good practice, even though I don't
> > think there's an actual bug here.
> 
> I have no way to test it, so if you want and can test it (perhaps with other
> changes), please take it over, or we throw it away, we don't have to fix all
> the warnings, we just should skim through it to find if there aren't any
> actual bugs.

I think this is done now. The patch needed to be changed a little because
HSA_STATUS_INFO_BREAK is also a valid return value (meaning that the iterator
exited early).

Reply via email to