This is an archive of the discontinued LLVM Phabricator instance.

[clang][dataflow] Add a test for a struct that is directly self-referential through a reference.
ClosedPublic

Authored by mboehme on Jul 4 2023, 2:06 AM.

Details

Summary

The ongoing migration to strict handling of value
categories (see https://discourse.llvm.org/t/70086) will change the way we
handle fields of reference type, and I want to put a test in place that makes
sure we continue to handle this special case correctly.

Depends On D154420

Diff Detail

Event Timeline

mboehme created this revision.Jul 4 2023, 2:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 4 2023, 2:06 AM
mboehme requested review of this revision.Jul 4 2023, 2:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 4 2023, 2:06 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
gribozavr2 accepted this revision.Jul 4 2023, 2:09 AM
This revision is now accepted and ready to land.Jul 4 2023, 2:09 AM
xazax.hun accepted this revision.Jul 4 2023, 3:29 AM
mboehme updated this revision to Diff 537029.Jul 4 2023, 3:42 AM

Actually make the test pass. It was missing an indirection.

No idea how I missed this on the initial upload...

Pre-merge failure looks unrelated

This revision was landed with ongoing or failed builds.Jul 4 2023, 5:06 AM
This revision was automatically updated to reflect the committed changes.