================
@@ -7557,6 +7560,34 @@ bool
AArch64AsmParser::parseDirectiveCFINegateRAStateWithPC() {
return false;
}
+/// parseDirectiveCFILLVMSetRAState
+/// ::= .cfi_set_ra_state ra_state, offset
+/// ::= .cfi_set_ra_state ra_state, pac_sym
+bool AArch64AsmParser::parseDirectiveCFILLVMSetRAState() {
----------------
s-barannikov wrote:
The boolean `true` returned by the old `ParseDirective` was umbiguous. It meant
either "parsing failed" or "directive doesn't need target-specific handling".
To disambiguate, we check if there are any pending errors after the function
returns. The new `parseDirective` returns a three-state type; targets should
migrate to using it. I migrated a couple of targets that had negative tests for
directive parsing, but AArch64 was over my head.
https://github.com/llvm/llvm-project/pull/209949
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
