[PATCH] D133239: [RISCV][MC] Add minimal support for Ztso extension
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 6 09:29:34 PDT 2022
craig.topper added a comment.
In D133239#3772318 <https://reviews.llvm.org/D133239#3772318>, @reames wrote:
> In D133239#3769781 <https://reviews.llvm.org/D133239#3769781>, @asb wrote:
>
>> There are a few other places we typically add some test coverage for a new extension (which might be overkill - something to reconsider in the future perhaps)
>>
>> - clang/test/Preprocessor/riscv-target-features.c
>> - llvm/test/CodeGen/RISCV/attributes.ll
>> - llvm/test/MC/attribute-arch.s
>>
>> Otherwise, changes look good to me.
>
> I could not find llvm/test/MC/attribute-arch.s. Did you have a different file in mind? Added the other ones you mentioned.
llvm/test/MC/RISCV/attribute-arch.s
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133239/new/
https://reviews.llvm.org/D133239
More information about the llvm-commits
mailing list