• @scrion
    link
    19
    edit-2
    5 months ago

    This comment would be much better if it featured an explanation of your concerns with that particular opt setting.

    • @[email protected]
      link
      fedilink
      85 months ago

      There is no deep explanation or anything.

      Release profile defaults to -O3. -Oz compromises runtime performance for binary size.

      When you start seeing for example network services which are not going to be running in constrained environments, nor do they restrict themselves from using heap allocations and all, when you see them set -Oz in their release profiles. Then you start to see small beginner projects setting it like OP here. You start to wonder if it’s a pattern, and the settings are being copied from somewhere else, with potential misconceptions like thinking it can meaningfully reduce runtime memory usage or something.

      I actually think I found the culprit, thanks to one of the projects I was hinting at mentioning it:

      https://github.com/johnthagen/min-sized-rust

      Looks like we have a second wave of net-negative common wisdom in the ecosystem, the first being optimizing for dependencies’ compile times. But this one is not nearly as bad, as it doesn’t affect libraries.