[PATCH] bug fix for PR20020: anti-dependency-breaker causes miscompilation

hfinkel at anl.gov hfinkel at anl.gov
Mon Jun 30 15:12:45 PDT 2014


Can you please upload the patch with more context lines (as explained here: http://llvm.org/docs/Phabricator.html)?

================
Comment at: lib/CodeGen/CriticalAntiDepBreaker.cpp:260
@@ +259,3 @@
+      // FIXME: The tied value should already have been checked in
+      // PrescanInstruction(). Do we need to do this check again?
+
----------------
Can you put an assert inside the existing check that the tied registers are in KeepRegs?

http://reviews.llvm.org/D4351






More information about the llvm-commits mailing list