Enable CSE in McLachlan

Create issue
Issue #538 new
Erik Schnetter created an issue

Kranc supports CSE, which has the potential to reduce code size and improve performance. We should enable it in McLachlan.


Comments (4)

  1. Ian Hinder
    • removed comment

    Does CSE lead to improved performance on some systems? I believe Barry found that it gave improvements for !WeylScal4 but not for !McLachlan, but I might be misremembering.

  2. Barry Wardell
    • removed comment

    Yes, when I was testing it, I found that it actually slowed !McLachlan down a bit, but did give a speedup for !WeylScal4. This was before a lot of the recent speed improvements in !McLachlan so it would be worth checking again. I don't think we should just blindly enable it, though.

    The benefits of CSE might also depend on e.g. whether Jacobians are enabled or not, how aggressive the CSE is (this can be tuned), etc.

  3. Log in to comment