[ovs-dev] [PATCH 2/2] cccl: Enable compiler optimization by default.

[Gurucharan Shetty]

On Thu, Aug 28, 2014 at 10:45 AM, Eitan Eliahu <eliahue at vmware.com> wrote:
>
> Hi Guru,
> Do we distinguish between Debug and Releases builds? ( I recall we discussed this before but forgot what was the outcome)
> Enabling optimization may cause some issues for the debugger to match the source to the code.
We currently do not have a way to do that. I currently remove the
"/O2" flag manually from cccl file to debug. (I do the same in
Makefile for Linux, i.e., manually remove it)

> Thanks,
> Eitan
>
> -----Original Message-----
> From: dev [mailto:dev-bounces at openvswitch.org] On Behalf Of Gurucharan Shetty
> Sent: Thursday, August 28, 2014 9:39 AM
> To: dev at openvswitch.org
> Cc: Gurucharan Shetty
> Subject: [ovs-dev] [PATCH 2/2] cccl: Enable compiler optimization by default.
>
> There is no 1-1 mapping between gcc's compiler optimization flags and MSVC's optimization flags making the automatic conversions trickier.
>
> This commit adds the /O2 option which is the recommended option for fast code.
>
> Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
> ---
>  build-aux/cccl |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/build-aux/cccl b/build-aux/cccl index c18629a..eb4d2aa 100644
> --- a/build-aux/cccl
> +++ b/build-aux/cccl
> @@ -45,7 +45,7 @@ esac
>  # We'll assume cl to start out
>  prog=cl
>  # opts specifies the command line to pass to the MSVC program -clopt="${slash}nologo ${slash}FS"
> +clopt="${slash}nologo ${slash}FS ${slash}O2"
>  linkopt="${slash}nologo"
>  # gotparam is 0 if we didn't ever see a param, in which case we show usage()  gotparam=
> --
> 1.7.9.5
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://urldefense.proofpoint.com/v1/url?u=http://openvswitch.org/mailman/listinfo/dev&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=yTvML8OxA42Jb6ViHe7fUXbvPVOYDPVq87w43doxtlY%3D%0A&m=rCJ17Ge41i0k52k%2Flw%2Fe5lhNCL6sMVLGKVunJDqNJn8%3D%0A&s=da8438d4f8f35a860f10ab4bf59394e2a762c6b932b13335d006568bca58d399