[PATCH] D59316: [HIP-Clang] propagate -mllvm options to opt and llc

Fri Mar 15 14:43:42 PDT 2019

yaxunl added a comment.

In D59316#1431302 <https://reviews.llvm.org/D59316#1431302>, @arsenm wrote:

>

> ML workloads are extremely unlikely to use a call. We should have an execution tests with noinline somewhere to stress this

I compiled and ran a test with noinline function and I saw function call in ISA and it passes. I agree that we should have such a test and will keep it in mind.

Repository:
  rC Clang

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

https://reviews.llvm.org/D59316