[PATCH] D94403: [RISCV] Implement new architecture extension macros

Simon Cook via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 18 10:09:43 PST 2021


simoncook added a comment.

In D94403#2504046 <https://reviews.llvm.org/D94403#2504046>, @kito-cheng wrote:

> Thanks you implement that on clang, I think it's really great to included that in LLVM 12 release.
>
> I would like to define marco for sub-extension too, I know that's my fault, I didn't specify the behavior of sub-extension clearly on the spec, but I think it would be great if we also define sub-extension marcos, since it would be easier to check when some core only implement sub-extension, and the code can just check the sub-extensio rather than check both.

That's no problem, I'll extend to cover sub-extensions before landing. I notice the zvamo/zvlsseg extensions aren't yet accepted by the -march flag, so I will put a patch together for those and likely land those together.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94403



More information about the cfe-commits mailing list