This is an archive of the discontinued LLVM Phabricator instance.

[clang][dataflow] Use `IntegerValue` instead of `StructValue` in `ValueTest`.
ClosedPublic

Authored by mboehme on Jul 11 2023, 2:25 AM.

Details

Summary

Soon, it will no longer be possible to default-construct StructValue. For details, see https://discourse.llvm.org/t/70086.

For completeness, also add a test that areEquivalentValues() on different IntegerValues returns false.

Diff Detail

Event Timeline

mboehme created this revision.Jul 11 2023, 2:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 11 2023, 2:25 AM
mboehme requested review of this revision.Jul 11 2023, 2:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 11 2023, 2:25 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
mboehme edited the summary of this revision. (Show Details)
gribozavr2 accepted this revision.Jul 11 2023, 4:26 AM
This revision is now accepted and ready to land.Jul 11 2023, 4:26 AM
xazax.hun accepted this revision.Jul 11 2023, 7:14 AM
This revision was landed with ongoing or failed builds.Jul 11 2023, 9:52 PM
This revision was automatically updated to reflect the committed changes.