[PATCH] D118084: [CUDA, NVPTX] Pass byval aggregates directly

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 26 04:25:57 PST 2022


lebedev.ri added a comment.

In D118084#3271110 <https://reviews.llvm.org/D118084#3271110>, @tra wrote:

> In D118084#3271073 <https://reviews.llvm.org/D118084#3271073>, @jdoerfert wrote:
>
>> @lebedev.ri wanted to teach SROA how to deal with dynamic indices before, IIRC. It seems to be generally useful.
>
> Interesting. I'd like to hear more.

I guess i, too, would like to hear more about the problem.
My last idea was about allowing splitting

  struct {
    int a;
    int b[2];
  } a;

into

  // not in a struct anymore!
  int a;
  int b[2]

But given just the `int b[2];` i'm not sure what can be done.

>> This patch can wait till then?
>
> Yes.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118084



More information about the cfe-commits mailing list