diff options
| author | Guillaume Chatelet <gchatelet@google.com> | 2022-12-06 15:50:02 +0000 |
|---|---|---|
| committer | Guillaume Chatelet <gchatelet@google.com> | 2022-12-07 14:29:16 +0000 |
| commit | c621c1a8e81856e6bf2be79714767d80466e9ede (patch) | |
| tree | 51e2842164cb84620c624cf90a436aa73c0492f7 /llvm/lib/MC/MCStreamer.cpp | |
| parent | dcfe2e7a13d4c8c2829f1d002e632a5854412d36 (diff) | |
| download | llvm-c621c1a8e81856e6bf2be79714767d80466e9ede.zip llvm-c621c1a8e81856e6bf2be79714767d80466e9ede.tar.gz llvm-c621c1a8e81856e6bf2be79714767d80466e9ede.tar.bz2 | |
[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 *) {} |