[PATCH] D67608: [ARM] Preserve fpu behaviour for '-crypto'

This revision was automatically updated to reflect the committed changes. Closed by commit rG2cb43b45713d: [ARM] Preserve fpu behaviour for -crypto (authored by dnsampaio). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67608/new/

[PATCH] D67608: [ARM] Preserve fpu behaviour for '-crypto'

peter.smith accepted this revision. peter.smith added a comment. This revision is now accepted and ready to land. Thanks for the update. That looks good to me. Will be good to wait for a day before committing to give US timezone a chance to object. Repository: rG LLVM Github Monorepo

[PATCH] D67608: [ARM] Preserve fpu behaviour for '-crypto'

dnsampaio updated this revision to Diff 224323. dnsampaio added a comment. Attending review request: - Fixed to only add '-crypto' when not passing -fno-integrated-as - Fixed test to use arm-none-none-eabi - Changed comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D67608: [ARM] Preserve fpu behaviour for '-crypto'

peter.smith added a comment. After retesting on a failing example and experimenting a bit, I think that we should only preserve +crypto with -fno-integrated-as. It would also be good to mention D66018 and maybe add the original author as a reviewer.

[PATCH] D67608: [ARM] Preserve fpu behaviour for '-crypto'

dnsampaio created this revision. dnsampaio added reviewers: peter.smith, labrinea. Herald added subscribers: cfe-commits, dmgreen, kristof.beyls. Herald added a project: clang. This patch restores the behaviour that -fpu overwrites the architecture obtained from -march or -mcpu flags, not