Skip to content

Conversation

@andykaylor
Copy link
Collaborator

This change reverse the polarity of the cir::RecordType incomplete attribute, making the attribute name "complete" and reversing all checks in the code accordingly. The asm layout still prints "incomplete" for records that are not complete and treats the absence of this text as meaning that the definition provided is complete.

No functional change is intended.

This change reverse the polarity of the cir::RecordType incomplete attribute,
making the attribute name "complete" and reversing all checks in the code
accordingly. The asm layout still prints "incomplete" for records that
are not complete and treats the absence of this text as meaning that
the definition provided is complete.

No functional change is intended.
@andykaylor
Copy link
Collaborator Author

As requested in llvm/llvm-project#135105

@andykaylor andykaylor merged commit cdef49c into llvm:main Apr 15, 2025
10 checks passed
@andykaylor andykaylor deleted the flip-incomplete branch April 15, 2025 20:00
terapines-osc-cir pushed a commit to Terapines/clangir that referenced this pull request Sep 2, 2025
…vm#1563)

This change reverse the polarity of the cir::RecordType incomplete
attribute, making the attribute name "complete" and reversing all checks
in the code accordingly. The asm layout still prints "incomplete" for
records that are not complete and treats the absence of this text as
meaning that the definition provided is complete.

No functional change is intended.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants