[PATCH] D88325: IR: Reject unsized sret in verifier

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 10:27:15 PST 2022


arsenm added a comment.

In D88325#3931206 <https://reviews.llvm.org/D88325#3931206>, @efriedma wrote:

> What do we use the sret type for, anyway?  I tried looking around, but the only actual use I can find is Value::getPointerAlignment.  (SelectionDAG stores the type, but doesn't actually use it for anything, as far as I can tell.  And I'm not seeing any other usage.)  We should probably consider just dropping the type from sret.

objectsize looks at the allocated size at least.


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

https://reviews.llvm.org/D88325



More information about the llvm-commits mailing list