stevewan marked 3 inline comments as done. stevewan added inline comments.
================ Comment at: clang/lib/Driver/ToolChains/AIX.cpp:68 + if (Inputs.size() != 1) + llvm_unreachable("Invalid number of input files."); + const InputInfo &II = Inputs; ---------------- hubert.reinterpretcast wrote: > `llvm_unreachable` is not the right solution if this can be reached by "user > error". We should produce a proper error message. One risky condition I can think of is the user passes in multiple assembly sources to the driver, which may lead to multiple assembler inputs. To verify how the driver handles such a case, I added a new test into `aix-as.c` below, whose results suggested that this is okay because the driver would invoke `as` for each and every input files respectively. Looking into the code, the driver would construct an action list for each input files individually, which again matches the behaviour we observed in the testing results. That said, I believe the `llvm_unreachable` here is indeed not reachable by "user errors" like this, and if it triggers, it's likly an internal error. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69620/new/ https://reviews.llvm.org/D69620 _______________________________________________ cfe-commits mailing list firstname.lastname@example.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits