[PATCH] D40542: [SystemZ] Bugfix in adjustSubwordCmp()

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 29 01:09:53 PST 2017


jonpa updated this revision to Diff 124701.
jonpa added a comment.

> So I'm wondering if the right fix isn't to only replace the *chain* users?

Hmm, yes I think you are right. I must have missed the point that the original load may actually remain... (plus the fact that it only has one user)

Patch updated to only update the chain users, and also to do this in lowerBITCAST().


https://reviews.llvm.org/D40542

Files:
  lib/Target/SystemZ/SystemZISelLowering.cpp
  test/CodeGen/SystemZ/dag-combine-02.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40542.124701.patch
Type: text/x-patch
Size: 10928 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171129/2843286c/attachment.bin>


More information about the llvm-commits mailing list