Expose pub(crate) fn new constructors for value types

[corbanvilla]

Description

Exposing these constructors is helpful when working with LLVM code which isn't yet part of the C-API.

Related Issue

• InstructionValue::new is pub(crate) instead of pub #424
• Public constructors for wrapper types #366

Checklist

• I have read the Contributing Guide

[TheDan64]

@corbanvilla The newly pub fns need doc comments. Also, Value::new should not be pub since it is internal. Once those are fixed we can merged this

[corbanvilla]

@TheDan64 apologies for the delay, just added doc comments!

[TheDan64]

@corbanvilla there appear to be some merge conflicts that need to be resolved

[corbanvilla]

@TheDan64 Could you clarify where you're referring to.... it looks up to date under the files changed tab.

[TheDan64]

For some reason I see

[TheDan64]

ah okay it looks like I can do a regular merge instead of a rebase...