<div dir="ltr">Sure I will add an IR based test case. The end-to-end test is still good to have though .<div><br></div><div>David</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 30, 2017 at 11:02 AM, Vedant Kumar via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vsk added a comment.<br>
<br>
With an IR-based test case, <a href="https://reviews.llvm.org/D40663" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D40663</a> shouldn't be necessary --<br>
<br>
  ; RUN opt -instrprof -S -o - <%s -do-counter-promotion=1 | FileCheck<br>
  ; CHECK: store<br>
<br>
  target datalayout = "e-m:o-i64:64-f80:128-n8:16:<wbr>32:64-S128"<br>
  target triple = "x86_64-apple-macosx10.12.0"<br>
<br>
  @__profn_foo = private constant [3 x i8] c"foo"<br>
<br>
  define void @foo() {<br>
  entry:<br>
    br label %while.body<br>
<br>
  while.body:                                       ; preds = %entry, %while.body<br>
    call void @llvm.instrprof.increment(i8* getelementptr inbounds ([3 x i8], [3 x i8]* @__profn_foo, i32 0, i32 0), i64 14813359968, i32 1, i32 0)<br>
    call void (...) @bar() #2<br>
    br label %while.body<br>
  }<br>
<br>
  declare void @bar(...)<br>
<br>
  declare void @llvm.instrprof.increment(i8*, i64, i32, i32) #0<br>
<br>
  attributes #0 = { nounwind }<br>
<br>
  !llvm.module.flags = !{!0, !1}<br>
  !llvm.ident = !{!2}<br>
<br>
  !0 = !{i32 1, !"wchar_size", i32 4}<br>
  !1 = !{i32 7, !"PIC Level", i32 2}<br>
  !2 = !{!"clang version 6.0.0 (trunk 319373) (llvm/trunk 319391)"}<br>
<br>
<br>
<a href="https://reviews.llvm.org/D40662" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D40662</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>