[PATCH] D68397: [DAGCombiner] add operation legality checks before creating shift ops (PR43542)

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 3 08:43:46 PDT 2019


lebedev.ri added a comment.

I've always wondered why in DAGCombine we create nodes rarely consulting with `TLI.isOperation*()`.
This patch explains it :/ Looks ok to me..


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68397/new/

https://reviews.llvm.org/D68397





More information about the llvm-commits mailing list