[PATCH] D70000: [DAGCombine][NFC] Initialize the default operation action for SIGN_EXTEND_INREG for vector type as 'expand' instead of 'legal'

Qing Shan Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 8 17:08:57 PST 2019


steven.zhang added a comment.

In D70000#1738927 <https://reviews.llvm.org/D70000#1738927>, @jsji wrote:

> Why do you call this `[NFC]`? Also why you think this do NOT need any testcases?


This patch just change the default action operation from legal to expand, and mark it as legal for each target that supports it. It is nfc in essential. And I agree that, it might change the action from legal to expand for those target mis-set it before.


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

https://reviews.llvm.org/D70000





More information about the llvm-commits mailing list