[PATCH] D86578: [TargetLowering] Combine known bits for icmp in SimplifySetCC (PR41182)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 15 09:51:31 PST 2020


RKSimon added a comment.
Herald added subscribers: dexonsmith, ecnelises, pengfei.

@darcangelo.matthew are you still looking at this?



================
Comment at: llvm/include/llvm/Support/KnownBits.h:145
+    return Val;
+  }
+
----------------
Please add unit test coverage in llvm\unittests\Support\KnownBitsTest.cpp


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86578



More information about the llvm-commits mailing list