[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