craig.topper added a comment.

In D137517#4012298 <https://reviews.llvm.org/D137517#4012298>, @pcwang-thead 
wrote:

> In D137517#4009175 <https://reviews.llvm.org/D137517#4009175>, @fpetrogalli 
> wrote:
>
>> @pcwang-thead, I addressed some of your comments.
>>
>> The value of `EnumFeatures` is now computed dynamicaly from the
>> `Features` field of the `Processor` class.
>
> Thanks! That sounds great to me!
>
>> As for generating `MArch` out of the `Features` field, @craig.topper
>> pointed me at
>> https://github.com/riscv-non-isa/riscv-toolchain-conventions/issues/11. From
>> the reading of it, it seems that the alphabetical order is enough to
>> build the string that carries `MArch`. Am I missing something?
>
> Currently, I think the alphabetical order is OK. If we relax the checking of 
> arch string someday, there is no doubt that we should change the 
> implementation here too.

The currently accepted order isn’t alphabetical. The single letter extensions 
have a specific order. The z extensions are ordered by looking up the second 
letter in the single letter order. If we alphabetize here i don’t think it will 
be accepted by the frontend.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137517

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to