diff options
| author | River Riddle <riddleriver@gmail.com> | 2022-02-04 20:47:01 -0800 |
|---|---|---|
| committer | River Riddle <riddleriver@gmail.com> | 2022-02-07 19:03:57 -0800 |
| commit | d7f0083dcae45e6bf774af23533a2d5e18aaf253 (patch) | |
| tree | 9f9e345c5105b994a1fb88bd328ba09b342322e3 /clang/lib/Sema/SemaModule.cpp | |
| parent | 45084eab5e63550bf2fdbf325d8fa5498263130d (diff) | |
| download | llvm-d7f0083dcae45e6bf774af23533a2d5e18aaf253.zip llvm-d7f0083dcae45e6bf774af23533a2d5e18aaf253.tar.gz llvm-d7f0083dcae45e6bf774af23533a2d5e18aaf253.tar.bz2 | |
[mlir:ODS] Deprecate Op parser/printer fields in favor of a new hasCustomAssemblyFormat field
Currently if an operation wants a C++ implemented parser/printer, it specifies inline
code blocks. This is quite problematic for various reasons, e.g. it requires defining
C++ inside of Tablegen which is discouraged when possible, but mainly because
nearly all usages simply forward to static functions (e.g. `static void parseSomeOp(...)`)
with users devising their own standards for how these are defined.
This commit adds support for a `hasCustomAssemblyFormat` bit field that specifies if
a C++ parser/printer is needed, and when set to 1 declares the parse/print methods for
operations to override. For migration purposes, the existing behavior is untouched. Upstream
usages will be replaced in a followup to keep this patch focused on the new implementation.
Differential Revision: https://reviews.llvm.org/D119054
Diffstat (limited to 'clang/lib/Sema/SemaModule.cpp')
0 files changed, 0 insertions, 0 deletions