On 8/13/21 12:15 PM, Bill Schmidt wrote: > Honestly, I don't see how it matters. So far as I can tell, all you've done > here is hand-inlined what build_simple_mem_ref would do. So I guess I have > a slight preference for your original patch (but with the new test case, > of course).
Ok, I ended up pushing the original patch then with the expanded test case. I'll let this bake on trunk for a bit before back porting. Thanks everyone. Peter