[PATCH] [nvptx] Only support the 'm' inline assembly memory constraint. NFC.

Daniel Sanders daniel.sanders at imgtec.com
Tue Mar 10 09:55:37 PDT 2015


Hi jholewinski,

NVPTX doesn't seem to support any additional constraints. Therefore remove
the target hook.

No functional change intended. Depends on D8173.

http://reviews.llvm.org/D8209

Files:
  lib/Target/NVPTX/NVPTXISelLowering.h

Index: lib/Target/NVPTX/NVPTXISelLowering.h
===================================================================
--- lib/Target/NVPTX/NVPTXISelLowering.h
+++ lib/Target/NVPTX/NVPTXISelLowering.h
@@ -497,12 +497,6 @@
                                     std::vector<SDValue> &Ops,
                                     SelectionDAG &DAG) const override;
 
-  unsigned getInlineAsmMemConstraint(
-      const std::string ConstraintCode) const override {
-    // FIXME: Map different constraints differently.
-    return InlineAsm::Constraint_m;
-  }
-
   const NVPTXTargetMachine *nvTM;
 
   // PTX always uses 32-bit shift amounts

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8209.21597.patch
Type: text/x-patch
Size: 628 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150310/8b65d81d/attachment.bin>


More information about the llvm-commits mailing list