[PATCH] D104734: [llvm-diff] Explicitly check ConstantStructs for differences

Wed Jun 23 11:03:08 PDT 2021

rjmccall added a comment.

In D104734#2835045 <https://reviews.llvm.org/D104734#2835045>, @void wrote:

> In D104734#2834999 <https://reviews.llvm.org/D104734#2834999>, @rjmccall wrote:
>
>> That should work, thanks.  What's going on with `StructStack`?
>
> Structures are the only types which can reference themselves. If I don't stop processing already seen types then it's possible they could infinitely recurse. The stack / std::find code is very hackish. I'm open to suggestions on making it non-gross. :-)

Well, they can't reference themselves as sub-aggregates; you'll never see another field of this exact type.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104734