gallivm: (trivial) fix min / max variable names
Calling the variable min when it's really max and vice versa seems a bit confusing. Reviewed-by: Brian Paul <brianp@vmware.com> Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
This commit is contained in:
parent
a20cc2796f
commit
9405e15f51
|
@ -205,17 +205,17 @@ lp_build_min_simple(struct lp_build_context *bld,
|
||||||
nan_behavior != GALLIVM_NAN_BEHAVIOR_UNDEFINED &&
|
nan_behavior != GALLIVM_NAN_BEHAVIOR_UNDEFINED &&
|
||||||
nan_behavior != GALLIVM_NAN_RETURN_OTHER_SECOND_NONNAN &&
|
nan_behavior != GALLIVM_NAN_RETURN_OTHER_SECOND_NONNAN &&
|
||||||
nan_behavior != GALLIVM_NAN_RETURN_NAN_FIRST_NONNAN) {
|
nan_behavior != GALLIVM_NAN_RETURN_NAN_FIRST_NONNAN) {
|
||||||
LLVMValueRef isnan, max;
|
LLVMValueRef isnan, min;
|
||||||
max = lp_build_intrinsic_binary_anylength(bld->gallivm, intrinsic,
|
min = lp_build_intrinsic_binary_anylength(bld->gallivm, intrinsic,
|
||||||
type,
|
type,
|
||||||
intr_size, a, b);
|
intr_size, a, b);
|
||||||
if (nan_behavior == GALLIVM_NAN_RETURN_OTHER) {
|
if (nan_behavior == GALLIVM_NAN_RETURN_OTHER) {
|
||||||
isnan = lp_build_isnan(bld, b);
|
isnan = lp_build_isnan(bld, b);
|
||||||
return lp_build_select(bld, isnan, a, max);
|
return lp_build_select(bld, isnan, a, min);
|
||||||
} else {
|
} else {
|
||||||
assert(nan_behavior == GALLIVM_NAN_RETURN_NAN);
|
assert(nan_behavior == GALLIVM_NAN_RETURN_NAN);
|
||||||
isnan = lp_build_isnan(bld, a);
|
isnan = lp_build_isnan(bld, a);
|
||||||
return lp_build_select(bld, isnan, a, max);
|
return lp_build_select(bld, isnan, a, min);
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
return lp_build_intrinsic_binary_anylength(bld->gallivm, intrinsic,
|
return lp_build_intrinsic_binary_anylength(bld->gallivm, intrinsic,
|
||||||
|
@ -381,17 +381,17 @@ lp_build_max_simple(struct lp_build_context *bld,
|
||||||
nan_behavior != GALLIVM_NAN_BEHAVIOR_UNDEFINED &&
|
nan_behavior != GALLIVM_NAN_BEHAVIOR_UNDEFINED &&
|
||||||
nan_behavior != GALLIVM_NAN_RETURN_OTHER_SECOND_NONNAN &&
|
nan_behavior != GALLIVM_NAN_RETURN_OTHER_SECOND_NONNAN &&
|
||||||
nan_behavior != GALLIVM_NAN_RETURN_NAN_FIRST_NONNAN) {
|
nan_behavior != GALLIVM_NAN_RETURN_NAN_FIRST_NONNAN) {
|
||||||
LLVMValueRef isnan, min;
|
LLVMValueRef isnan, max;
|
||||||
min = lp_build_intrinsic_binary_anylength(bld->gallivm, intrinsic,
|
max = lp_build_intrinsic_binary_anylength(bld->gallivm, intrinsic,
|
||||||
type,
|
type,
|
||||||
intr_size, a, b);
|
intr_size, a, b);
|
||||||
if (nan_behavior == GALLIVM_NAN_RETURN_OTHER) {
|
if (nan_behavior == GALLIVM_NAN_RETURN_OTHER) {
|
||||||
isnan = lp_build_isnan(bld, b);
|
isnan = lp_build_isnan(bld, b);
|
||||||
return lp_build_select(bld, isnan, a, min);
|
return lp_build_select(bld, isnan, a, max);
|
||||||
} else {
|
} else {
|
||||||
assert(nan_behavior == GALLIVM_NAN_RETURN_NAN);
|
assert(nan_behavior == GALLIVM_NAN_RETURN_NAN);
|
||||||
isnan = lp_build_isnan(bld, a);
|
isnan = lp_build_isnan(bld, a);
|
||||||
return lp_build_select(bld, isnan, a, min);
|
return lp_build_select(bld, isnan, a, max);
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
return lp_build_intrinsic_binary_anylength(bld->gallivm, intrinsic,
|
return lp_build_intrinsic_binary_anylength(bld->gallivm, intrinsic,
|
||||||
|
|
Loading…
Reference in New Issue