User Details
- User Since
- Mar 30 2018, 8:00 AM (286 w, 4 d)
May 4 2023
Apr 18 2023
Thanks for the fix!
Mar 31 2023
Mar 30 2023
Mar 21 2023
LGTM! Thanks!
Jan 11 2023
Thank you, LGTM
LGTM, thank you!
Jan 6 2023
Dec 17 2022
Thanks, LGTM
Dec 15 2022
LGTM
Oct 6 2022
Change initialization functions' names and correctly use ArrayRef
Oct 5 2022
Jun 15 2022
The patch looks correct to me, but looking at https://github.com/ARM-software/acle/blob/main/main/acle.md where all the ACLE macros are defined, I'm not sure that __ARM_FEATURE_RCPC is there currently. Maybe you also want to submit a patch there, or at least explain the context of this change please?
Apr 2 2022
Apr 1 2022
Refactor fp16 feature handling and add more fp16 tests for v8.5 and later
Mar 7 2022
Yeah that sounds good. Maybe aarch64-cortex-cpus.c might be a good start, to avoid having too many small files.
Further split aarch64-cpus-1.c
Mar 3 2022
Thanks for making this change. However, I think a better naming would be
Mar 2 2022
Update logic to define crypto extensions, strictly following the old implementation
Mar 1 2022
Taking into account remarks from @tmatheson, I'm reverting my latest changes
consisting in caching architecture feature into a variable, which makes the
getAArch64ArchFeaturesFrom* interfaces more complicated.
Feb 25 2022
Feb 22 2022
Feb 21 2022
Revert latest change, as it is likely that people still rely on __ARM_FEATURE_CRYPTO
Add checks for default ACLE macros for different architecture versions
Rebase
Feb 20 2022
Cache architecture feature to avoid scanning the feature list over and over again.
Feb 18 2022
Feb 17 2022
Remove reference to unused ACLE macro __ARM_FEATURE_CRYPTO
Feb 10 2022
Feb 9 2022
Turn off warnings for negative tests
Fix buildbots failures
Feb 8 2022
Reverted due to buildbots failures
Feb 7 2022
This is superseded by https://reviews.llvm.org/D118199. Comments have been addressed in the new patch. I should have commandeered this patch instead (did not know about this, sorry...).
Feb 3 2022
Add more tests
Feb 2 2022
Sorry for the linking issue, this has been fixed in commit ec00c9cdeb5ee9fc7846cb3d2a4d53eba2c35a43
Already fixed by commit ec00c9cdeb5ee9fc7846cb3d2a4d53eba2c35a43
TL;DR, I think these Has* flags have different meaning in different places of the code base.
Feb 1 2022
Remove prefix from tests
Update tests with clang -Xclang -verify
Update tests with clang_cc1 -verify
Revert latest patchset, as it should be put into a separate commit for clarity
Add support for +nomops
Jan 31 2022
Add more tests