This is an archive of the discontinued LLVM Phabricator instance.

[MLIR][Bytecode] Add missing field initializer in constructor initializer list
ClosedPublic

Authored by mehdi_amini on Aug 2 2023, 10:55 PM.

Details

Summary

Leaving this field unitialized could led to crashes when it'll diverge from the
IRNumbering phase.

Diff Detail

Event Timeline

mehdi_amini created this revision.Aug 2 2023, 10:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 2 2023, 10:55 PM
mehdi_amini requested review of this revision.Aug 2 2023, 10:55 PM
rriddle accepted this revision.Aug 2 2023, 11:13 PM
This revision is now accepted and ready to land.Aug 2 2023, 11:13 PM
This revision was landed with ongoing or failed builds.Aug 2 2023, 11:31 PM
This revision was automatically updated to reflect the committed changes.

Good catch. Thank you!