Page MenuHomePhabricator

[flang] Add simplified version of minval
Needs ReviewPublic

Authored by Leporacanthicus on Oct 6 2022, 7:10 AM.

Details

Summary

Similar to the maxval function already in place.

Diff Detail

Event Timeline

Leporacanthicus created this revision.Oct 6 2022, 7:10 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptOct 6 2022, 7:10 AM
Herald added a subscriber: mehdi_amini. · View Herald Transcript
Leporacanthicus requested review of this revision.Oct 6 2022, 7:10 AM
flang/lib/Optimizer/Transforms/SimplifyIntrinsics.cpp
274

I guess this can share more code with Maxval.

flang/test/Transforms/simplifyintrinsics.fir
1002

Has this been extended to multi-dimensional arrays like SUM? If so, add a testcase for that as well.

LGTM modulo Kiran's remarks.