This is an archive of the discontinued LLVM Phabricator instance.

[TableGen] Allow BitsInit to init integer in pseudo expansion

Authored by sepavloff on Mar 22 2021, 12:53 AM.

Diff Detail

Event Timeline

sepavloff requested review of this revision.Mar 22 2021, 12:53 AM
sepavloff created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2021, 12:53 AM

I would declare BI as BitsInit *BI, for consistency.


Please include a message in the assert, in the usual way.

sepavloff updated this revision to Diff 332319.Mar 22 2021, 9:12 AM

Addressed reviewer's notes

I presume this change passes all the TableGen tests?

I presume this change passes all the TableGen tests?

I ran check-llvm.

Looks good to me.

This revision is now accepted and ready to land.Mar 22 2021, 9:51 AM
This revision was landed with ongoing or failed builds.Mar 22 2021, 9:52 PM
This revision was automatically updated to reflect the committed changes.
craig.topper added inline comments.

Just use a cast instead of checking dyn_cast with an assert.

sepavloff added inline comments.Mar 22 2021, 10:56 PM