[Mlir-commits] [mlir] [mlir] Fix GCC compilation warning in TuneExtensionOps.cpp (PR #168850)

[Rolf Morel]

================

----------------
rolfmorel wrote:

If we are able to make sure `selectedRegionAttr` is of unsigned type/only contains non-negative values (if possible, allowing arbitrary bitwidths), I think this call could be changed to `getZExtValue`.

If we aren't going to fix `selectedRegionAttr`'s type in tablegen, I think the actually safe solution is indeed to change `std::optional<size_t> selectedRegionIdx` to `std::optional<int64_t> selectedRegionIdx;`. The checks that are currently proposed to be axed in that case make sure to raise an error on encountering negative values for the index.

https://github.com/llvm/llvm-project/pull/168850