Jon Ross-Perkins

Results 205 comments of Jon Ross-Perkins

Reopening since I think the intent is to pick this back up.

I think it's reasonable to rewrite. The current approach isn't something we're committed to. As chandlerc noted on Discord, we stopped focusing on this because (while private) we felt spread...

> So when we see something like shown bellow, both rules apply and without context sensitive parsing/lexing how to know if they are identifiers or sized_type_literals ? > > ```...

I'm thinking through where your confusion may be, and maybe it's the mixing of types and expressions (which I think differs slightly from C++). In Carbon, types like String are...

Your version is missing a comma, so just to copy what's in the doc: ``` choice IntResult { Success(value: i32), Failure(error: String), Cancelled } ``` The names aren't supported syntax...

I suspect we'd recommend `Success(_: i32)` if we allow names. Using [struct types](https://github.com/carbon-language/carbon-lang/blob/trunk/docs/design/README.md#struct-types) probably should work. It would likely be a little extra overhead syntactically though, probably both in terms...

There are notes on using LLVM here that may be useful: https://github.com/carbon-language/carbon-lang/blob/trunk/docs/project/contribution_tools.md#clang-and-llvm Note in particular we use some non-default flags for compilation. That said -- right now https://source.corp.google.com/piper///depot/google3/third_party/carbon/lang/bazel/cc_toolchains/clang_configuration.bzl only detects...

It may be helpful to ask on #build-help on Discord (for responsiveness). There are a few things to check first, just to be sure of which binary is in use:...

As a reminder, here is the alternative considered in #731 : > We considered replacing the `adapter` keyword with the alternate spelling of "adaptor". Both spellings can be used for...

Note, in third-party c++ code I can easily search, `adapter` and `adaptor` are in almost even use.