[PATCH] D155622: [X86] Recognize standalone `(1 << nbits) - 1` pattern as bzhi

[Danila Malyutin via Phabricator via llvm-commits]

danilaml marked an inline comment as done.
danilaml added inline comments.


================
Comment at: llvm/test/CodeGen/X86/extract-lowbits.ll:516
+; X86-BMI2-NEXT:    retl
+;
 ; X64-NOBMI-LABEL: bzhi64_a1_indexzext:
----------------
goldstein.w.n wrote:
> Can you split the new tests to a seperate patch?
> 
> I.e 
> commit 1: tests (updated with LLVM before this commit)
> commit 2: impl (update tests with the impl)
> 
> Then submit both to phab and use "Edit Related Revisions" to make the tests patch the child of the impl patch.
> 
> That way it easier to track the changes caused by the impl itself.
commit 1 would be empty in this case. Everything below this line is just label renumbering caused by insertion of the new test function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155622