[flang-commits] [PATCH] D138150: [flang] Lower F08 NORM2 intrinsic

[Tarun Prabhu via Phabricator via flang-commits]

tarunprabhu updated this revision to Diff 479459.
tarunprabhu edited the summary of this revision.
tarunprabhu added a comment.

@jeanPerier, in the previous patch, I said that I had changed the runtime API when all I did was create a null value and pass that along to the runtime. Sorry, that was a silly thing to do and say, This one _actually_ changes the runtime API.

- Change the runtime API to not require (even optionally) a mask since the `norm2` intrinsic in Fortran does not accept a mask.

- Fix the tests to not explicitly check for `fastmath<contract>` but instead match against any fastmath specification that appears.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138150/new/

https://reviews.llvm.org/D138150

Files:
  flang/include/flang/Optimizer/Builder/Runtime/Reduction.h
  flang/include/flang/Runtime/reduction.h
  flang/lib/Lower/IntrinsicCall.cpp
  flang/lib/Optimizer/Builder/Runtime/Reduction.cpp
  flang/runtime/extrema.cpp
  flang/test/Lower/Intrinsics/norm2.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D138150.479459.patch
Type: text/x-patch
Size: 17763 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/flang-commits/attachments/20221201/f7fd6b20/attachment-0001.bin>