[PATCH] D54154: [ELF][MIPS] Use MIPS R6 `sigrie` as a trap instruction

James Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 6 15:12:15 PST 2018


jrtc27 added a comment.

In https://reviews.llvm.org/D54154#1289531, @ruiu wrote:

> Maybe we probably should change the type of TrapInstr from uint32_t to uint8_t[4]?


That would make sense to clearly convey it's in the target endianness rather than the host; it's certainly a little surprising to have a `uint32_t` in the "wrong" endianness floating around. My only concern with making it an array is whether it's clear that you need to *repeat* the trap instruction if it's 1 or 2 bytes, but that same argument could be made for the current `uint32_t`, so probably irrelevant.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D54154





More information about the llvm-commits mailing list