[MLIR] Apply clang-tidy fixes for readability-simplify-boolean-expr in OpenACC.cpp (NFC)

joker-eph · joker-eph · commit 5af5c3aa2cee · 2025-08-28T15:49:25.000-07:00
diff --git a/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp b/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp
@@ -173,9 +173,7 @@ void OpenACCDialect::initialize() {
 //===----------------------------------------------------------------------===//
 
 static bool hasDeviceTypeValues(std::optional<mlir::ArrayAttr> arrayAttr) {
-  if (arrayAttr && *arrayAttr && arrayAttr->size() > 0)
-    return true;
-  return false;
+  return arrayAttr && *arrayAttr && arrayAttr->size() > 0;
 }
 
 static bool hasDeviceType(std::optional<mlir::ArrayAttr> arrayAttr,

Original file line number	Diff line number	Diff line change
`@@ -173,9 +173,7 @@ void OpenACCDialect::initialize() {`
`173`	`173`	`//===----------------------------------------------------------------------===//`
`174`	`174`
`175`	`175`	`static bool hasDeviceTypeValues(std::optional<mlir::ArrayAttr> arrayAttr) {`
`176`		`- if (arrayAttr && *arrayAttr && arrayAttr->size() > 0)`
`177`		`- return true;`
`178`		`- return false;`
	`176`	`+ return arrayAttr && *arrayAttr && arrayAttr->size() > 0;`
`179`	`177`	`}`
`180`	`178`
`181`	`179`	`static bool hasDeviceType(std::optional<mlir::ArrayAttr> arrayAttr,`