[flang-commits] [flang] [flang] Implement SECOND intrinsic (PR #98881)

Thirumalai Shaktivel via flang-commits flang-commits at lists.llvm.org
Wed Jul 17 03:37:10 PDT 2024


================
@@ -6140,6 +6144,27 @@ IntrinsicLibrary::genScan(mlir::Type resultType,
   return readAndAddCleanUp(resultMutableBox, resultType, "SCAN");
 }
 
+// SECOND
+fir::ExtendedValue
+IntrinsicLibrary::genSecond(std::optional<mlir::Type> resultType,
+                            mlir::ArrayRef<fir::ExtendedValue> args) {
+  assert(args.size() == 1 && !resultType || args.empty() && resultType);
----------------
Thirumalai-Shaktivel wrote:

This causes the build to fail, can you please check?
```
[...]
/home/tshaktiv/Projects/llvm-project/flang/lib/Optimizer/Builder/IntrinsicCall.cpp:6151:27: error: suggest parentheses around ‘&&’ within ‘||’ [-Werror=parentheses]
 6151 |   assert(args.size() == 1 && !resultType || args.empty() && resultType);
      |          ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
cc1plus: all warnings being treated as errors
```

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


More information about the flang-commits mailing list