<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 2, 2020 at 4:24 PM Matthew Voss via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">ormris added a comment.<br>
<br>
> I would rather to fix this in the memop size transformation -- to detect this in the heuristic.<br>
<br>
I think it would be valuable to guard against this pattern within the transform, but I'm concerned that putting the error in the pass would make it harder to debug this issue. If we detect this in llvm-profdata, the user is more likely to have/provide the profraw files causing this behavior.<br></blockquote><div>I don't think putting this into transformation will make it more difficult to debug. There are ready many legality heuristics there.</div><div>To me, the merge operation should be a straightforward translation from raw format to index format. We should not put filters there.</div><div>A warning is OK but not a filter of profile records.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Should we do something in profile runtime to prevent this.<br>
<br>
Do you think llvm-profdata could be causing this issue? I was thinking that profdata might be merging two profiles incorrectly. Either way, having the profraw files would be valuable for debugging...<br></blockquote><div>There are chances of llvm-prodate bugs but more chances in the compiler-rt runtime.</div><div>Providing the raw profile and the index profile will be very helpful for the debuging.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D92074/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D92074/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D92074" rel="noreferrer" target="_blank">https://reviews.llvm.org/D92074</a><br>
<br>
</blockquote></div></div>