diff options
| author | Vladislav Vinogradov <vlad.vinogradov@intel.com> | 2021-08-20 14:25:42 +0300 |
|---|---|---|
| committer | Vladislav Vinogradov <vlad.vinogradov@intel.com> | 2021-09-20 10:39:10 +0300 |
| commit | ec03bbe8a74ae593d0ea5d8bf55c337e395873d1 (patch) | |
| tree | 4488eaee5e77201bd0cc83be8b6e151506e208e2 /clang/lib/Frontend/CreateInvocationFromCommandLine.cpp | |
| parent | 9a2255dfa012ad0334eacb2d09da7aa4db249c51 (diff) | |
| download | llvm-ec03bbe8a74ae593d0ea5d8bf55c337e395873d1.zip llvm-ec03bbe8a74ae593d0ea5d8bf55c337e395873d1.tar.gz llvm-ec03bbe8a74ae593d0ea5d8bf55c337e395873d1.tar.bz2 | |
[mlir] Fix bug in partial dialect conversion
The discussion on forum:
https://llvm.discourse.group/t/bug-in-partial-dialect-conversion/4115
The `applyPartialConversion` didn't handle the operations, that were
marked as illegal inside dynamic legality callback.
Instead of reporting error, if such operation was not converted to legal set,
the method just added it to `unconvertedSet` in the same way as unknown operations.
This patch fixes that and handle dynamically illegal operations as well.
The patch includes 2 fixes for existing passes:
* `tensor-bufferize` - explicitly mark `std.return` as legal.
* `convert-parallel-loops-to-gpu` - ugly fix with marking visited operations
to avoid recursive legality checks.
Reviewed By: rriddle
Differential Revision: https://reviews.llvm.org/D108505
Diffstat (limited to 'clang/lib/Frontend/CreateInvocationFromCommandLine.cpp')
0 files changed, 0 insertions, 0 deletions