[PATCH] D110277: [compiler-rt][profile] Account for binary IDs size in test

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

In D110277#3019229 <https://reviews.llvm.org/D110277#3019229>, @phosek wrote:

> I'd prefer including `profile/InstrProfData.inc` and using types rather than just hardcoding the offsets. In the past, some fields have already moved around (for example between version 6 and 7 binary IDs size has moved). If this happens again, this test would break. Using types should be less error prone and more readable.

that sgtm


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110277