richard.barton.arm added a comment. I think the behaviour for missing flang is fine for now, and I think we can improve on it later on. We ought to codify (if it is not done already) where flang looks for tools to exec, because I think PATH is probably not the only place it could look to (directory of clang binary, other relative paths, other environment variables and commandline options, etc.) The new test will need revising once we get there.
I think this patch is looking good to be committed - what do you think @hfinkel ? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63607/new/ https://reviews.llvm.org/D63607 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
