[PATCH] D134157: [LoongArch] Add inline asm support for constraints f/l/I/K

2022-09-22 Thread Lu Weining via Phabricator via cfe-commits
SixWeining added a comment. In D134157#3805855 , @xen0n wrote: > I don't know if the clang changes should be split into its own commit (or the > title of this commit amended to mention `[Clang]` but I don't know if this is > appropriate), but the rest

[PATCH] D134157: [LoongArch] Add inline asm support for constraints f/l/I/K

2022-09-22 Thread Lu Weining via Phabricator via cfe-commits
SixWeining updated this revision to Diff 462139. SixWeining added a comment. Use double dashes for --mtriple. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134157/new/ https://reviews.llvm.org/D134157 Files:

[PATCH] D134157: [LoongArch] Add inline asm support for constraints f/l/I/K

2022-09-21 Thread WÁNG Xuěruì via Phabricator via cfe-commits
xen0n added a comment. I don't know if the clang changes should be split into its own commit (or the title of this commit amended to mention `[Clang]` but I don't know if this is appropriate), but the rest looks reasonable. Comment at:

[PATCH] D134157: [LoongArch] Add inline asm support for constraints f/l/I/K

2022-09-18 Thread Lu Weining via Phabricator via cfe-commits
SixWeining created this revision. SixWeining added reviewers: xen0n, MaskRay, xry111, rengolin, myhsu, wangleiat, gonglingqin. Herald added subscribers: StephenFan, JDevlieghere, hiraditya. Herald added a project: All. SixWeining requested review of this revision. Herald added projects: clang,