[PATCH] D80101: [StackColoring] When remapping alloca's move the To alloca if the From alloca is before it.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D80101#2041001 <https://reviews.llvm.org/D80101#2041001>, @jdoerfert wrote:

> Did we have a bug with reproducer that we can run? I hope there is also a way to run only this pass and get the output (IR). Though, I don't know how :)
>
>  ---
>
> I find the patch fine but I want someone else to say something as well.
>  Nit: maybe put the first sentence of the commit message as a comment before the conditional.


We have this C++ test which we can get IR for llc from.

  class d {
    float b[4];
  };
  
  d operator-(d);
  struct e {
    struct f {
      int *c[4];
    };
    void h(const d &);
  };
  
  struct j {
    int temp;
    e::f k();
  };
  d i;
  
  void g() {
    e a;
    a.h(-i);
    j b;
    b.k();
  }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80101