This is an archive of the discontinued LLVM Phabricator instance.

[SelectionDAG] Use Align/MaybeAlign for ConstantPoolSDNode.
ClosedPublic

Authored by craig.topper on May 5 2020, 10:40 AM.

Details

Summary

This patch stores the alignment for ConstantPoolSDNode as an
Align and updates the getConstantPool interface to take a MaybeAlign.

Removing getAlignment() will be done as a follow up.

Diff Detail

Event Timeline

craig.topper created this revision.May 5 2020, 10:40 AM

@efriedma I see you've approved D79459 which depends on this. Can you review this one too? Thanks.

efriedma accepted this revision.May 8 2020, 2:40 PM

LGTM

This revision is now accepted and ready to land.May 8 2020, 2:40 PM
This revision was automatically updated to reflect the committed changes.