Min,

Yes to confirm with the output-mismatch, I just checked with very basic
commandline *BasketballDrive_1920x1080_50.y4m --preset ultrafast .*
 The outputs are getting mismatched in all three builds(8/10/12 builds) in
vc12-x86_64. I builded each build separately and compared.
As praveen said only binary file sizes remained same, but outputs
mismatched.

I checked on changeset-d20b78d6d138 , and applying your patch on top of
this changeset ..

Can we know at which tip you tested ? and anyother way you compared ?


Thanks,
Aasaipriya

On Fri, Sep 23, 2016 at 4:04 PM, Praveen Tiwari <
[email protected]> wrote:

> Hi Min,
>          Can you please verify for VC12 ? I double checked on this I am
> getting different output for this patch. 8-bit encoded file size is same
> but different binary (compared using beyond compare), 10 and 12 bit size
> and binary both are different. I applied you patch build once (like 8 bit
> build)  and collected all depth outputs (8, 10 and 12), compared with three
> builds of x265 i.e 8 bit, 10 bit and 12 bit.
>
> Regards,
> Praveen
>
>
> On Fri, Sep 23, 2016 at 2:47 AM, chen <[email protected]> wrote:
>
>> Hi Praveen,
>>
>> I test your cmdlind on my VS2008 build.
>> I build three bit-depth version and compare with one bit-depth version,
>> but the output are still matched in both 10 and 12 bit.
>>
>> Regards,
>> Min
>>
>> At 2016-09-22 14:39:50,"Praveen Tiwari" <[email protected]>
>> wrote:
>>
>> Hi Min,
>>
>>  After this patch outputs are changing, tested for following command line
>> for 10-bit and 12-bit outputs.
>>
>> --input=NebutaFestival_2560x1600_60_10bit_crop.yuv --input-res=2560x1600
>> --fps=60  --numa-pools="NULL" --output-depth=12 --hash=1 -o  NFOut12.hevc
>>
>>
>>
>>
>> Regards,
>> Praveen
>>
>> On Thu, Sep 15, 2016 at 1:55 AM, chen <[email protected]> wrote:
>>
>>> From ea50e494473623ed0dbff2907194aaf268dc449a Mon Sep 17 00:00:00 2001
>>> From: Min Chen <[email protected]>
>>> Date: Wed, 14 Sep 2016 15:23:38 -0500
>>> Subject: [PATCH] [multi-lib] Support 8+10+12 bits in single DLL
>>> (Workaround)
>>>
>>> ---
>>>  source/CMakeLists.txt |   40 +++++++++++++++++++++++++++++++++++++++-
>>>  1 files changed, 39 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/source/CMakeLists.txt b/source/CMakeLists.txt
>>> index dd19d28..c2c2f7f 100644
>>> --- a/source/CMakeLists.txt
>>> +++ b/source/CMakeLists.txt
>>> @@ -36,6 +36,7 @@ configure_file("${PROJECT_SOURCE_DIR}/x265.def.in"
>>>  configure_file("${PROJECT_SOURCE_DIR}/x265_config.h.in"
>>>                 "${PROJECT_BINARY_DIR}/x265_config.h")
>>>
>>> +
>>>  SET(CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/cmake"
>>> "${CMAKE_MODULE_PATH}")
>>>
>>>  # System architecture detection
>>> @@ -396,6 +397,39 @@ if(WIN32)
>>>      endif(WINXP_SUPPORT)
>>>  endif()
>>>
>>> +
>>> +if(ENABLE_SHARED AND LINKED_10BIT AND LINKED_12BIT)
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def
>>> "?setParamAspectRatio@x265@@YAXPEAUx265_param@@HH@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def
>>> "?getParamAspectRatio@x265@@YAXPEAUx265_param@@AEAH1@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?general_log_file@x265
>>> @@YAXPEBUx265_param@@PEBDH1ZZ\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?general_log@x265
>>> @@YAXPEBUx265_param@@PEBDH1ZZ\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def
>>> "?x265_api_get_94@x265_10bit@@YAPEBUx265_api@@H@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def
>>> "?x265_api_get_94@x265_12bit@@YAPEBUx265_api@@H@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def
>>> "?x265_api_query@x265_10bit@@YAPEBUx265_api@@HHPEAH@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def
>>> "?x265_api_query@x265_12bit@@YAPEBUx265_api@@HHPEAH@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?x265_mdate@x265
>>> @@YA_JXZ\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def
>>> "?x265_picturePlaneSize@x265@@YAIHHHH@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?x265_ssim2dB@x265
>>> @@YANN@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?x265_ssim2dB@x265
>>> @@YANN@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?x265_report_simd@x265
>>> @@YAXPEAUx265_param@@@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?x265_fopen@x265
>>> @@YAPEAU_iobuf@@PEBD0@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?x265_malloc@x265
>>> @@YAPEAX_K@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?x265_free@x265
>>> @@YAXPEAX@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?x265_atoi@x265
>>> @@YAHPEBDAEA_N@Z\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?start@Thread@x265@
>>> @QEAA_NXZ\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?stop@Thread@x265@
>>> @QEAAXXZ\n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "??0Thread@x265@@QEAA@XZ
>>> \n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "??1Thread@x265@@UEAA@XZ
>>> \n")
>>> +    file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?g_maxCUDepth@x265
>>> @@3IA\n")
>>> +    if(WINXP_SUPPORT)
>>> +        file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?cond_init@x265
>>> @@YAHPEAUConditionVariable@1@@Z\n")
>>> +        file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?cond_wait@x265
>>> @@YAHPEAUConditionVariable@1@PEAU_RTL_CRITICAL_SECTION@@K@Z\n")
>>> +        file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?cond_destroy@x265
>>> @@YAXPEAUConditionVariable@1@@Z\n")
>>> +        file(APPEND ${PROJECT_BINARY_DIR}/x265.def "?cond_broadcast@x265
>>> @@YAXPEAUConditionVariable@1@@Z\n")
>>> +    endif()
>>> +endif()
>>> +
>>>  include(version) # determine X265_VERSION and X265_LATEST_TAG
>>>  include_directories(. common encoder "${PROJECT_BINARY_DIR}")
>>>
>>> @@ -608,7 +642,11 @@ if(ENABLE_CLI)
>>>          if(WIN32 OR NOT ENABLE_SHARED OR INTEL_CXX)
>>>              # The CLI cannot link to the shared library on Windows, it
>>>              # requires internal APIs not exported from the DLL
>>> -            target_link_libraries(cli x265-static ${PLATFORM_LIBS})
>>> +            if(ENABLE_SHARED AND LINKED_10BIT AND LINKED_12BIT)
>>> +                target_link_libraries(cli x265-shared ${PLATFORM_LIBS})
>>> +            else()
>>> +                target_link_libraries(cli x265-static ${PLATFORM_LIBS})
>>> +            endif()
>>>          else()
>>>              target_link_libraries(cli x265-shared ${PLATFORM_LIBS})
>>>          endif()
>>> --
>>> 1.7.9.msysgit.0
>>>
>>>
>>> _______________________________________________
>>> x265-devel mailing list
>>> [email protected]
>>> https://mailman.videolan.org/listinfo/x265-devel
>>>
>>>
>>
>> _______________________________________________
>> x265-devel mailing list
>> [email protected]
>> https://mailman.videolan.org/listinfo/x265-devel
>>
>>
>
> _______________________________________________
> x265-devel mailing list
> [email protected]
> https://mailman.videolan.org/listinfo/x265-devel
>
>
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

Reply via email to