On Fri, Jan 15, 2021 at 3:24 PM Yonghong Song <yhs@xxxxxx> wrote: > > > > On 1/15/21 1:53 PM, Sedat Dilek wrote: > > En plus, I encountered breakage with GCC v10.2.1 and LLVM=1 and > > CONFIG_DEBUG_INFO_DWARF4. > > So might be good to add a "depends on !DEBUG_INFO_BTF" in this combination. Can you privately send me your configs that repro? Maybe I can isolate it to a set of configs? > > I suggested not to add !DEBUG_INFO_BTF to CONFIG_DEBUG_INFO_DWARF4. > It is not there before and adding this may suddenly break some users. > > If certain combination of gcc/llvm does not work for > CONFIG_DEBUG_INFO_DWARF4 with pahole, this is a bug bpf community > should fix. Is there a place I should report bugs? > > > > > I had some other small nits commented in the single patches. > > > > As requested in your previous patch-series, feel free to add my: > > > > Tested-by: Sedat Dilek <sedat.dilek@xxxxxxxxx> Yeah, I'll keep it if v6 is just commit message changes. -- Thanks, ~Nick Desaulniers
