Skip to content

Commit ed7d5fe

Browse files
fandreuzsimonis
authored andcommitted
8360304: Redundant condition in LibraryCallKit::inline_vector_nary_operation
Reviewed-by: shade, vlivanov
1 parent be6c15e commit ed7d5fe

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/hotspot/share/opto/vectorIntrinsics.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -383,15 +383,15 @@ bool LibraryCallKit::inline_vector_nary_operation(int n) {
383383
// When using mask, mask use type needs to be VecMaskUseLoad.
384384
VectorMaskUseType mask_use_type = is_vector_mask(vbox_klass) ? VecMaskUseAll
385385
: is_masked_op ? VecMaskUseLoad : VecMaskNotUsed;
386-
if ((sopc != 0) && !arch_supports_vector(sopc, num_elem, elem_bt, mask_use_type)) {
386+
if (!arch_supports_vector(sopc, num_elem, elem_bt, mask_use_type)) {
387387
log_if_needed(" ** not supported: arity=%d opc=%d vlen=%d etype=%s ismask=%d is_masked_op=%d",
388388
n, sopc, num_elem, type2name(elem_bt),
389389
is_vector_mask(vbox_klass) ? 1 : 0, is_masked_op ? 1 : 0);
390390
return false; // not supported
391391
}
392392

393393
// Return true if current platform has implemented the masked operation with predicate feature.
394-
bool use_predicate = is_masked_op && sopc != 0 && arch_supports_vector(sopc, num_elem, elem_bt, VecMaskUsePred);
394+
bool use_predicate = is_masked_op && arch_supports_vector(sopc, num_elem, elem_bt, VecMaskUsePred);
395395
if (is_masked_op && !use_predicate && !arch_supports_vector(Op_VectorBlend, num_elem, elem_bt, VecMaskUseLoad)) {
396396
log_if_needed(" ** not supported: arity=%d opc=%d vlen=%d etype=%s ismask=0 is_masked_op=1",
397397
n, sopc, num_elem, type2name(elem_bt));

0 commit comments

Comments
 (0)