[PATCH] ARM: Windows on ARM Hazard recognition support

Saleem Abdulrasool abdulras at fb.com
Wed Apr 2 08:50:29 PDT 2014



================
Comment at: lib/Target/ARM/ARMHazardRecognizer.cpp:69
@@ +68,3 @@
+  const Triple TT(MI->getParent()->getParent()->getTarget().getTargetTriple());
+  if (TT.isOSWindows()) {
+    if (LastMI && LastMI->getOpcode() == ARM::t2MOVi16 &&
----------------
Renato Golin wrote:
> Maybe we should set a feature flag for this somewhere a bit lower, and mark it true on Windows triples?
Might be useful ... as much as I dislike the check as it currently stands, I think that addressing that once more of the CG is in place might make it easier to come up with a nice way to describe this scenario.


http://llvm-reviews.chandlerc.com/D3254



More information about the llvm-commits mailing list