[PATCH] D28700: [NVPTX] Let there be One True Way to set NVVMReflect params.

[Justin Lebar via Phabricator via llvm-commits]

jlebar created this revision.
jlebar added a reviewer: tra.
jlebar added a subscriber: llvm-commits.
Herald added a subscriber: jholewinski.

Previously there were three ways to inform the NVVMReflect pass whether
you wanted to flush denormals to zero:

- An LLVM command-line option
- Parameters to the NVVMReflect constructor
- Metadata on the module itself.

This change removes the first two, leaving only the third.

The motivation for this change, aside from simplifying things, is that
we want LLVM to be aware of whether it's operating in FTZ mode, so other
passes can use this information.  Ideally we'd have a target-generic
piece of metadata on the module.  This change moves us in that
direction.


https://reviews.llvm.org/D28700

Files:
  llvm/docs/NVPTXUsage.rst
  llvm/lib/Target/NVPTX/NVPTX.h
  llvm/lib/Target/NVPTX/NVVMReflect.cpp
  llvm/test/CodeGen/NVPTX/nvvm-reflect.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28700.84374.patch
Type: text/x-patch
Size: 11836 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170113/6a14f519/attachment.bin>