FarisRehman marked 2 inline comments as done. FarisRehman added inline comments.
================ Comment at: flang/test/Flang-Driver/include-header.f90:38 +!----------------------------------------------------- +! EXPECTED OUTPUT FOR /Inputs/ FOLDER SPECIFIED FIRST +!----------------------------------------------------- ---------------- sameeranjoshi wrote: > Nit - Shouldn't this be `Inputs/` starting with a `/` changes the meaning(at > least on Linux systems) ? I've fixed this, thanks. ================ Comment at: flang/test/Flang-Driver/include-header.f90:59 +#endif +end ---------------- tskeith wrote: > `-I` also is supposed to affect INCLUDE lines so it would be good to have a > test for that too. They are handled during preprocessing and so can be tested > the same way. > > It also affects searching for .mod files but I think to test that requires > semantics (i.e. to report an error if the module can't be found). It would be > good to test that somewhere too. Thanks, I've updated the test to check INCLUDE lines and added a new test for module files. I've added some `.mod` files to the Inputs directory, however these files are not real module files and are just there for the test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93453/new/ https://reviews.llvm.org/D93453 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits