This is an archive of the discontinued LLVM Phabricator instance.

[yaml] Error on alias nodes that have properties attached
Needs ReviewPublic

Authored by njames93 on Jun 29 2022, 3:45 AM.

Details

Summary

Alias nodes are explicitly forbidden from having properties.
Currently this is handled by just ignoring the properties which is not correct and will lead to confusion if a user tries to attach a property to an alias and not get the expected behaviour.

Diff Detail

Event Timeline

njames93 created this revision.Jun 29 2022, 3:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 29 2022, 3:45 AM
njames93 requested review of this revision.Jun 29 2022, 3:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 29 2022, 3:45 AM
njames93 updated this revision to Diff 441321.Jun 30 2022, 2:46 AM

Emit fix hints for removing the tags.

jhenderson resigned from this revision.Jul 6 2022, 12:11 AM