RAS extensions are part of ARMv8.2-A. This change enables them by introducing a
new instruction to ARM and AArch64 targets and several system registers.
Details
Diff Detail
Event Timeline
This mostly looks reasonable, just one consistency niggle with the naming:
include/llvm/Support/ARMTargetParser.def | ||
---|---|---|
127 | I'd just use "+/-ras". The "ext" is a bit redundant. |
include/llvm/Support/ARMTargetParser.def | ||
---|---|---|
127 | Unfortunately "ras" is already taken for an unrelated thing in the ARM target. And it seems to conflict with this. Doable but I'm afraid the patch will be slightly noisier. |
Changed +/-rasext to +/-ras. Also rename unrelated existing "ras" feature in ARM target to something more descriptive.
Bother, forgot to hit submit...
Thanks for updating the patch. I think it's an improvement, both are pretty short names but at least the new RAS has documentary support. LGTM!
I'd just use "+/-ras". The "ext" is a bit redundant.