[PATCH] D28964: [PGO] Value profile support for value ranges

[Rong Xu via Phabricator via llvm-commits]

xur marked an inline comment as done.
xur added a comment.

One reply to vsk's comment in inlined.

I'll add a test per vsk's suggestion. Patch description will also be updated when I upload the new patch.



================
Comment at: lib/profile/InstrProfilingValue.c:251
+
+  __llvm_profile_instrument_target(TargetValue, Data, CounterIndex);
+}
----------------
vsk wrote:
> IIUC we get precise counts if TargetValue is in [PreciseRangeStart, PreciseRangeLast], or if LargeValue == INT64_MIN. If so, that doesn't seem quite right, because I thought you only wanted precise counts for in-range values.
I don't see a problem here: if LargeValue is INT64_MIN,  we will go the second if in the else branch  -- this collapse the out of range values.


https://reviews.llvm.org/D28964