[PATCH] D143602: Allow 32-bit pointers to be written in 64-bit slots

[David M. Lloyd via Phabricator via llvm-commits]

dmlloyd added a comment.

In D143602#4481817 <https://reviews.llvm.org/D143602#4481817>, @efriedma wrote:

> We have the following options here:
>
> 1. Leave it as, i.e. a backend error
> 2. Commit this as-is, i.e. depending on the target, it might work, but it could also either crash in the assembler or generate an invalid relocation.
> 3. Teach the assembler to emit padding for .quad.
> 4. Emit explicit padding in the AsmPrinter
>
> (2) seems like the worst of these options.

As the original author, I agree; when I wrote this I had wrongly assumed that all assemblers already supported this. I wouldn't mind if (3) or (4) were done but this would likely be beyond my experience with C++.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143602/new/

https://reviews.llvm.org/D143602