diff options
| author | Guillaume Chatelet <gchatelet@google.com> | 2022-12-07 14:54:03 +0000 |
|---|---|---|
| committer | Guillaume Chatelet <gchatelet@google.com> | 2022-12-07 14:54:03 +0000 |
| commit | 7203a8614a9b7fe31fc4b0eca457d3595d5556fe (patch) | |
| tree | 51a3f4d42e065faae77bedacd718558e9dcccfd7 /llvm/lib/MC/MCStreamer.cpp | |
| parent | b822063669641570ab5edae72956d18a5bcde8c4 (diff) | |
| download | llvm-7203a8614a9b7fe31fc4b0eca457d3595d5556fe.zip llvm-7203a8614a9b7fe31fc4b0eca457d3595d5556fe.tar.gz llvm-7203a8614a9b7fe31fc4b0eca457d3595d5556fe.tar.bz2 | |
[reland][Alignment] Use Align in MCStreamer emitZeroFill/emitLocalCommonSymbol
Before performing this change, I checked that `ByteAlignment` was never `0` inside `MCAsmStreamer:emitZeroFill` and `MCAsmStreamer::emitLocalCommonSymbol`.
I believe it is NFC as `0` values are illegal in `emitZeroFill` anyways, `Log2(ByteAlignment)` would be undefined.
And currently, all calls to `emitLocalCommonSymbol` are provably `>0`.
Differential Revision: https://reviews.llvm.org/D139439
Diffstat (limited to 'llvm/lib/MC/MCStreamer.cpp')
| -rw-r--r-- | llvm/lib/MC/MCStreamer.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/MC/MCStreamer.cpp b/llvm/lib/MC/MCStreamer.cpp index 968bd8b..c87b841 100644 --- a/llvm/lib/MC/MCStreamer.cpp +++ b/llvm/lib/MC/MCStreamer.cpp @@ -1202,7 +1202,7 @@ void MCStreamer::emitELFSize(MCSymbol *Symbol, const MCExpr *Value) {} void MCStreamer::emitELFSymverDirective(const MCSymbol *OriginalSym, StringRef Name, bool KeepOriginalSym) {} void MCStreamer::emitLocalCommonSymbol(MCSymbol *Symbol, uint64_t Size, - unsigned ByteAlignment) {} + Align ByteAlignment) {} void MCStreamer::emitTBSSSymbol(MCSection *Section, MCSymbol *Symbol, uint64_t Size, unsigned ByteAlignment) {} void MCStreamer::changeSection(MCSection *, const MCExpr *) {} |