[PATCH] D136841: [LoongArch] Support inline asm operand modifier 'z'

2022-11-02 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers added a comment. In D136841#3893956 , @xen0n wrote: > Inching closer to ClangBuiltLinux on LoongArch it seems! Cool! Stay in touch when this is closer to working and we'll wire up support in CI to keep the build green. Repository: rG

[PATCH] D136841: [LoongArch] Support inline asm operand modifier 'z'

2022-10-30 Thread Lu Weining via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe415cb1d61e7: [LoongArch] Support inline asm operand modifier 'z' (authored by SixWeining). Changed prior to commit: https://reviews.llvm.org/D136

[PATCH] D136841: [LoongArch] Support inline asm operand modifier 'z'

2022-10-29 Thread Lu Weining via Phabricator via cfe-commits
SixWeining updated this revision to Diff 471730. SixWeining added a comment. Fix testcase file name. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136841/new/ https://reviews.llvm.org/D136841 Files: clang/test/CodeGen/LoongArch/inline-asm-operan

[PATCH] D136841: [LoongArch] Support inline asm operand modifier 'z'

2022-10-29 Thread Lu Weining via Phabricator via cfe-commits
SixWeining added a comment. In D136841#3893966 , @xen0n wrote: > Ah. There's a typo in the name of the testcase file... :(- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136841/new/ https://reviews.llvm.o

[PATCH] D136841: [LoongArch] Support inline asm operand modifier 'z'

2022-10-29 Thread WÁNG Xuěruì via Phabricator via cfe-commits
xen0n added a comment. Ah. There's a typo in the name of the testcase file... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136841/new/ https://reviews.llvm.org/D136841 ___ cfe-commits mailing list cfe-c

[PATCH] D136841: [LoongArch] Support inline asm operand modifier 'z'

2022-10-29 Thread Lu Weining via Phabricator via cfe-commits
SixWeining updated this revision to Diff 471729. SixWeining added a comment. Fix nit. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136841/new/ https://reviews.llvm.org/D136841 Files: clang/test/CodeGen/LoongArch/inline-asm-operand-modifiers.c

[PATCH] D136841: [LoongArch] Support inline asm operand modifier 'z'

2022-10-29 Thread WÁNG Xuěruì via Phabricator via cfe-commits
xen0n accepted this revision. xen0n added a comment. This revision is now accepted and ready to land. Inching closer to ClangBuiltLinux on LoongArch it seems! Comment at: llvm/lib/Target/LoongArch/LoongArchAsmPrinter.cpp:64 + break; + // TODO: handle other extra codes

[PATCH] D136841: [LoongArch] Support inline asm operand modifier 'z'

2022-10-27 Thread Lu Weining via Phabricator via cfe-commits
SixWeining created this revision. SixWeining added reviewers: xen0n, xry111, MaskRay, wangleiat, gonglingqin. Herald added subscribers: StephenFan, pengfei, hiraditya. Herald added a project: All. SixWeining requested review of this revision. Herald added projects: clang, LLVM. Herald added subscri