aganea added a comment. In D99363#2653476 <https://reviews.llvm.org/D99363#2653476>, @abhina.sreeskantharajan wrote:
> There were a lot of similar patches so reverting all of them might be more > work than isolating the change that caused it and reverting that. It seems > that the patch you initially commented on did not contain the problematic > change since reverting the change doesn't fix your issue. I created the > following patch https://reviews.llvm.org/D99426 based on @rnk suggestion. I > created a new flag for OF_TextWithCRLF on Windows and made sure my most > recent text changes use the OF_Text flag while all other uses were changed to > OF_TextWithCRLF. This should solve any CRLF issues that were introduced > recently by my patches. If you have time, would you be able to test if that > patch fixes your issue? I've applied https://reviews.llvm.org/D99426#2656738 over rGc4d5b956170dd85941c1c2787abaa2e01575234c <https://reviews.llvm.org/rGc4d5b956170dd85941c1c2787abaa2e01575234c> but I'm still seeing the issue in https://reviews.llvm.org/D96363#2650460. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99363/new/ https://reviews.llvm.org/D99363 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
