[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-16 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski added a comment. Apologies, this is my fault. Should be fixed: https://github.com/llvm/llvm-project/commit/46446e398b5202303280d9e18676c07529f1d0b0. I pushed it without a review as it felt rather straightforward and I wanted to make sure that you are unblocked. If for whatever reaso

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-16 Thread Pete Steinfeld via Phabricator via cfe-commits
PeteSteinfeld added a comment. For what it's worth, here's my cmake command that results in a successful static full build: cmake -G Ninja ../llvm \ -DCMAKE_BUILD_TYPE=Release \ -DLLVM_LIT_ARGS=-v \ -DFLANG_ENABLE_WERROR=On \ -DCMAKE_C_COMPILER=/home/sw/thirdparty/gcc/gcc-9.3.0

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-16 Thread Peter Klausler via Phabricator via cfe-commits
klausler added a comment. No; I'm hitting this problem with an in-tree static build of llvm-project/main. Weird. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104305/new/ https://reviews.llvm.org/D104305 _

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-16 Thread Pete Steinfeld via Phabricator via cfe-commits
PeteSteinfeld added a comment. In D104305#2822589 , @klausler wrote: > The non-shared-library build of the current llvm-project/main tree fails on > this new test; it looks as if gfortran is being invoked and passed an option > that it does not recogniz

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-16 Thread Peter Klausler via Phabricator via cfe-commits
klausler added a comment. The non-shared-library build of the current llvm-project/main tree fails on this new test; it looks as if gfortran is being invoked and passed an option that it does not recognize. The shared library build passes all tests. Repository: rG LLVM Github Monorepo CHAN

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-16 Thread Andrzej Warzynski via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGa6be6e31f181: [flang][driver] Add `-fdebug-dump-all` (authored by awarzynski). Changed prior to commit: https://reviews.llvm.org/D104305?vs=352234&id=352364#toc Repository: rG LLVM Github Monorepo C

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-15 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski updated this revision to Diff 352234. awarzynski added a comment. Add missing test (forgot to add when making the submission) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104305/new/ https://reviews.llvm.org/D104305 Files: clang/incl

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-15 Thread Pete Steinfeld via Phabricator via cfe-commits
PeteSteinfeld accepted this revision. PeteSteinfeld added a comment. This revision is now accepted and ready to land. Looks great! Thanks for doing this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104305/new/ https://reviews.llvm.org/D104305 _

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-15 Thread Pete Steinfeld via Phabricator via cfe-commits
PeteSteinfeld added a comment. Stay tuned a little, I think I spotted the problem ... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104305/new/ https://reviews.llvm.org/D104305 ___ cfe-commits mailing li

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-15 Thread Pete Steinfeld via Phabricator via cfe-commits
PeteSteinfeld added a comment. Thanks for doing this. These changes look good, but I can't get them to build successfully. I'm using the GNU 9.3 C++ compiler. My build fails to link the bin/flang-new executable. Here's an excerpt of the error messages: [4436/4439] Generating ../../../../i

[PATCH] D104305: [flang][driver] Add `-fdebug-dump-all`

2021-06-15 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski created this revision. Herald added a reviewer: sscalpone. Herald added a subscriber: dang. awarzynski requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The new option will run the semantic checks and then dump the parse tree and a