[PATCH] x86 NaCl: Emit LAHF/SAHF instead of PUSHF/POPF

Tim Northover t.p.northover at gmail.com
Fri Dec 12 07:17:55 PST 2014


================
Comment at: lib/Target/X86/X86InstrInfo.cpp:3281-3282
@@ +3280,4 @@
+      if (FromEFLAGS) {
+        BuildMI(MBB, MI, DL, get(X86::LAHF));
+        BuildMI(MBB, MI, DL, get(Mov), Reg).addReg(AX);
+      }
----------------
I don't think this works. This expansion occurs post-RA doesn't it? In which case AX could be live.

http://reviews.llvm.org/D6629

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






More information about the llvm-commits mailing list