Enable optimization on bpf2c generated code

[Alan-Jowett]

Description

Enable optimization flag when generating release versions of native binaries.

Testing

CI/CD

Documentation

No.

Installation

No.

[shpalani]

Can you help clarify the significance of the below options?

[shpalani]

Can you help clarify the significance of the below options?

 <ClCompile>
      <Optimization>MaxSpeed</Optimization>
      <FunctionLevelLinking>true</FunctionLevelLinking>
      <IntrinsicFunctions>true</IntrinsicFunctions>
      <BasicRuntimeChecks>Default</BasicRuntimeChecks>
    </ClCompile>

This will also help to know why is BasicRuntimeChecks not used in the kernel mode. Thanks.

[Alan-Jowett]

Can you help clarify the significance of the below options?

 <ClCompile>
      <Optimization>MaxSpeed</Optimization>
      <FunctionLevelLinking>true</FunctionLevelLinking>
      <IntrinsicFunctions>true</IntrinsicFunctions>
      <BasicRuntimeChecks>Default</BasicRuntimeChecks>
    </ClCompile>

This will also help to know why is BasicRuntimeChecks not used in the kernel mode. Thanks.

BasicRuntimeChecks are not supported in kernel mode.

The full list of options for this configuration node are here:
https://learn.microsoft.com/en-us/visualstudio/msbuild/cl-task?view=vs-2022

[shpalani]

Can you help clarify the significance of the below options?

 <ClCompile>
      <Optimization>MaxSpeed</Optimization>
      <FunctionLevelLinking>true</FunctionLevelLinking>
      <IntrinsicFunctions>true</IntrinsicFunctions>
      <BasicRuntimeChecks>Default</BasicRuntimeChecks>
    </ClCompile>

This will also help to know why is BasicRuntimeChecks not used in the kernel mode. Thanks.

BasicRuntimeChecks are not supported in kernel mode.

The full list of options for this configuration node are here: https://learn.microsoft.com/en-us/visualstudio/msbuild/cl-task?view=vs-2022

Thanks for sharing the link. Going through the list of options, I found the below to be useful too!
Optimization -- Full - /Ox
SmallerTypeCheck -- true
WholeProgramOptimization - true

[dthaler]

This if tests whether the unit tests were compiled for Debug or Release, not whether bpf.sys was compiled for Debug or Release. As such, this test will fail in some manual cases. It would be better to accept both sizes, not only one size.