On Tue, Oct 12, 2021 at 5:21 PM Martin Liška <mli...@suse.cz> wrote: > > On 10/11/21 15:45, Richard Biener wrote: > > Btw, I'd be more comfortable when the move of the code would be > > independent of the adjustment to not rely on AUTODETECT_VALUE. > > Can we do the latter change first (IIRC the former one failed already)? > > All right, so I'm doing the first step by eliminating AUTODETECT_VALUE. > Note we can't easily use EnabledBy, the option logic is more complicated > (like optimize >= 1). > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed?
Let's split this ;) The debug_inline_points part is OK. How can debug_variable_location_views be ever -1? But the debug_variable_location_views part looks OK as well. More or less all parts that have the variable assigned in a single place in gcc/ are OK (dwarf2out_as_locview_support). But the main flag_var_tracking* cases need more thorough view, maybe we can convert them to single-set code first? Richard. > Thanks, > Martin
