[PATCH] D96884: [flang][driver] Add more -fdebug options

2021-02-19 Thread Faris Rehman via Phabricator via cfe-commits
FarisRehman updated this revision to Diff 324926. FarisRehman added a comment. Remove -fdebug-parsing-log Remove `-fdebug-parsing-log` from this patch as it requires additional work that can be implemented in a separate patch. Also address review comments by @awarzynski Repository: rG LLVM G

[PATCH] D96884: [flang][driver] Add more -fdebug options

2021-02-18 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski added a comment. I've just realized that for `-fdebug-parsing-log` we need to set `Fortran::parser::instrumentedParse`. I don't see a good place for that just yet. It's probably best extracting that option into a separate patch. Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D96884: [flang][driver] Add more -fdebug options

2021-02-18 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski accepted this revision. awarzynski added a comment. This revision is now accepted and ready to land. Thank you for working on this @FarisRehman ! This semantics implemented here are identical to what `f18` does. `-fdebug-instrumented-parse` from `f18` becomes `-fdebug-parsing-log` in

[PATCH] D96884: [flang][driver] Add more -fdebug options

2021-02-18 Thread Faris Rehman via Phabricator via cfe-commits
FarisRehman updated this revision to Diff 324590. FarisRehman added a comment. Rebase off main Rebase off main and add a comment to MeasurementVisitor. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96884/new/ https://reviews.llvm.org/D96884 Files

[PATCH] D96884: [flang][driver] Add more -fdebug options

2021-02-17 Thread Faris Rehman via Phabricator via cfe-commits
FarisRehman created this revision. Herald added subscribers: jansvoboda11, dang, mgorny. Herald added a reviewer: sscalpone. Herald added a reviewer: awarzynski. FarisRehman requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1. Heral