[PATCH] D140087: [X86] Replace (31/63 -/^ X) with (NOT X) and ignore (32/64 ^ X) when computing shift count

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D140087#4019772 <https://reviews.llvm.org/D140087#4019772>, @goldstein.w.n wrote:

> In D140087#4019617 <https://reviews.llvm.org/D140087#4019617>, @lebedev.ri wrote:
>
>> Please be sure to precommit the test changes before committing the change itself.
>
> You want the new tests `invalid_add31` / `invalid_sub31` split into a seperate commit?

In general, when adding new tests, if the new tests do not crash the opt/llc before the change,
they should be committed first, so the change shows the diff of CHECK lines, not just the new CHECK lines.

>> Looks good to me now. Thanks!




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140087