Static bool isReferencingMDNode(const Instruction &I) { if (const auto

Question

Static bool isReferencingMDNode(const Instruction &I) { if (const auto

*CI = dyn_cast<CallInst>(&I))
if (Function *F = CI->getCalledFunction())
if (F->isIntrinsic())
for (auto &Op : I.operands())
if (auto *V = dyn_cast_or_null<MetadataAsValue>(Op))
if (isa<MDNode>(V->getMetadata()))
return true;
return false;
}

Как бы вы отрефакторили этот кусок?🤔

#c #embedded #programming #russian

0

25.12.2021

8 ответов

33 просмотра

abc cba

В вашем случае вот так примерно: static bool isReferencingMDNode(const Instruction &I) { if ((const auto *CI = dyn_cast<CallInst>(&I)) == nullptr) { return false; } if ((Function *F = CI->getCalledFunction()) == nullptr) { return false; } ... return true; }

0

25.12.2021

Dmitriy [Отпуск]

abc cba
Например я бы писал так: if (smth) { return false...

Ещё можно классический do-break-while(false)

0

25.12.2021

Alexander Karaev

abc cba
В вашем случае вот так примерно: static bool isRef...

if ((auto a = 1) == 2) - невалидная конструкция

0

25.12.2021

Boris Usievich

abc cba
Например я бы писал так: if (smth) { return false...

return not smth; Ну или !smth если больше нравится :)

0

25.12.2021

abc cba

Alexander Karaev
if ((auto a = 1) == 2) - невалидная конструкция

Думал такая конструкция сработает. Жаль

0

25.12.2021

Andrei Tokmakov

abc cba
Думал такая конструкция сработает. Жаль

может тогда заменить на if (auto a = 1; a == 2) {} ?

0

25.12.2021

Alexander Karaev

Andrei Tokmakov
может тогда заменить на if (auto a = 1; a == 2) {...

Тогда скоуп для a будет меньше ожидаемого

0

25.12.2021

abc cba · Accepted Answer

abc cba

Например я бы писал так: if (smth) { return false; } return true; вместо if (!smth) { return true; } return false; Так как это сделает меньше вложенности в случае большого количества условий

0

25.12.2021

170 похожих чатов

Static bool isReferencingMDNode(const Instruction &I) { if (const auto

8 ответов

Похожие вопросы