This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Add a parameter for passing default values to `StringRefParameter`
ClosedPublic

Authored by fmorac on Jun 29 2023, 8:37 AM.

Details

Summary

For an explanation of these patches see D154153.

Commit message:
Currently the StringRefParameter parameter doesn't support default values,
this patch allows the usage of them, eg:
StringRefParameter<"description", [{"default_value"}]>

Diff Detail

Event Timeline

fmorac created this revision.Jun 29 2023, 8:37 AM
Herald added a project: Restricted Project. · View Herald Transcript
fmorac updated this revision to Diff 536060.Jun 29 2023, 5:17 PM

Rebasing.

fmorac edited the summary of this revision. (Show Details)Jun 29 2023, 6:11 PM
fmorac edited the summary of this revision. (Show Details)Jun 29 2023, 6:16 PM
fmorac added a reviewer: mehdi_amini.
fmorac updated this revision to Diff 536086.Jun 29 2023, 7:08 PM

Rebasing.

fmorac published this revision for review.Jun 30 2023, 6:27 AM

Could you use this in the test dialect to showcase that it works as intended?

Could you use this in the test dialect to showcase that it works as intended?

Yes, I'll add it.

fmorac updated this revision to Diff 538724.Jul 10 2023, 10:29 AM

Added test to the Test dialect.

mehdi_amini accepted this revision.Jul 10 2023, 12:50 PM
This revision is now accepted and ready to land.Jul 10 2023, 12:50 PM