[all-commits] [llvm/llvm-project] 28fd2c: [BOLT] Fix trap value for non-X86

[treapster via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 28fd2ca142462c3ed9a9dca43a3dfd98edbd4151
      https://github.com/llvm/llvm-project/commit/28fd2ca142462c3ed9a9dca43a3dfd98edbd4151
  Author: Denis Revunov <revunov.denis at huawei-partners.com>
  Date:   2023-08-24 (Thu, 24 Aug 2023)

  Changed paths:
    M bolt/include/bolt/Core/MCPlusBuilder.h
    M bolt/lib/Core/BinaryEmitter.cpp
    M bolt/lib/Rewrite/RewriteInstance.cpp
    M bolt/lib/Target/AArch64/AArch64MCPlusBuilder.cpp
    M bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp
    M bolt/lib/Target/X86/X86MCPlusBuilder.cpp

  Log Message:
  -----------
  [BOLT] Fix trap value for non-X86

The trap value used by BOLT was assumed to be single-byte instruction.
It made some functions unaligned on AArch64(e.g exceptions-instrumentation test)
and caused emission failures. Fix that by changing fill value to StringRef.

Reviewed By: rafauler

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


  Commit: 82ed7896cf24a725066be8989ea73ddc3c723b5f
      https://github.com/llvm/llvm-project/commit/82ed7896cf24a725066be8989ea73ddc3c723b5f
  Author: Denis Revunov <rnovds at gmail.com>
  Date:   2023-08-24 (Thu, 24 Aug 2023)

  Changed paths:
    A bolt/test/runtime/mark-funcs.c

  Log Message:
  -----------
  [BOLT] Add test for emitting trap value

Reviewed By: rafauler

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


Compare: https://github.com/llvm/llvm-project/compare/5042e1e56d39...82ed7896cf24