[PATCH] D157872: [SelectionDAG] Use TypeSize variant of ComputeValueVTs to compute correct offsets for scalable aggregate types.

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added a comment.
Herald added a subscriber: sunshaoce.

In D157872#4592711 <https://reviews.llvm.org/D157872#4592711>, @craig.topper wrote:

> Should we backport this to LLVM 17?

I wasn't sure of the relative importance given the original work landed prior to LLVM 16.  For AArch64 there's no significant need, I only discovered the bug due to some follow on work that I'm hoping to make public soon.

That said, I've just tried and the patches don't just cherry-pick so if you think this is something you're likely to hit for LLVM 17 just let me know and I'll try the branch root given I guess I'll need to learn how pull request work anyway :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157872