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

Commit 1fb463eb authored by Mathieu Chartier's avatar Mathieu Chartier
Browse files

Disable cdex for non background dexopt 

Avoid regressing app install until there are enough benefits to
justify the tradeoff.

Test: make
Bug: 63756964
Change-Id: I1c5b3c097a456b6c8e18c34a228122b976d62da5
parent 69acdd75

cmds/installd/dexopt.cpp

+3 −1
Original line number Diff line number Diff line
@@ -412,7 +412,9 @@ static void run_dex2oat(int zip_fd, int oat_fd, int input_vdex_fd, int output_vd 


    // Disable cdex if update input vdex is true since this combination of options is not

    // supported.

    const bool disable_cdex = input_vdex_fd == output_vdex_fd;

    // Disable cdex for non-background compiles since we don't want to regress app install until

    // there are enough benefits to justify the tradeoff.

    const bool disable_cdex = !background_job_compile || (input_vdex_fd == output_vdex_fd);



    const char* argv[9  // program name, mandatory arguments and the final NULL

                     + (have_dex2oat_isa_variant ? 1 : 0)