[PATCH] D144911: adding bf16 support to NVPTX

[Kushan Ahmadian via Phabricator via cfe-commits]

kushanam added inline comments.


================
Comment at: llvm/lib/Target/NVPTX/NVPTXISelLowering.cpp:615
     setFP16OperationAction(Op, MVT::v2f16, Legal, Expand);
-  }
-
-  for (const auto &Op : {ISD::FADD, ISD::FMUL, ISD::FSUB, ISD::FMA}) {
     setBF16OperationAction(Op, MVT::bf16, Legal, Promote);
     setBF16OperationAction(Op, MVT::v2bf16, Legal, Expand);
----------------
tra wrote:
> There's still something odd with this patch. The `setBF16OperationAction` is not in the upstream, but it does not show up in the diff on phabricator. 
> 
> Please do rebase on top of the LLVM and make sure that all your changes are on the git branch you use to send the patch to phabricator. If in doubt how to get `arc` to do it correctly, you can always create and upload the diff manually as described here: https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface
It is in the first commit, isn't it?https://reviews.llvm.org/D144911?id=500896


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144911