[PATCH] D112961: [fir] Add fir.extract_value and fir.insert_value conversion

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added inline comments.


================
Comment at: flang/lib/Optimizer/CodeGen/DescriptorModel.h:90
+template <>
+TypeBuilderFunc getModel<Fortran::ISO::CFI_index_t>() {
+  return [](mlir::MLIRContext *context) -> mlir::Type {
----------------
This breaks the [[ https://lab.llvm.org/buildbot/#/builders/172/builds/5299 | windows build ]]:
```
C:\Users\buildbot-worker\minipc-ryzen-win\flang-x86_64-windows\llvm-project\flang\lib\Optimizer\CodeGen\DescriptorModel.h(95): error C2766: explicit specialization; 'fir::TypeBuilderFunc (__cdecl *fir::getModel<__int64>(void))(mlir::MLIRContext *)' has already been defined
C:\Users\buildbot-worker\minipc-ryzen-win\flang-x86_64-windows\llvm-project\flang\lib\Optimizer\CodeGen\DescriptorModel.h(70): note: see previous definition of 'getModel'
```

`CFI_index_t` is a typedef of `ptrdiff_t` which is a 64 bit signed integer. That is, the same as `getModel<long long>` template specialization. Don't know why it works under Linux or why the Windows pre-merge check didn't run.



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112961/new/

https://reviews.llvm.org/D112961