]> granicus.if.org Git - llvm/commit
[NVPTX] Let there be One True Way to set NVVMReflect params.
authorJustin Lebar <jlebar@google.com>
Sun, 15 Jan 2017 16:54:35 +0000 (16:54 +0000)
committerJustin Lebar <jlebar@google.com>
Sun, 15 Jan 2017 16:54:35 +0000 (16:54 +0000)
commit352f7fdadc58bcc8d9abcca7c766e8e92c28e522
tree4e9d1b1453b8b22ccb54b45d95d28a38b2c27949
parent50520f329ad3e2e150a67e82ac8725b167daeee3
[NVPTX] Let there be One True Way to set NVVMReflect params.

Summary:
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.

Reviewers: tra

Subscribers: jholewinski, llvm-commits

Differential Revision: https://reviews.llvm.org/D28700

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@292068 91177308-0d34-0410-b5e6-96231b3b80d8
docs/NVPTXUsage.rst
lib/Target/NVPTX/NVPTX.h
lib/Target/NVPTX/NVVMReflect.cpp
test/CodeGen/NVPTX/nvvm-reflect.ll