[all-commits] [llvm/llvm-project] 7da1db: [MachO] Remove redundant bounds check (#100176)
Daniel Bertalan via All-commits
all-commits at lists.llvm.org
Thu Aug 1 10:14:33 PDT 2024
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 7da1dbb632ca490c2ab3c2d6ff46cccda38c7acd
https://github.com/llvm/llvm-project/commit/7da1dbb632ca490c2ab3c2d6ff46cccda38c7acd
Author: Daniel Bertalan <dani at danielbertalan.dev>
Date: 2024-08-01 (Thu, 01 Aug 2024)
Changed paths:
M llvm/lib/Object/MachOObjectFile.cpp
Log Message:
-----------
[MachO] Remove redundant bounds check (#100176)
The condition was duplicated, the correct one for this message would
have been `ImportsEnd > SymbolsEnd`. However, this is a subset of
`ImportEnd > Symbols` (since `Symbols <= SymbolsEnd`), so it can be
removed altogether.
I made this thinko in 686d8ce.
Note that that change wasn't intended to be permanent, and served as a
quick stopgap to facilitate testing chained fixups in LLD before Apple
upstreamed their implementation.
Fixes #90662
Fixes #87203
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