dang added a comment. In D121936#3391001 <https://reviews.llvm.org/D121936#3391001>, @zixuw wrote:
> In D121936#3390920 <https://reviews.llvm.org/D121936#3390920>, @dang wrote: > >> In D121936#3389991 <https://reviews.llvm.org/D121936#3389991>, @ributzka >> wrote: >> >>> Nice! Thank you for adding support for multiple headers. Is this a step >>> towards the json input file list? >> >> My understanding was that for this iteration at least we were going with >> just feeding the headers directly to clang without an input file list. Maybe >> we should talk about this offline as I might be missing something. > > In my initial proposed design, clang-extract-api takes in multiple headers > directly as inputs. What would be the need to have a json file list? It might be better if we need to pass in information about specific headers. We can discuss more offline but I don't think it is necessary for this iteration at least. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121936/new/ https://reviews.llvm.org/D121936 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
