[Mlir-commits] [mlir] [MLIR] Fix generic assembly syntax for ArrayAttr containing hex float (PR #94583)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Thu Jun 6 00:14:45 PDT 2024
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff 11c08327dab425b67d80e99850e654e4c6c17864 480abfda103f7b047da4cc7eb898978172cc0126 -- mlir/lib/IR/AsmPrinter.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/mlir/lib/IR/AsmPrinter.cpp b/mlir/lib/IR/AsmPrinter.cpp
index ba47283e37..6a362afc52 100644
--- a/mlir/lib/IR/AsmPrinter.cpp
+++ b/mlir/lib/IR/AsmPrinter.cpp
@@ -2061,7 +2061,8 @@ void AsmPrinter::Impl::printLocationInternal(LocationAttr loc, bool pretty,
/// Print a floating point value in a way that the parser will be able to
/// round-trip losslessly.
-static void printFloatValue(const APFloat &apValue, raw_ostream &os, bool *printedHex = nullptr) {
+static void printFloatValue(const APFloat &apValue, raw_ostream &os,
+ bool *printedHex = nullptr) {
// We would like to output the FP constant value in exponential notation,
// but we cannot do this if doing so will lose precision. Check here to
// make sure that we only output it in exponential format if we can parse
@@ -2281,7 +2282,8 @@ void AsmPrinter::Impl::printAttributeImpl(Attribute attr,
printFloatValue(floatAttr.getValue(), os, &printedHex);
// FloatAttr elides the type if F64.
- if (typeElision == AttrTypeElision::May && floatAttr.getType().isF64() && !printedHex)
+ if (typeElision == AttrTypeElision::May && floatAttr.getType().isF64() &&
+ !printedHex)
return;
} else if (auto strAttr = llvm::dyn_cast<StringAttr>(attr)) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/94583
More information about the Mlir-commits
mailing list