Thanks Roland for the approval. I don't have commit access. Can you help pls to commit this? The author line should be --author="Shimin Cui <scui@ca.ibm.com>".

To Address comments from Roland - fix the comment in def and add missing vec_free in BuildLibCalls.cpp

In D94710#2513352, @RolandF wrote:

Test case?

Thanks Jeeva. I need your help to commit this, I don't have commit access yet. The author line should be --author="Shimin Cui <scui@ca.ibm.com>".

In D88669#2345905, @jdoerfert wrote:

Reverse ping. More comments (nits) and notice that we should merge this, D89830 reimplemented it in the meantime, see my comment there too.

Updated to address review comments: I adopted the use of Log2_32 from D89830, removed the parenthesis comment, and added a TODO comment in the test case.

Rewrite the code using getKnowledgeValidInContext. The code is much cleaner than the previous version, but the result computed is less accurate if the size of the alignment bundle is three. I'll leave the improvement for the future, together with the improvement of value tracking of GEP (an example is that, with the future improvement, we can fold away the branch "br i1 %3, ...") in f2 of the added case assume-align.ll).

In D88669#2328334, @efriedma wrote:

What does the third operand of an align bundle mean? It doesn't seem to be documented in LangRef.

Anyone has any comments for this patch? Any comments or suggestions are appreciated.

In D87850#2322762, @efriedma wrote:

LGTM

Do you have commit access? If not, I can commit it for you; what should I put in the git "Author" line?

Remove the added extra cases from the existing test files

Adjust the test cases.

Anyone has any comments for this patch? Any comments or suggestions are appreciated.

Fix previous reverion of 2004-11-27-SetCCForCastLargerAndConstant.ll and PR17073.ll

Revert 2004-11-27-SetCCForCastLargerAndConstant.ll and PR17073.ll