почему погромисты считают что инструкция goto в c++ очень очень плохая

Потому, что ты можешь написать кучу кода, а потом такой GOTO и выкинуть исполнение вообще хрен знает куда.

потому что она разрушает структуру кода, усложняет его поддержку и увеличивает риск ошибок.

потому что с goto код становится как лапша у бабки на кухне вообще хрен разберёшься

Не вписывается ни в функциональную, ни в объектоориентированную парадигму

потому

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

Потому, что избыточное использование goto уменьшает надёжность кода и затрудняет его понимание / модификацию. Использование goto провоцирует написание спагетти-кода, в котором очень сложно разбираться.

В языке C (без плюсов) goto может использоваться для обработки ошибок, но в C++ есть структурные исключения (try-catch), которые делают бессмысленным использование goto в этих целях.

Единственное, где goto имеет смысл использовать - это имитация многоуровневого break (завершение сразу нескольких вложенных циклов), отсутствующего в C/C++. Но намного лучше обернуть эти несколько циклов в функцию и делать return: от разбиения большого монолита на небольшие функции код становится куда понятнее.

P.S. Чаще всего goto используется не в коде, написанном человеком, а в коде, созданном всяческими генераторами по заданному набору правил. Программу, генерирующую код с goto, написать намного проще, чем программу, генерирующую код без goto.

В качестве примера просто посмотри, сколько goto в модуле парсинга даты/времени в исходниках PHP:

• Вот это - шаблон, описывающий структуру и способы обработки даты/времени:
  https://github.com/php/php-src/blob/master/ext/date/lib/parse_date.re
• А вот это - готовый C-код, автоматически сгенерированный по шаблону: https://github.com/php/php-src/blob/master/ext/date/lib/parse_date.c

Ну так сказал Дейкстра.
Ну а компилятор превратит все их старания в машинный код, состоящий чуть менее чем полностью из goto, точнее, инструкций условных и безусловных переходов процессора.

Она не плохая, она просто не нужна.
Можно использовать ее в качестве оптимизации для выхода из циклов глубокой вложенности, когда один goto всяко лучше кучи if/break, но более она просто не нужна ни для чего.

C++ все же язык высокого уровня, в котором все эти низкоуровневые штучки ака goto отданы на откуп компилятору. Если компилятор ставит го-ту где нужно + добавляете вы свои, выходит как письмо Дяди Федора дописанное котом и Шариком. Смысл есть - но не понятно где чей)