[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-12 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof created this revision. Herald added a reviewer: sscalpone. Herald added a project: All. alexiprof requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. this pull request contains a fix for the problem with displaying options -fsyntax_o

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-12 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof updated this revision to Diff 452382. alexiprof added a comment. This is commit fix flang unit tests in using help command Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 Files: flang/test/Dri

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-12 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof updated this revision to Diff 452383. alexiprof added a comment. Fix revision Differential Revision: https://reviews.llvm.org/D131808 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 Files: cl

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-13 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof updated this revision to Diff 452393. alexiprof added a comment. fix tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 Files: clang/docs/ClangCommandLineReference.rst clang/docs/UsersMan

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-13 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof added a comment. In D131808#3720910 , @awarzynski wrote: > Thanks, mostly makes sense! https://github.com/llvm/llvm-project/issues/57033 > mentions `-O{n}` as well :) Could you fix that too? More comments inline. > > CI is still failing :( Are

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-13 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof updated this revision to Diff 452405. alexiprof added a comment. Set new text hint for option fsyntax-only Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 Files: clang/docs/ClangCommandLineRef

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-13 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof added inline comments. Comment at: clang/include/clang/Driver/Options.td:2798-2799 def fsyntax_only : Flag<["-"], "fsyntax-only">, - Flags<[NoXarchOption,CoreOption,CC1Option,FC1Option]>, Group; + Flags<[NoXarchOption,CoreOption,CC1Option,FC1Option,FlangOption]>, Gr

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-13 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof updated this revision to Diff 452429. alexiprof added a comment. this pull request contains a fix for the problem with displaying options -fsyntax_only in clang and fland-new in help https://github.com/llvm/llvm-project/issues/57033 Before: $ clang -help | grep syntax -objcmt-mi

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-13 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof updated this revision to Diff 452431. alexiprof added a comment. remove Unrelated change Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 Files: clang/docs/ClangCommandLineReference.rst clang

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-13 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof added a comment. I do not have commit access, i do not have merge :-( Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 ___ cfe-commits mailing list cfe

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-14 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof updated this revision to Diff 452487. alexiprof added a comment. replase text hint Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 Files: clang/docs/ClangCommandLineReference.rst clang/docs/

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-15 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof added a comment. I can't merge the changes(( Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 ___ cfe-commits mailing list cfe-commits@lists.llvm.org h

[PATCH] D131808: [clang,flang] Add missing options fsyntax-only in help

2022-08-15 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof added a comment. Could someone land this patch please? Alexander Malkov Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 ___ cfe-commits mailing list

[PATCH] D131808: [clang,flang] Add help text for -fsyntax-only

2022-08-19 Thread Alexander Malkov via Phabricator via cfe-commits
alexiprof added a comment. ping @awarzynski @MaskRay @vzakhari Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131808/new/ https://reviews.llvm.org/D131808 ___ cfe-commits mailing list cfe-commits@lists.ll