[PATCH] D99501: ignore -flto= options recognized by GCC

2021-04-15 Thread Florian Hahn via Phabricator via cfe-commits
fhahn added a comment. In D99501#2689090 , @tejohnson wrote: > Just approved the test case patch. Sorry I missed the lack of test on this! Great, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99501/

[PATCH] D99501: ignore -flto= options recognized by GCC

2021-04-14 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson added a comment. Just approved the test case patch. Sorry I missed the lack of test on this! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99501/new/ https://reviews.llvm.org/D99501 ___ cfe-com

[PATCH] D99501: ignore -flto= options recognized by GCC

2021-04-14 Thread Matthias Klose via Phabricator via cfe-commits
doko added a comment. that's now https://reviews.llvm.org/D100484 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99501/new/ https://reviews.llvm.org/D99501 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D99501: ignore -flto= options recognized by GCC

2021-04-09 Thread Florian Hahn via Phabricator via cfe-commits
fhahn added a comment. In D99501#2679394 , @doko wrote: > here in the same place, or in a different merge request? This one is closed now I think, so a new review would be ideal. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:/

[PATCH] D99501: ignore -flto= options recognized by GCC

2021-04-09 Thread Matthias Klose via Phabricator via cfe-commits
doko added a comment. here in the same place, or in a different merge request? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99501/new/ https://reviews.llvm.org/D99501 ___ cfe-commits mailing list cfe-co

[PATCH] D99501: ignore -flto= options recognized by GCC

2021-04-08 Thread Florian Hahn via Phabricator via cfe-commits
fhahn added a comment. In D99501#2668774 , @lebedev.ri wrote: > Tests missing. > This somehow wasn't sent to cfe-commits list @doko could you add a test? e.g. in `clang/test/Driver/clang_f_opts.c`, which already contains a check for `-ffat-lto-objects`

[PATCH] D99501: ignore -flto= options recognized by GCC

2021-04-05 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. Tests missing. This somehow wasn't sent to cfe-commits list Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99501/new/ https://reviews.llvm.org/D99501 ___ cfe-commits mailing li

[PATCH] D99501: ignore -flto= options recognized by GCC

2021-04-05 Thread Sylvestre Ledru via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG162848654842: ignore -flto= options recognized by GCC (authored by sylvestre.ledru). Herald added a project: clang. Herald added a subscriber: cfe-co