[PATCH] D89412: [TargetLowering] Replace Log2_32_Ceil with Log2_32 in SimplifySetCC ctpop combine.

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 14 11:35:43 PDT 2020


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM - thanks for explaining it.



================
Comment at: llvm/test/CodeGen/X86/ctpop-combine.ll:164
 
 define i1 @ctpop_trunc_non_power2(i255 %x) {
+; CHECK-LABEL: ctpop_trunc_non_power2:
----------------
Add 'nounwind' attribute to avoid cfi noise?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89412



More information about the llvm-commits mailing list