[all-commits] [llvm/llvm-project] 8a2965: [llvm-readobj] Remove a leftover comment from 6ad4...

Martin Storsjö via All-commits all-commits at lists.llvm.org
Fri Nov 28 13:02:46 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8a2965dfa929b49ecc3ba7e508d2f6970ac418af
      https://github.com/llvm/llvm-project/commit/8a2965dfa929b49ecc3ba7e508d2f6970ac418af
  Author: Martin Storsjö <martin at martin.st>
  Date:   2025-11-28 (Fri, 28 Nov 2025)

  Changed paths:
    M llvm/tools/llvm-readobj/ARMWinEHPrinter.cpp

  Log Message:
  -----------
  [llvm-readobj] Remove a leftover comment from 6ad4fdacaeea4777e98a3ab41512c49d3d1b6151. NFC.

This case did get documented upstream, in
https://github.com/MicrosoftDocs/cpp-docs/pull/4202, and the
way that llvm-readobj prints it, implemented in that commit, is
correct.


  Commit: e99d8adf8d34da521d9243ba225995ac543745df
      https://github.com/llvm/llvm-project/commit/e99d8adf8d34da521d9243ba225995ac543745df
  Author: Martin Storsjö <martin at martin.st>
  Date:   2025-11-28 (Fri, 28 Nov 2025)

  Changed paths:
    M llvm/lib/MC/MCWin64EH.cpp

  Log Message:
  -----------
  [MC] [Win64EH] Fix the operator ordering for UOP_SaveFPLRX. NFC.

The encoded offset should be (OffsetInBytes/8)-1 due to an
implicit offset of 1. Previously the operator ordering was
inverted. As the offset is a multiple of 8, the incorrect
operator ordering did produce the right result in all cases
anyway.


Compare: https://github.com/llvm/llvm-project/compare/4dc29b8a5d40...e99d8adf8d34

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