From 64de80839eebc1612f2300a45743c7cc5714f49c Mon Sep 17 00:00:00 2001 From: Lioncash Date: Fri, 12 Apr 2019 14:44:14 -0400 Subject: [PATCH] A64/impl: Reorganize peculiar void use in V_scalar To a reader this might look particularly strange, given the function itself has a void return value, but this is actually valid, given the function in the return statement also has a void return value. This instead alters it to be a little easier to parse and potentially be a little less confusing at a glance. --- src/frontend/A64/translate/impl/impl.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/frontend/A64/translate/impl/impl.cpp b/src/frontend/A64/translate/impl/impl.cpp index 6250a574..4716e882 100644 --- a/src/frontend/A64/translate/impl/impl.cpp +++ b/src/frontend/A64/translate/impl/impl.cpp @@ -226,7 +226,8 @@ IR::UAnyU128 TranslatorVisitor::V_scalar(size_t bitsize, Vec vec) { void TranslatorVisitor::V_scalar(size_t bitsize, Vec vec, IR::UAnyU128 value) { if (bitsize == 128) { - return V(128, vec, value); + V(128, vec, value); + return; } // TODO: Optimize ir.SetQ(vec, ir.ZeroExtendToQuad(value));