[PATCH] [GVN] Eliminate redundant loads whose addresses are dependent on the result of a select instruction.

Philip Reames listmail at philipreames.com
Tue Mar 24 17:20:04 PDT 2015


================
Comment at: lib/Transforms/Scalar/GVN.cpp:2025
@@ +2024,3 @@
+  // with a select on the two values instead.
+  if (loadGEPSelect(L, Dep))
+    return true;
----------------
Ah, you need to check for a local dependence if you're making that assumption.  Spotted that immediately after sending last comments.  :)

http://reviews.llvm.org/D8120

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






More information about the llvm-commits mailing list