=?utf-8?q?Iñaki?= Amatria Barral <inaki.amat...@appentra.com> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/130...@github.com>
https://github.com/DavidTruby approved this pull request. This LGTM now when @tarunprabhu is happy, thanks for seeing this through despite my lengthy comments! This isn't to hold up this patch, but as a separate thought: I wonder if there's even any reason to have `types::TY_PP_Fortran` and `types::TY_Fortran`? Do we ever actually treat them differently..? https://github.com/llvm/llvm-project/pull/130268 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
