[PATCH] D73915: AMDGPU: Add flag to control mem intrinsic expansion

[Matt Arsenault via Phabricator via llvm-commits]

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


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULowerIntrinsics.cpp:68
   ConstantInt *CI = dyn_cast<ConstantInt>(Size);
-  return !CI || (CI->getZExtValue() > MaxStaticSize);
+  return !CI || (CI->getSExtValue() > MaxStaticSize);
 }
----------------
rampitec wrote:
> Why signed?
To handle using negative values with the flag


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73915/new/

https://reviews.llvm.org/D73915