ilinpv added a comment. In D127812#4085171 <https://reviews.llvm.org/D127812#4085171>, @tmatheson wrote:
> This patch has made it considerably harder to understand what is going on in > the TargetParser. If you get a chance, please could you add some clarifying > comments and tidy-ups. I appreciate that a lot of this is following the lead > of the pre-existing TargetParser code, but lets try to improve it as we go. I fully agree, thank you for valuable comments! Let me address them in separate patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127812/new/ https://reviews.llvm.org/D127812 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits