[PATCH] D113096: [X86][MS-InlineAsm] Add constraint *m for memory access w/ global var

Kan Shengchen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 4 20:09:37 PDT 2021


skan added a comment.

In D113096#3110546 <https://reviews.llvm.org/D113096#3110546>, @skan wrote:

> In D113096#3110507 <https://reviews.llvm.org/D113096#3110507>, @thakis wrote:
>
>> Looks like this breaks tests on Mac: http://45.33.8.238/mac/38266/step_11.txt
>>
>> Please take a look and revert for now if it takes a while to fix.
>
> I can quickly fix it.
>
> It's the issue of ambiguous triple.  You are using the `-mtriple=x86_64-apple-mac*` while I am using `-mtriple=x86_64-unknown-linux-gnu`

Fixed by 6d03227c16ee <https://reviews.llvm.org/rG6d03227c16ee1950db0e1aa05fbc3201770248eb>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113096



More information about the cfe-commits mailing list