Опции компилятора Gw/Gy

Fakels · Регистрация: 07.03.2015

Может кто разъяснит на, что конкретно влияют опции компилятора Gw и Gy, когда их устанавливаешь одновременно обе то почему-то ниже приведенная функция возвращает "рандомно" не правильные значения.
При этом если поменять тип переменно result с double на float то ошибки при использовании опций не будет.
В чем причина такого поведения?

C++

int GetStatBonus(fo::GameObject* critter, const fo::SkillInfo* info, int skill, int points) {
    double result = 0;
    for (int i = 0; i < 7; i++) {
        result += stat_level(critter, i) * multipliers[skill * 7 + i];  // double*  multipliers 
    }
    result += points * info->skillPointMulti;
    result += info->base;
    return static_cast<int>(result);
}

Ассемблерный код функции при отключении одной из опций никак не меняется (или я не заметил различий)

Fakels · 15.01.2019, 21:37 **[ТС]**

Забыл указать компилятор - vs15

@DrOffset · 20.01.2019, 17:13

Fakels, multipliers - это глобальный указатель? Можете показать как он объявлен и как инициализируются данные, на которые он указывает?

Да, /Gw влияет на глобальные данные.

И это, учитывая, что контракты у вас не закреплены интерфейсом, а реализуются через умолчания, я бы, для успокоения души, расставил везде ассерты на эти умолчания. (я про магические числа 7 и индексацию данных через голые указатели). Оно может у вас и правильно все, но вам точно будет легче, если вы исключите UB из-за выходов за диапазон ваших массивов.

Fakels · 26.01.2019, 04:11 **[ТС]**

Сообщение от DrOffset

multipliers - это глобальный указатель? Можете показать как он объявлен и как инициализируются данные, на которые он указывает?

да

C++

1
2
3

static double* multipliers;
---
multipliers = new double[7 * fo::SKILL_count]();

Добавил nullptr к multipliers, и ошибка пропала
потом в коде asm я разобрался адрес для multipliers брался неизвестно от какого места.

Fakels 34 / 20 / 4 Регистрация: 07.03.2015 Сообщений: 325
	15.01.2019, 21:37 [ТС]	2
	Забыл указать компилятор - vs15 0

@DrOffset 14034 / 7517 / 1779 Регистрация: 30.01.2014 Сообщений: 12,570
	20.01.2019, 17:13	3
	Fakels, `multipliers` - это глобальный указатель? Можете показать как он объявлен и как инициализируются данные, на которые он указывает? Да, `/Gw` влияет на глобальные данные. И это, учитывая, что контракты у вас не закреплены интерфейсом, а реализуются через умолчания, я бы, для успокоения души, расставил везде ассерты на эти умолчания. (я про магические числа 7 и индексацию данных через голые указатели). Оно может у вас и правильно все, но вам точно будет легче, если вы исключите UB из-за выходов за диапазон ваших массивов. 0

Опции темы