[llvm-commits] [PATCH] - Fix a bug in DAGCombining of extract-load

Rotem, Nadav nadav.rotem at intel.com
Wed May 11 00:33:04 PDT 2011


Hi,

Please review the attached patch.

This patch fixes a bug in the DAGCombiner.  LoadSDNodes have two values (data, chain).
If there is a store after the load node, then there is a chain, which means that there is another user.
Thus, asking hasOneUser would fail. Instead we ask  hasNUsesOfValue on the 'data' value.

Thanks,
Nadav
---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110511/5a9785c8/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: LN0_uses.diff
Type: application/octet-stream
Size: 1091 bytes
Desc: LN0_uses.diff
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110511/5a9785c8/attachment.obj>


More information about the llvm-commits mailing list