shchenz added a comment. In D100630#2697732 <https://reviews.llvm.org/D100630#2697732>, @dblaikie wrote:
> In D100630#2697728 <https://reviews.llvm.org/D100630#2697728>, @shchenz wrote: > >> In D100630#2694681 <https://reviews.llvm.org/D100630#2694681>, @probinson >> wrote: >> >>> If DBX is going to be really pedantic about not recognizing tags or >>> attributes that don't align with the DWARF version, maybe we're better off >>> with really supporting `-gstrict-dwarf` and just have DBX tuning imply that. >> >> Hi @probinson thanks for pointing out the way, agree with this new solution. >> After checking the codes, I found there are already `-gstrict-dwarf`, >> `-gno-strict-dwarf` options in clang code base, but there seem no users of >> them. Do you happen to know the story of these two options? I get a quick >> search, no clue yet. Can we reuse these options for the intention here. >> Thanks? > > I'd suggest checking the history of the commits that added them - but likely > they were added for command line compatibility with gcc but I expect they're > currently no-ops in Clang. Adding the expected functionality to them seems > fine to me. Yes, it should be for compatibility with gcc. I found same options in gcc. I create patch D100809 <https://reviews.llvm.org/D100809> for this support in clang. Could you please help to review? Thanks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100630/new/ https://reviews.llvm.org/D100630 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
