diff options
| author | Quentin Colombet <quentin.colombet@gmail.com> | 2022-12-02 14:41:29 +0000 |
|---|---|---|
| committer | Quentin Colombet <quentin.colombet@gmail.com> | 2022-12-02 15:26:58 +0000 |
| commit | 786cbb09edf2e4b6b057351444720e96c6dd2066 (patch) | |
| tree | c5f258abfb1211b7d54d286aadb7856a82825f40 /flang/lib/Frontend/CompilerInvocation.cpp | |
| parent | f24466cf6221132fcd6781c41850d901748b7904 (diff) | |
| download | llvm-786cbb09edf2e4b6b057351444720e96c6dd2066.zip llvm-786cbb09edf2e4b6b057351444720e96c6dd2066.tar.gz llvm-786cbb09edf2e4b6b057351444720e96c6dd2066.tar.bz2 | |
Re-apply "[mlir][MemRefToLLVM] Remove the code for lowering subview"
This reverts commit d0650d108929985760d536d59e9927838dccc9b8.
Original commit message:
Subviews are supposed to be expanded before we hit the lowering
code.
The expansion is done with the pass called
expand-strided-metadata.
Add a test that demonstrate how these passes can be linked up to achieve
the desired lowering.
This patch is NFC in spirit but not in practice because `subview` gets
lowered into `reinterpret_cast(extract_strided_metadata, <some math>)`
which lowers in two memref descriptors (one for `reinterpert_cast` and
one for `extract_strided_metadata`), which creates some noise of the
form: `extractvalue(unrealized_cast(extractvalue[0]))[0]` that is
currently not simplified within MLIR but that is really just noop in
that case.
Differential Revision: https://reviews.llvm.org/D136377
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions