[all-commits] [llvm/llvm-project] 4f5374: [Clnag] Prefer octal to hex for printf

Aiden Grossman via All-commits all-commits at lists.llvm.org
Sun Oct 19 20:10:04 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4f5374bef554eb7f5429c1a86d90833e4a3c98de
      https://github.com/llvm/llvm-project/commit/4f5374bef554eb7f5429c1a86d90833e4a3c98de
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-10-20 (Mon, 20 Oct 2025)

  Changed paths:
    M clang/test/Lexer/minimize_source_to_dependency_directives_utf8bom.c

  Log Message:
  -----------
  [Clnag] Prefer octal to hex for printf

The printf implementation on MacOS that gets picked up by the internal
shell does not support the \xff form as it is not mandated by POSIX.
Prefer octal which is supported and mandated by POSIX.

This is similar to c745c5497069a966a43e5ba03d137e0189921073.


  Commit: 04fc1aee564373e72048333a789c7b277e68f25a
      https://github.com/llvm/llvm-project/commit/04fc1aee564373e72048333a789c7b277e68f25a
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-10-20 (Mon, 20 Oct 2025)

  Changed paths:
    M clang/test/Modules/crash-vfs-relative-incdir.m
    M clang/test/Modules/crash-vfs-run-reproducer.m

  Log Message:
  -----------
  [Clang] Explicitly invoke bash to run shell scripts

Otherwise these tests run into an exec format error when attempting to run
them with the internal shell. This is similar to other patches that have
done the same for similar tests.


Compare: https://github.com/llvm/llvm-project/compare/7539fc5b33ad...04fc1aee5643

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list