[PATCH] Add infrastructure for support of multiple memory constraints.

hfinkel at anl.gov hfinkel at anl.gov
Mon Mar 9 14:15:50 PDT 2015


================
Comment at: include/llvm/Target/TargetLowering.h:2597
@@ -2596,1 +2596,3 @@
 
+  virtual unsigned getInlineAsmMemConstraint(const std::string ConstraintCode) const {
+    // FIXME: Actually look at the constraint code.
----------------
hfinkel wrote:
> Line is too long.
Also, you should take the std::string by const reference.

http://reviews.llvm.org/D8171

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list