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

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 4 08:04:24 PDT 2018


craig.topper added a comment.

It fails on trunk. Your patch probably fixes it by shrinking the immediate. Is there anything that can prevent the DAG combine from running that we should be concerned about? If there is we might not need to clamp the immediate in lowering or isel.


Repository:
  rL LLVM

https://reviews.llvm.org/D47690





More information about the llvm-commits mailing list