[PATCH] D27037: Introduce StringRefZ class to represent null-terminated string.

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 28 22:50:46 PST 2016


Rafael Avila de Espindola <rafael.espindola at gmail.com> writes:

> The code changes are fine, I just wonder how we would handle the error
> condition of a broken file that is missing a null terminator.

LGTM. We have an error if the string table is not null terminated, so
the strlen would never fail.

Cheers,
Rafael


More information about the llvm-commits mailing list