[PATCH] D56571: [RFC prototype] Implementation of asm-goto support in clang
Nick Desaulniers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu May 30 12:55:35 PDT 2019
nickdesaulniers added a comment.
Was this committed accidentally?
Today in master I see:
- r362106: Revert "clang support gnu asm goto." Erich Keane <erich.keane at intel.com>
- r362059: Mark CodeGen/asm-goto.c as x86 specific after r362045 Fangrui Song <maskray at google.com>
- r362045: clang support gnu asm goto. Jennifer Yu <jennifer.yu at intel.com>
and yet this Phabricator review is still open.
It may be easier to rebase this into a monorepo checkout, then commit with `git llvm push`.
As @kees noted, for new test files that contain x86 assembly, they need a `-triple` that specifies an x86 target, otherwise the implicit target is based on the host, which may not be x86. We still want to verify that non-x86 host can cross compile to x86 correctly. Sorry we did not catch this earlier in code review.
CHANGES SINCE LAST ACTION
More information about the cfe-commits