Re: [PATCH] make ggc pick up comp_dir_string() cache value. (dwarf2out.c)

2019-03-22 Thread Otto, Thomas
> > And I still think this function and the static variable which never changes > > once set does not require any GC. Just setting the cached_wd variable to > > the unchanged pointer from get_src_pwd() or allocating one in the function > > itself is enough. This solves the problem and relieves th

Re: [PATCH] make ggc pick up comp_dir_string() cache value. (dwarf2out.c)

2019-03-21 Thread Otto, Thomas
> > > > > > "ggc_collect() discarding/reusing remap_debug_filename() output, > > > > > > thus producing invalid objects" > > > > > > > > > > Hmm, but AFAICS it can end up on the heap if plain get_src_pwd () > > > > > result survives. I vaguely remember GC being happy with heap > > > > > strings (d

Re: [PATCH] make ggc pick up comp_dir_string() cache value. (dwarf2out.c)

2019-03-20 Thread Otto, Thomas
> > > > "ggc_collect() discarding/reusing remap_debug_filename() output, > > > > thus producing invalid objects" > > > > > > Hmm, but AFAICS it can end up on the heap if plain get_src_pwd () > > > result survives. I vaguely remember GC being happy with heap > > > strings (due to identifiers?), but

Re: [PATCH] make ggc pick up comp_dir_string() cache value. (dwarf2out.c)

2019-03-20 Thread Otto, Thomas
> > "ggc_collect() discarding/reusing remap_debug_filename() output, thus > > producing invalid objects" > > Hmm, but AFAICS it can end up on the heap if plain get_src_pwd () result > survives. I vaguely remember GC being happy with heap strings (due to > identifiers?), but not sure. Otherwise t

[PATCH] make ggc pick up comp_dir_string() cache value. (dwarf2out.c)

2019-03-20 Thread Otto, Thomas
See the corresponding thread on gcc-help: "ggc_collect() discarding/reusing remap_debug_filename() output, thus producing invalid objects" Regards Thomas make-ggc-pick-up-comp_dir_string-cache-value.patch Description: make-ggc-pick-up-comp_dir_string-cache-value.patch