mirror of
https://github.com/intel/llvm.git
synced 2026-02-01 17:07:36 +08:00
[ODS][NFC] Cast range.size() to int32_t in accumulation (#85629)
Using range.size() "as is" means we accumulate 'size_t' values into 'int32_t' variable. This may produce narrowing conversion warnings (particularly, on MSVC). The surrounding code seems to cast <x>.size() to 'int32_t' so following this practice seems safe enough. Co-authored-by: Ovidiu Pintican <ovidiu.pintican@intel.com>
This commit is contained in:
@@ -3057,7 +3057,7 @@ void OpEmitter::genCodeForAddingArgAndRegionForBuilder(
|
||||
body << llvm::formatv(
|
||||
"static_cast<int32_t>(std::accumulate({0}.begin(), {0}.end(), 0, "
|
||||
"[](int32_t curSum, ::mlir::ValueRange range) {{ return curSum + "
|
||||
"range.size(); }))",
|
||||
"static_cast<int32_t>(range.size()); }))",
|
||||
operandName);
|
||||
} else {
|
||||
body << "static_cast<int32_t>(" << getArgumentName(op, i) << ".size())";
|
||||
|
||||
Reference in New Issue
Block a user