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

[Bill Wendling via Phabricator via llvm-commits]

void added a comment.

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. :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104734