awarzynski added a comment. In D143301#4126884 <https://reviews.llvm.org/D143301#4126884>, @elmcdonough wrote:
> Thanks for the feedback everyone. This is my current understanding on what I > should do: I am to rename gfortran_unsupported_Group to flang_ignored_w_Group > and move the non W group gfortran options into another patch. I have these > changes locally and am currently building + testing. Please let me know if > I'm misinterpreting these instructions. Makes sense and sorry for confusion. Just to clarify, "w" in ` flang_ignored_w_Group` stands for "warning" :) I think that having two separate patches will simplify the discussion too ;-) And, most importantly, thanks for doing this! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143301/new/ https://reviews.llvm.org/D143301 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits