[PATCH] D96864: [flang][driver] Add -Xflang and make -test-io a frontend-only flag

2021-02-22 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski added a comment. Thanks for reviewing @SouraVX ! I did update that comment before merging. Hopefully it _will_ make sense when we come back to it at later time :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96864/new/

[PATCH] D96864: [flang][driver] Add -Xflang and make -test-io a frontend-only flag

2021-02-22 Thread Andrzej Warzynski via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGd81f633fe28f: [flang][driver] Add -Xflang and make -test-io a frontend-only flang (authored by awarzynski). Changed prior to commit: https://reviews.llvm.org/D96864?vs=324287=325405#toc Repository:

[PATCH] D96864: [flang][driver] Add -Xflang and make -test-io a frontend-only flag

2021-02-17 Thread Sourabh Singh Tomar via Phabricator via cfe-commits
SouraVX added inline comments. Comment at: flang/test/Frontend/input-output-file.f90:6 !-- +! NOTE: We need `-E` below in order to instruct the compiler driver to create a job representing a fronted compiler invocation ! TEST 1: Print to stdout

[PATCH] D96864: [flang][driver] Add -Xflang and make -test-io a frontend-only flag

2021-02-17 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski added inline comments. Comment at: flang/test/Frontend/input-output-file.f90:6 !-- +! NOTE: We need `-E` below in order to instruct the compiler driver to create a job representing a fronted compiler invocation ! TEST 1: Print to stdout

[PATCH] D96864: [flang][driver] Add -Xflang and make -test-io a frontend-only flag

2021-02-17 Thread Sourabh Singh Tomar via Phabricator via cfe-commits
SouraVX added inline comments. Comment at: flang/test/Frontend/input-output-file.f90:6 !-- +! NOTE: We need `-E` below in order to instruct the compiler driver to create a job representing a fronted compiler invocation ! TEST 1: Print to stdout

[PATCH] D96864: [flang][driver] Add -Xflang and make -test-io a frontend-only flag

2021-02-17 Thread Sourabh Singh Tomar via Phabricator via cfe-commits
SouraVX added a comment. Thanks! for the patch @awarzynski . I support this enhancement. LGTM(Not accepting formally :) ), Let's wait and hear from others also, what's their take on this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96864/new/