[all-commits] [llvm/llvm-project] 1c8c36: [mlir][bytecode] Check that bytecode source buffer...

Christian Sigg via All-commits all-commits at lists.llvm.org
Sun Sep 17 04:46:14 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1c8c365de2ac59198e6ef9d61a5557b846ce0bca
      https://github.com/llvm/llvm-project/commit/1c8c365de2ac59198e6ef9d61a5557b846ce0bca
  Author: Christian Sigg <chsigg at users.noreply.github.com>
  Date:   2023-09-17 (Sun, 17 Sep 2023)

  Changed paths:
    M mlir/lib/Bytecode/Reader/BytecodeReader.cpp
    M mlir/unittests/Bytecode/BytecodeTest.cpp
    M utils/bazel/llvm-project-overlay/mlir/unittests/BUILD.bazel

  Log Message:
  -----------
  [mlir][bytecode] Check that bytecode source buffer is sufficiently aligned. (#66380)

Before this change, the `ByteCode` test failed on CentOS 7 with
devtoolset-9, because strings happen to be only 8 byte aligned. In
general though, strings have no alignment guarantee.

Increase resource alignment in test to 32 bytes. 
Adjust test to sufficiently align buffer.
Add test to check error when buffer is insufficiently aligned.




More information about the All-commits mailing list