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).