[PATCH] D120592: [X86] Fix handling of Address-Size override prefix

Mon Feb 28 16:56:00 PST 2022

skan added a comment.

In D120592#3349375 <https://reviews.llvm.org/D120592#3349375>, @Amir wrote:

> @skan, @craig.topper 
> Thank you for the review! I didn't communicate the intent clearly. There's a BOLT bug report (https://github.com/llvm/llvm-project/issues/54066#issuecomment-1050327548) in which BOLT removes `addr32` prefix from `call imm`. I've root caused it to instruction printer and emitter ignoring the explicit (redundant) prefix. The code is produced by g++ (GCC) 9.2.1 20200225, with `-O3 -g -std=gnu++17 -Wl,-Bsymbolic-functions -Wl,--dynamic-list-cpp-new`. The linker wasn't mentioned.

I didn't apply the patch and rebuild the compiler. But I think the bug can reproduce w/o BOLT, right?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120592