[Lldb-commits] [lldb] [LLDB] Add `ScalarLiteralNode` and literal parsing in DIL (PR #152308)

[Pavel Labath via lldb-commits]

https://github.com/labath commented:

I have a feeling that this would actually be better off as two node types (float and integer), as each stage (lexer, parser and evaluator) handles the two differently. Maybe then the integer node could use llvm::APInt directly, which would make the `is_unsigned` field look less out of place.

The implementation makes sense to me, but I don't know if we've reached consensus that this is the way this should work.

https://github.com/llvm/llvm-project/pull/152308