[PATCH] D112961: [fir] Add fir.extract_value and fir.insert_value conversion
Michael Kruse via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 5 12:04:48 PDT 2021
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
More information about the llvm-commits
mailing list