[PATCH] D100919: [AArch64] Support customizing stack protector guard

[David Spickett via Phabricator via llvm-commits]

DavidSpickett added inline comments.


================
Comment at: clang/test/Driver/stack-protector-guard.c:59
+// INVALID-VALUE-AARCH64: error: invalid value 'tls' in 'mstack-protector-guard=','valid arguments to '-mstack-protector-guard=' are:sysreg global'
+// INVALID-REG-AARCH64: error: invalid value 'foo' in 'mstack-protector-guard-reg=','for AArch64'
----------------
I'm not sure if this is due to your code or the error machinery itself but these errors are strangely written.

I'd expect:
```
error: invalid value 'tls' in 'mstack-protector-guard='tls', valid arguments to '-mstack-protector-guard=' are:sysreg global'
```

Maybe it's assuming that there could be multiple values and `','` means that list option treats the value as a list? Or it's not using the right value and the comma is meant to be after the `=''` as in my example.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100919/new/

https://reviews.llvm.org/D100919