Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5e72d9ce authored by Cindy Zhou's avatar Cindy Zhou
Browse files

Enable cfi in hevc library 

Bug: 158010610
Bug: 173497308
Test: ran atest DecoderConformanceTest, ImageReaderDecoderTest,
VideoDecoderPerfTest, VideoEncoderTest, CodecDecoderSurfaceTest and VideoEncoderDecoderTest.
Compare the before and after enabling CFI results since there was a number of tests
that were initially failing.

Change-Id: I763f717590704d860babd27336a077c76fb5d743
parent a0969954

media/codec2/components/hevc/Android.bp

+2 −0
Original line number Diff line number Diff line
@@ -3,6 +3,7 @@ cc_library { 
    defaults: [

        "libcodec2_soft-defaults",

        "libcodec2_soft_sanitize_signed-defaults",

        "libcodec2_soft_sanitize_cfi-defaults",

    ],



    srcs: ["C2SoftHevcDec.cpp"],
@@ -16,6 +17,7 @@ cc_library { 
    defaults: [

        "libcodec2_soft-defaults",

        "libcodec2_soft_sanitize_signed-defaults",

        "libcodec2_soft_sanitize_cfi-defaults",

    ],



    srcs: ["C2SoftHevcEnc.cpp"],

media/libstagefright/codecs/hevcdec/Android.bp

+3 −0
Original line number Diff line number Diff line
@@ -17,6 +17,9 @@ cc_library_shared { 
            "signed-integer-overflow",

        ],

        cfi: true,

        config: {

            cfi_assembly_support: true,

        },

    },



    // We need this because the current asm generates the following link error: