[PATCH] [mips] Always clobber $1 for MIPS inline asm.

Daniel Sanders daniel.sanders at imgtec.com
Tue Dec 16 05:24:22 PST 2014


LGTM


================
Comment at: lib/Basic/Targets.cpp:5752
@@ -5751,3 +5751,3 @@
   const char *getClobbers() const override {
-    // FIXME: Implement!
-    return "";
+    // FIXME: Are there any other registers which should be clobbered ?
+    return "~{$1}";
----------------
Not as far as I know. The user normally has to specify clobbers.

http://reviews.llvm.org/D6638

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






More information about the cfe-commits mailing list