[PATCH] D92020: [AllocaInst] Update `getAllocationSizeInBits` to return `TypeSize`.

[Francesco Petrogalli via Phabricator via llvm-commits]

fpetrogalli marked 2 inline comments as done.
fpetrogalli added inline comments.


================
Comment at: llvm/lib/Transforms/Coroutines/CoroFrame.cpp:589
   sort(FrameData.Allocas, [&](const auto &Iter1, const auto &Iter2) {
-    return GetAllocaSize(Iter1) > GetAllocaSize(Iter2);
+    return TypeSize::isKnownGT(GetAllocaSize(Iter1), GetAllocaSize(Iter2));
   });
----------------
sdesmalen wrote:
> This should maintain the original `operator>` and instead `GetAllocaSize` should return `RetSize->getFixedSize()`.
Good point! Done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92020