[PATCH] D41628: [DAGCombine] Fix for PR35763
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 3 08:41:00 PST 2018
RKSimon added a comment.
In https://reviews.llvm.org/D41628#965775, @samparker wrote:
> Not simply, the original test case is a bit odd and the bug easily fails to trigger. Any particular reason for not using the original reduced test case?
Just that more complex test cases can sometimes result in the underlying issue it was testing being 'hidden' by future (unrelated) patches, the simpler the test case the better likelihood that we're still testing for the same thing.
Please can you commit pr35763.ll with the trunk's current codegen and then update this patch just to show the codegen diff?
https://reviews.llvm.org/D41628
More information about the llvm-commits
mailing list