[all-commits] [llvm/llvm-project] e45ef5: [mlir][ods] (NFC) Remove warning in `AttrOrTypeDef`

Jeff Niu via All-commits all-commits at lists.llvm.org
Tue Jul 19 09:17:36 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e45ef5ebf4402e553c9a0b10e8765811cc33bbdd
      https://github.com/llvm/llvm-project/commit/e45ef5ebf4402e553c9a0b10e8765811cc33bbdd
  Author: Jeff Niu <jeff at modular.com>
  Date:   2022-07-19 (Tue, 19 Jul 2022)

  Changed paths:
    M mlir/lib/TableGen/AttrOrTypeDef.cpp

  Log Message:
  -----------
  [mlir][ods] (NFC) 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.




More information about the All-commits mailing list