This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGe1da3297d253: [flang][driver] Add support for the
"-init-only" option (authored by stuartellis, committed by
awarzynski).
Changed prior to commit:
stuartellis updated this revision to Diff 349307.
stuartellis added a comment.
Address review comments
Renaming InitializationOnly to InitOnly
Add missing line in init-only.f90
Updating commit message with init-only/test-io comparison
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACT
awarzynski added a comment.
Hi @stuartellis ,
Thank you for preparing this! This looks good to me, but I would appreciate a
comment somewhere (e.g. in the commit message) comparing `-init-only` and
`-test-io`. Essentially, both options are for testing purposes only. IMHO, we
should only requir
AMDChirag added a comment.
LGTM. I'll let someone else take a look too.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102849/new/
https://reviews.llvm.org/D102849
___
cfe-commits mailing list
cfe-commits
stuartellis created this revision.
Herald added a reviewer: sscalpone.
Herald added a subscriber: dang.
Herald added a reviewer: awarzynski.
stuartellis requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Adding the -init-only option and corres
