Jakub Jelinek <ja...@redhat.com> writes: >> I am not sure, but I am wondering if we shouldn't save the previous uid >> of 'stmt' here before setting it, and then restore it before getting out >> of this function. > > No, gimple uids are AFAIK undefined at the start of passes, passes that use > them are supposed to initialize them before use (new statements created > during the pass will get 0 there by default), and don't have to clean them > up anyway at the end of pass.
Yeah, this is what I figured by grepping other passes, but I wasn't sure :-) Maybe I should follow up with a doc patch for the (otherwise very terse) comment of gimple_set_uid and gimple_uid accessors. Thanks. -- Dodji