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

Lu Weining via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 22 05:16:57 PDT 2022


SixWeining added a comment.

In D134157#3805855 <https://reviews.llvm.org/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 looks reasonable.

Yes, I ever thought about spliting the patches into 2 (clang+llvm). But I finally decide to use a single one becuse I think this may help others a bit to understand how these inline asm(constrainsts, ...) are used and translated from frontend to backend.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134157



More information about the llvm-commits mailing list