On Fri, Apr 22, 2011 at 12:59:21AM -0400, Jason Merrill wrote: > On 04/21/2011 10:55 PM, Nathan Froyd wrote: >> On Thu, Apr 21, 2011 at 10:49:05PM -0400, Jason Merrill wrote: >>> Hunh. How does that work? They fill in CASE_LABEL later? Can that be >>> changed? >> >> Yeah, tree-eh.c:lower_try_finally_switch. I don't know how easy it is >> to fix; it certainly looks non-trivial. > > Well, I tried adjusting it and regression testing seems fine so far. I > can't think what the comment would be talking about with pointers not > providing a stable order; I don't see anything that would rely on > that.
Based on discussion downthread, I plan to commit something like your patch (I think `label' is unused after this, so requires trivial changes) on your behalf tomorrow, unless you beat me to it or unless somebody yells. I'd rather have this not mixed up with the rest of the build_case_label changes. -Nathan