[PATCH] D60031: Split tailcallelim into tailcallmark and tailcallelim

Hideto Ueno via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 17 21:59:21 PDT 2019


uenoku marked an inline comment as done.
uenoku added inline comments.


================
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
----------------
jdoerfert wrote:
> 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.
You are right. AA should be preserved.

I submit a patch with all the context. I am sorry to have inconvenienced you.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60031





More information about the llvm-commits mailing list