Overall LGTM, though please take a look at the Jenkins -1s, seem native build related.
I'm also kind of shocked by the extract_values change. I guess we have a lucky implicit return since extract_values_delim will set EAX and hopefully it bubbles up, but that seems compiler/optimization-level dependent. I think this is undefined behavior.
Ping Varun Vasudev and Abin Shahab as the original authors, as a heads up on this bug.