[flang-commits] [flang] [Flang] Allow Intrinsic simpification with min/maxloc dim and scalar result (PR #76194)

David Green via flang-commits flang-commits at lists.llvm.org
Fri Dec 22 03:45:10 PST 2023


davemgreen wrote:

> All builds and tests correctly.
> 
> It would be great if the inlining were controlled by a compilation option.

Thanks for checking. From the other ticket it looks like this will not be done at -O0, which at least gives some control over when this happens.

> LGTM. (I'm not entirely sure I understand what was wrong with the "old" version, but that's probably OK)

Cheers. The important change is adding the typename to the generated function name, as it is not by default encoded like the non-Dim versions.
```
  if (isDim)
    nameOS << '_' << inputType;
```

https://github.com/llvm/llvm-project/pull/76194


More information about the flang-commits mailing list