[PATCH] D47690: [X86][BMI][TBM] Only demand bottom 16-bits of the BEXTR control op (PR34042)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 4 03:42:44 PDT 2018


RKSimon marked an inline comment as done.
RKSimon added a comment.

In https://reviews.llvm.org/D47690#1120298, @craig.topper wrote:

> Hilariously, the tbm-builtins.c test in clang fails -O0 isel on one of the TBM bextr tests because the immediate is too large.


I'm not seeing this with the latest version of my patch - do you still have a repro?


Repository:
  rL LLVM

https://reviews.llvm.org/D47690





More information about the llvm-commits mailing list