int64_tまたはuint64_tオペランドを使用した乗算演算がCでオーバーフローした場合に、効率的で移植性のあるチェック方法はありますか?
たとえば、uint64_tを追加するには、次のようにします。
if (UINT64_MAX - a < b) overflow_detected();
else sum = a + b;
しかし、私は乗算のための同様の単純な式に到達することはできません。
私に起こるのは、オペランドを高いuint32_t部分と低いuint32_t部分に分割し、オーバーフローをチェックしながらそれらの部分の乗算を実行することだけです。これは本当に醜く、おそらく非効率的です。
アップデート1:いくつかのアプローチを実装するいくつかのベンチマークコードが追加されました
アップデート2:JensGustedtメソッドが追加されました
ベンチマークプログラム:
#include <stdio.h>
#include <stdint.h>
#include <stdlib.h>
#define N 100000000
int d = 2;
#define POW_2_64 ((double)(1 << 31) * (double)(1 << 31) * 4)
#define calc_b (a + c)
// #define calc_b (a + d)
int main(int argc, char *argv[]) {
uint64_t a;
uint64_t c = 0;
int o = 0;
int opt;
if (argc != 2) exit(1);
opt = atoi(argv[1]);
switch (opt) {
case 1: /* faked check, just for timing */
for (a = 0; a < N; a++) {
uint64_t b = a + c;
if (c > a) o++;
c += b * a;
}
break;
case 2: /* using division */
for (a = 0; a < N; a++) {
uint64_t b = a + c;
if (b && (a > UINT64_MAX / b)) o++;
c += b * a;
}
break;
case 3: /* using floating point, unreliable */
for (a = 0; a < N; a++) {
uint64_t b = a + c;
if ((double)UINT64_MAX < (double)a * (double)b) o++;
c += b * a;
}
break;
case 4: /* using floating point and division for difficult cases */
for (a = 0; a < N; a++) {
uint64_t b = a + c;
double m = (double)a * (double)b;
if ( ((double)(~(uint64_t)(0xffffffff)) < m ) &&
( (POW_2_64 < m) ||
( b &&
(a > UINT64_MAX / b) ) ) ) o++;
c += b * a;
}
break;
case 5: /* Jens Gustedt method */
for (a = 0; a < N; a++) {
uint64_t b = a + c;
uint64_t a1, b1;
if (a > b) { a1 = a; b1 = b; }
else { a1 = b; b1 = a; }
if (b1 > 0xffffffff) o++;
else {
uint64_t a1l = (a1 & 0xffffffff) * b1;
uint64_t a1h = (a1 >> 32) * b1 + (a1l >> 32);
if (a1h >> 32) o++;
}
c += b1 * a1;
}
break;
default:
exit(2);
}
printf("c: %lu, o: %u\n", c, o);
}
これまでのところ、浮動小数点を使用してほとんどのケースをフィルタリングするケース4は、オーバーフローが非常に異常であると想定される場合に最速です。少なくとも、何もしない場合よりも2倍遅いだけの私のコンピューターではそうです。
ケース5は4よりも30%遅いですが、常に同じように実行されます。4の場合のように、処理を遅くする必要のある特別なケース番号はありません。