[PATCH] D45108: [InstCombine] Get rid of select of bittest (PR36950 / PR17564)

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 6 11:22:43 PDT 2018


lebedev.ri added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineSelect.cpp:407
+/// instructions, but we got rid of select.
+/// https://rise4fun.com/Alive/uiH
+static Instruction *foldSelectICmpAndAnd(Type *SelType, const ICmpInst *IC,
----------------
spatel wrote:
> IIRC, we don't want Alive links in the LLVM source (but Alive links in commit messages and bug reports are fine) - avoid any potential legal/licensing problems. 
First time i'm reading about this, will remove.
Source only, it's ok in tests?


Repository:
  rL LLVM

https://reviews.llvm.org/D45108





More information about the llvm-commits mailing list