[PATCH] D39595: [DAGCombine] Refactor ReduceLoadWidth

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 30 03:53:35 PST 2017


samparker added a subscriber: eli.friedman.
samparker added a comment.

@fhahn @RKSimon @eli.friedman I made some small changes before committing, could someone please check them out? My development branch only had x86, Arm and Aarch64 targets built and so when I went to run my pre-commit tests, some were failing for AMDGPU and NVPTX. The issues were that even though some checks are duplicated, I have an early return in the new block which still requires those checks. Also I had missed how any_extend nodes are combined, so I re-added some code that was lost during the refactor.


Repository:
  rL LLVM

https://reviews.llvm.org/D39595





More information about the llvm-commits mailing list