This is an archive of the discontinued LLVM Phabricator instance.

[fir][NFC] Move fir.select_type builder to cpp file
ClosedPublic

Authored by clementval on Sep 30 2021, 6:10 AM.

Details

Summary

Move the big builder out of the td file to the cpp file.

This patch is part of the upstreaming effort from fir-dev branch.

Diff Detail

Event Timeline

clementval created this revision.Sep 30 2021, 6:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 30 2021, 6:10 AM
clementval requested review of this revision.Sep 30 2021, 6:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 30 2021, 6:10 AM
mehdi_amini added inline comments.Sep 30 2021, 10:49 AM
flang/lib/Optimizer/Dialect/FIROps.cpp
2684

Can you fix the clang-tidy warning?

Also auto likely does not help readability compared to mlir::Block *d

(also: dest instead d?)

clementval marked an inline comment as done.

Address review comment (clang-tidy)

clementval added inline comments.Sep 30 2021, 1:12 PM
flang/lib/Optimizer/Dialect/FIROps.cpp
2684

Sure.

This revision is now accepted and ready to land.Sep 30 2021, 2:54 PM
clementval edited the summary of this revision. (Show Details)Oct 1 2021, 12:19 AM
This revision was automatically updated to reflect the committed changes.