[PATCH] D60031: Split tailcallelim into tailcallmark and tailcallelim

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

I inlined a question. Also, did you run the tests with assertions enabled?



================
Comment at: test/Other/opt-Os-pipeline.ll:80
+; CHECK-NEXT:         Basic Alias Analysis (stateless AA impl)
+; CHECK-NEXT:         Function Alias Analysis Results
 ; CHECK-NEXT:         Tail Call Elimination
----------------
Why do the analysis appear here? I would have expected them to be somewhere on the left as well.

Btw. please submit patches with *all* the context so I can see more than 3 lines if I want.


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

https://reviews.llvm.org/D60031