[PATCH] D41705: Place undefined globals in .bss instead of .data

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 30 12:54:30 PST 2018


varkor via Phabricator via llvm-commits <llvm-commits at lists.llvm.org>
writes:

> varkor updated this revision to Diff 131707.
> varkor added a comment.
> Herald added a subscriber: javed.absar.
>
> Fixed an ARM CodeGen test whose behaviour was changed by the patch. (The data for `arr8` is now placed in the BSS, so the simplest fix seemed just to be no longer checking for `p2align`, which is no longer present.)

Why was p2align dropped? bss or not it should still be aligned, no?

Cheers,
Rafael


More information about the llvm-commits mailing list