[lldb] SwiftLanguageRuntime::MethodName::Parse should reject objc methods

[bulbazord]

According to the Swift Reference Manual, swift identifiers are not
allowed to start with '+', '-', or '['. Reject any names starting with
those characters.

cc @adrian-prantl @kastiglione

[bulbazord]

@swift-ci please test

[bulbazord]

@adrian-prantl I don't have commit access to apple/llvm-project, could you commit this for me? Thanks!

[kastiglione]

swiftc also rejects function names with +, -, and [. Should we check the whole string not just the first char?

I tried

func `a+b`() { ... }
func `a-b`() { ... }
func `a[b]`() { ... }

[bulbazord]

swiftc also rejects function names with +, -, and [. Should we check the whole string not just the first char?

I tried

func `a+b`() { ... }
func `a-b`() { ... }
func `a[b]`() { ... }

Sure, I can add those too. I would like to also (in a different PR/change) add some tests to SwiftLanguageRuntime::MethodName in the same way we test CPlusPlusLanguage::MethodName. Sound good to you?

[bulbazord]

Updated to reject names with +, -, and [ in general. :)

[bulbazord]

@swift-ci please test

[kastiglione]

The follow up would have worked for me also. Thanks.