[PATCH] D137141: [SDAG] Allow scalable vectors in ComputeNumSignBits

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 11:02:29 PST 2022


reames added a comment.

Realized I hadn't included a bailout before ComputeNumSignBitsForTargetNode in this patch.  This was unintentional, but there are few enough implementations of this that a quick audit was easy.  I didn't see anything concerning.  So I'm going to leave this as is instead of inserting the bailout and then removing it again.  :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137141



More information about the llvm-commits mailing list