fhossein-quic wrote: > ectorConvertRemove (EnableConversionsRemoval) and XqfPostRADiagnosis > (EnablePostRAXqfCompliance)
You're absolutely right, and I appreciate the feedback. These two passes evolved together and are pretty tightly coupled. These two passes evolved together over many iterations and are pretty tightly coupled. Each pass itself also went through several rounds of fixes and corrections internally, but I realized those intermediate commits can't be landed individually since each one could produce incorrect codegen on its own. I did pull out what I could and created multiple separate PRs for the part that could have taken out of this PR. I also pulled out VectorConvertRemove and XqfPostRADiagnosis and will upstream them as separate follow-up PRs per Santanu's suggestion. I'm open to any feedback that helps me in splitting this furthur. https://github.com/llvm/llvm-project/pull/198902 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
