simon_tatham added inline comments.
================
Comment at: clang/lib/Basic/Targets/ARM.cpp:492
HW_FP |= HW_FP_SP | HW_FP_HP;
+ } else if (Feature >= "+cdecp0" && Feature <= "+cdecp7") {
+ unsigned Coproc = Feature.back() - '0';
----------------
As far as I can see, feature names have not already been checked against any
whitelist of known strings by the time we get here.
So this test will catch strings like `+cdecp0sdfsdfsdf` as well as the eight
strings you actually wanted.
And then the next line makes an out-of-range shift count from the last
character.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75843/new/
https://reviews.llvm.org/D75843
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
