On 07/16/2018 05:10 PM, Tom de Vries wrote: > On 07/16/2018 03:50 PM, Richard Biener wrote: >> On Mon, 16 Jul 2018, Tom de Vries wrote: >>> Any comments? >> >> Interesting idea. I wonder if that should be generalized >> to other places > > I kept the option name general, to allow for that. > > And indeed, this is a point-fix patch. I've been playing around with a > more generic patch that adds nops such that each is_stmt .loc is > associated with a unique insn, but that was embedded in an > fkeep-vars-live branch, so this patch is minimally addressing the first > problem I managed to reproduce on trunk. > >> and how we can avoid compare-debug failures >> (var-tracking usually doesn't change code-generation). >> >
I'll post this patch series (the current state of my fkeep-vars-live branch) in reply to this email: 1 [debug] Add fdebug-nops 2 [debug] Add fkeep-vars-live 3 [debug] Add fdebug-nops and fkeep-vars-live to Og only Bootstrapped and reg-tested on x86_64. ChangeLog entries and function header comments missing. Comments welcome. Thanks, - Tom