[PATCH] D29473: [AMDGPU] Unroll preferences improvements

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec marked an inline comment as done.
rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp:59
+        Type *Ty = Alloca->getAllocatedType();
+        unsigned AllocaSize = Ty->isSized() ? DL.getTypeAllocSize(Ty) : 0;
+        if (AllocaSize > MaxAlloca)
----------------
arsenm wrote:
> Can you add a test where isSized is necessary? I thought it was illegal to alloca such a type
It happens when you have opaque type, like image. I doubt I can easily write a test like this, but when it happens getTypeAllocSize() asserts, so it is better to keep it.


Repository:
  rL LLVM

https://reviews.llvm.org/D29473