[PATCH] D137329: [flang] Add -f[no-]associative-math and -mreassociate

2022-11-06 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski added a comment. In D137329#3910249 , @kiranchandramohan wrote: > In D137329#3909943 , @awarzynski > wrote: > >> In D137329#3909082 , @clementval >> wrote: >

[PATCH] D137329: [flang] Add -f[no-]associative-math and -mreassociate

2022-11-05 Thread Kiran Chandramohan via Phabricator via cfe-commits
kiranchandramohan added a comment. In D137329#3909943 , @awarzynski wrote: > In D137329#3909082 , @clementval > wrote: > >> Wouldn't it be good to have a RFC for all these options and what they will >> do in Fl

[PATCH] D137329: [flang] Add -f[no-]associative-math and -mreassociate

2022-11-05 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski added a comment. In D137329#3909082 , @clementval wrote: > Wouldn't it be good to have a RFC for all these options and what they will do > in Flang instead of just adding them all? Or did I miss the RFC? +1 Repository: rG LLVM Github Mon

[PATCH] D137329: [flang] Add -f[no-]associative-math and -mreassociate

2022-11-04 Thread Valentin Clement via Phabricator via cfe-commits
clementval added a comment. Herald added a subscriber: jdoerfert. Wouldn't it be good to have a RFC for all these options and what they will do in Flang instead of just adding them all? Or did I miss the RFC? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[PATCH] D137329: [flang] Add -f[no-]associative-math and -mreassociate

2022-11-04 Thread Tom Eccles via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGc4dc3c029416: [flang] Add -f[no-]associative-math and -mreassociate (authored by tblah). Herald added projects: clang, Flang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo