[PATCH] D43063: [DAGCombiner] Call ExtendUsesToFormExtLoad in (zext (and (load)))->(and (zextload)) when the load has multiple uses not when the and has multiple uses.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 10 04:31:17 PST 2018


RKSimon added a comment.

Could we just move the OneUse test inside ExtendUsesToFormExtLoad?

@tra Any comments? Also, a lot of the NVPTX tests can be tricky to maintain as most only use CHECK sparingly, often resulting in tests needing to add additional CHECKs, giving the impression that we've added instructions when they were there 'hidden' all along - would you be open to adding NVPTX support to update_llc_test_checks?


Repository:
  rL LLVM

https://reviews.llvm.org/D43063





More information about the llvm-commits mailing list