Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

<complex>: avoid unnecessary conversion of real arguments to complex #2855

Merged
merged 3 commits into from
Jul 14, 2022
Merged

<complex>: avoid unnecessary conversion of real arguments to complex #2855

merged 3 commits into from
Jul 14, 2022

Commits on Jul 11, 2022

  1. Swap arguments in commutative mixed complex/real operations to avoid … 

    …unnecessary conversion of the real argument to complex.
    @MattStephanson
    MattStephanson committed Jul 11, 2022
    Configuration menu
    Copy the full SHA
    c55dad6 View commit details
    Browse the repository at this point in the history

  2. review feedback

    @MattStephanson
    MattStephanson committed Jul 11, 2022
    Configuration menu
    Copy the full SHA
    b3c2dbc View commit details
    Browse the repository at this point in the history

Commits on Jul 12, 2022

  1. Call op= for complex - T and complex / T. 

    This transformation doesn't assume commutativity; operand order
is preserved. Like the changes for `complex + T` and `complex * T`,
this replaces 2 and 4 function calls with 1.
    @StephanTLavavej
    StephanTLavavej committed Jul 12, 2022
    Configuration menu
    Copy the full SHA
    89f9054 View commit details
    Browse the repository at this point in the history