[all-commits] [llvm/llvm-project] b2aaaf: [scudo] Use stdint types for internal types (redo)

Kostya Kortchinsky via All-commits all-commits at lists.llvm.org
Mon Aug 16 14:47:03 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b2aaafb8377ac1ab081e7c0d3ba92ee5eb4de07c
      https://github.com/llvm/llvm-project/commit/b2aaafb8377ac1ab081e7c0d3ba92ee5eb4de07c
  Author: Kostya Kortchinsky <kostyak at google.com>
  Date:   2021-08-16 (Mon, 16 Aug 2021)

  Changed paths:
    M compiler-rt/lib/scudo/standalone/combined.h
    M compiler-rt/lib/scudo/standalone/internal_defs.h
    M compiler-rt/lib/scudo/standalone/secondary.h
    M compiler-rt/lib/scudo/standalone/wrappers_c_checks.h

  Log Message:
  -----------
  [scudo] Use stdint types for internal types (redo)

This is a redo of D108089 that broke some 32-bit builds.

`scudo::uptr` was defined as an `unsigned long` on 32-b platform,
while a `uintptr_t` is usually defined as an `unsigned int`.
This worked, this was not consistent, particularly with regard to
format string specifiers.

As suggested by Vitaly, since we are including `stdint.h`, define
the internal scudo integer types to those.

Differential Revision: https://reviews.llvm.org/D108152




More information about the All-commits mailing list