getOperandsScalarizationOverhead()

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 23 14:30:25 PST 2017


On 02/22/2017 09:48 AM, Jonas Paulsson wrote:
> Hi,
>
> while writing CostModel tests for SystemZ, I needed to fix 
> getOperandsScalarizationOverhead() so that it can handle vector 
> argument types.

Why? (A test case would help).

+        Type *VecTy = nullptr;

Don't default initialize here (it is not needed).

+        }
+        else

else on the previous line with the }.

  -Hal

> This is what is passed to it from CostModel, when the default 
> implementation is used for the cost evaluation.
>
> I also added a check not to count an undef operand to extraction costs.
>
> /Jonas
>
>
>

-- 
Hal Finkel
Lead, Compiler Technology and Programming Languages
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-commits mailing list