[all-commits] [llvm/llvm-project] edfc4b: [mlir][ods] Remove warning in `AttrOrTypeDef`
Jeff Niu via All-commits
all-commits at lists.llvm.org
Fri Jul 22 08:29:37 PDT 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: edfc4bb9b9f40604c4a6bfd9e96c21b43bece77c
https://github.com/llvm/llvm-project/commit/edfc4bb9b9f40604c4a6bfd9e96c21b43bece77c
Author: Jeff Niu <jeff at modular.com>
Date: 2022-07-22 (Fri, 22 Jul 2022)
Changed paths:
M mlir/lib/TableGen/AttrOrTypeDef.cpp
Log Message:
-----------
[mlir][ods] Remove warning in `AttrOrTypeDef`
This warning was added because using attribute or type assembly formats
with `skipDefaultBuilders` set could cause compilation errors, since the
required builder prototype may not necessarily be generated and would
need to be checked by hand. This patch removes the warning because a
warning that the generated C++ "might" not compile is not particularly
useful. Attempting to address the TODO (i.e. detect whether a builder of
the correct prototype is provided) would be fragile since it would not
be possible to account for implicit conversions, etc.
In general, ODS should not be emitting warnings in cases like these.
Reviewed By: rriddle, wrengr
Differential Revision: https://reviews.llvm.org/D130102
More information about the All-commits
mailing list