Page MenuHomePhabricator

[MLIR] Don't pass separate LowerToLLVMOptions when we already pass a LLVMTypeConverter which contains those options already.
ClosedPublic

Authored by csigg on Thu, Jul 30, 1:32 AM.

Details

Summary

This also prevents passing inconsistent options.

Diff Detail

Event Timeline

csigg created this revision.Thu, Jul 30, 1:32 AM
Herald added a project: Restricted Project. · View Herald Transcript
csigg requested review of this revision.Thu, Jul 30, 1:32 AM
ftynse accepted this revision.Thu, Jul 30, 1:37 AM

Thanks for cleaning this up!

This revision is now accepted and ready to land.Thu, Jul 30, 1:37 AM
csigg updated this revision to Diff 281877.Thu, Jul 30, 4:39 AM

Rebase from master.