barannikov88 added a comment. @cor3ntin I've been working on pretty much the same functionality in our downstream fork. I was not aware of the paper, nor of the ongoing work in this direction, and so I unfortunately missed the review. Thanks for this patch, it significantly reduces the number of changes downstream and makes it easier to merge with upstream in the future.
I have a couple of questions about future work: - IIUC the paper initially addressed this issue with `#line` directive, but the changes were reverted(?). Is there any chance they can get back? - Are there any plans for making similar changes to asm statement parsing? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105759/new/ https://reviews.llvm.org/D105759 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits