aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp
diff options
context:
space:
mode:
authorRiver Riddle <riddleriver@gmail.com>2022-03-16 11:45:14 -0700
committerRiver Riddle <riddleriver@gmail.com>2022-03-16 14:53:41 -0700
commit50f82e68470c3efbb8ceae8f8c8d289a079d7031 (patch)
tree91c9b563927280b395647aa27e55acc8d9e9a4a6 /clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp
parent79f661edc13d389437b83d42f0af04b0615b920e (diff)
downloadllvm-50f82e68470c3efbb8ceae8f8c8d289a079d7031.zip
llvm-50f82e68470c3efbb8ceae8f8c8d289a079d7031.tar.gz
llvm-50f82e68470c3efbb8ceae8f8c8d289a079d7031.tar.bz2
[mlir] Fix missing verification after running an OpToOpAdaptorPass
The current decision of when to run the verifier is running on the assumption that nested passes can't affect the validity of the parent operation, which isn't true. Parent operations may attach any number of constraints on nested operations, which may not necessarily be captured (or shouldn't be captured) at a smaller granularity. This commit rectifies this by properly running the verifier after an OpToOpAdaptor pass. To avoid an explosive increase in compile time, we only run verification on the parent operation itself. To do this, a flag to mlir::verify is added to avoid recursive verification if it isn't desired. Fixes #54288 Differential Revision: https://reviews.llvm.org/D121836
Diffstat (limited to 'clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions