[llvm-commits] [patch] Use the new edge dominance functions in GVN

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Aug 10 17:25:46 PDT 2012


> Did you send the right patch?

Sorry, looks like I created a diff from the wrong branch. One from
trunk is attached.

A quick note on the patch. It still uses a simple dominance check in
GVN. With your previous patch I could now turn the two

+  if (!BBE.isSingleEdge())
+    return false;

into

assert(BBE.isSingleEdge());

and modify GVN to stop earlier if it finds multiple edges. Let me know
what you think.

> -Eli

Cheers,
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: t.patch
Type: application/octet-stream
Size: 11654 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120810/fe75fa49/attachment.obj>


More information about the llvm-commits mailing list