铿锵小姐汇编错误？

Question

在我看来，clang++ 错过了 g++ 拾取的汇编代码中的错误。还是我错过了一些clang的编译器标志？我是汇编代码的新手。

使用 clang++ 我编译并链接了我的应用程序错误和警告，但我遇到了令人讨厌的分段错误。切换到 g++，另一方面，我得到了这些错误：

GO_F_ImageColourConversion.cpp: Assembler messages:
GO_F_ImageColourConversion.cpp:4679: Error: `(%rsi,%edx,2)' is not a valid base/index expression 
GO_F_ImageColourConversion.cpp:4682: Error: `(%rcx,%edx,1)' is not a valid base/index expression

我正在使用这些编译器标志： -DLINUX -g -Wno-deprecated -D_GNU_SOURCE -D_REENTRANT -D__STDC_CONSTANT_MACROS -fPIC -fPIE

我有以下代码（省略不相关部分）：

Ipp8u * pSrc;
Ipp8u * pDst;
int x, y;

                asm volatile
                    (
                     "movl      (%1, %0, 2), %%eax;\n"
                     "shlw      $8, %%ax;\n"
                     "shrl      $8, %%eax;\n"
                     "movw      %%ax, (%2, %0, 1);\n"

                    : /* no output */
                    : "r" (x), "r" (pSrc), "r" (pDst)
                    : "eax", "memory");
            }

通过查看answer on SO，我意识到我遇到了 32/64 位问题（我正在移植到 64 位）。Ipp8u* 是 8 位，但在我的机器上只有 4 位。

将 int 更改为 uintptr_t x, y; 似乎可以解决问题。为什么clang编译不报错？

Answer 1

gcc 和 clang 都为我扼杀了你的代码：

6 : error: base register is 64-bit, but index register is not
"movl (%1, %0, 2), %%eax\n"
^
<inline asm>:1:13: note: instantiated into assembly here
movl (%rdi, %edx, 2), %eax

来自clang 3.8 on the godbolt compiler explorer，带有一个包裹着它的函数，所以它是可测试的，你没有在问题中提供。您确定您的 clang 正在构建 64 位代码吗？ （-m64，不是-m32 或-mx32）。

在godbolt上提供一个链接到您的代码，并使用某些版本的clang默默地错误编译它，否则我只能对您的实际问题说“无法重现”。

是的，您的问题是x 是int，而您的问题是寻址模式下的混合寄存器大小。 (%rsi,%edx,2) 不可编码。

---

使用%q0 得到%rdx 并不能保证寄存器的高32 位中没有垃圾（尽管可能性极小）。相反，您可以使用 "r" ((int64_t)x) 到 sign-extend x to 64bits。

为什么需要内联汇编？您的 C 版本的编译器输出有多糟糕？

如果您确实想使用内联 asm，这要好得多：

uint32_t asm_tmp = *(uint32_t *)(x*2 + (char*)pSrc);  // I think I've reproduced the same pointer math as the addressing mode you used.
asm ( "shlw      $8, %w[v]\n\t"    // e.g.  ax
      "shrl      $8, %k[v]\n\t"    // e.g. eax.  potential partial-register slowdown from reading eax after writing ax on older CPUs
      : [v] "+&r" (asm_tmp)
      );
*(uint16_t *)(x + (char*)pDst) = asm_tmp;  // store the low 16

这可以用 clang 很好地编译，但是 gcc is kinda braindead about generating the address。也许地址的表达方式不同？

您的代码以加载开始并以存储结束，从而违背了约束的目的。总是让编译器尽可能多地处理。如果没有内联汇编，您可能会从中获得更好的代码，并且编译器会理解它的作用，并且可能会自动矢量化或进行其他转换。消除对 asm 语句的需要 volatile 和 "memory" clobber 对优化器来说已经是一个很大的改进：现在它是一个纯函数，编译器知道只转换一个寄存器。

另请参阅the end of this answer，了解更多编写不烂的内联 asm 的指南。