diff options
| author | Marco Nelissen <marcone@google.com> | 2019-07-15 14:59:10 -0700 |
|---|---|---|
| committer | Marco Nelissen <marcone@google.com> | 2019-07-15 14:59:10 -0700 |
| commit | c7189aee5a66a077372f06301ee08c6ccad645eb (patch) | |
| tree | d579d5d513b9d7c234ec0d5a68afdb11b13726c6 /arm-wt-22k | |
| parent | 3d94279769f4117670127844e8b5a6864130a70e (diff) | |
| download | sonivox-c7189aee5a66a077372f06301ee08c6ccad645eb.tar.gz | |
Work around an issue in the coverage build
The coverage build is 32-bit and overrides the compiler optimzation
level to -O0. At -O0, the 1024 in __builtin_mul_overflow(var, 1024, &dst) is
inferred to be a 128-bit type, which then causes __muloti4 to be used, which
doesn't exist on 32-bit builds, causing linker failure.
Bug: 137542431
Test: still works on non-coverage build
Change-Id: If714b629cd1c0e73e9ff95249106c2f890bcb9ed
Diffstat (limited to 'arm-wt-22k')
| -rw-r--r-- | arm-wt-22k/lib_src/eas_smf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arm-wt-22k/lib_src/eas_smf.c b/arm-wt-22k/lib_src/eas_smf.c index b690de1..96ce167 100644 --- a/arm-wt-22k/lib_src/eas_smf.c +++ b/arm-wt-22k/lib_src/eas_smf.c @@ -877,7 +877,7 @@ static EAS_RESULT SMF_ParseMetaEvent (S_EAS_DATA *pEASData, S_SMF_DATA *pSMFData { // pSMFData->tickConv = (EAS_U16) (((temp * 1024) / pSMFData->ppqn + 500) / 1000); uint64_t temp64; - if (__builtin_mul_overflow(temp, 1024, &temp64) || + if (__builtin_mul_overflow(temp, 1024u, &temp64) || pSMFData->ppqn == 0 || (temp64 /= pSMFData->ppqn, false) || __builtin_add_overflow(temp64, 500, &temp64) || |