phyBrackets added a comment. In D125340#3505723 <https://reviews.llvm.org/D125340#3505723>, @martong wrote:
> What are the benefits of this renaming? I mean is there a name clash? Do we > have another kind of "import" in Clang or in some of the dependent projects, > don't we? AS it suggested by @balazske in the previous patch , and I also think if we moved the ImportError into it's own header then it's better to be named as ASTImportError . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125340/new/ https://reviews.llvm.org/D125340 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
