(#128) instruction: freeze

[dannypsnl]

TODO:

• update submodule ll once PR merged.

[mewmew]

Great job @dannypsnl! It's basically perfect. I only made a few suggestions to keep the code base consistent.

[mewmew]

Noticed an issue with freeze instruction that would give the error unable to locate local identifier "%x" of "@f", for the following LLVM IR:

define void @f(i32 %x) {
entry:
	add i32 %x, 42 ; works
	freeze i32 %x  ; fails
	ret void
}

[mewmew]

Hi @dannypsnl.

Glad you started this work. If anything in the review seem strange, just ping me on Discord and we can discuss :)

Cheers,
Robin

[dannypsnl]

Noticed an issue with freeze instruction that would give the error unable to locate local identifier "%x" of "@f", for the following LLVM IR:

define void @f(i32 %x) {
entry:
	add i32 %x, 42 ; works
	freeze i32 %x  ; fails
	ret void
}

This won't happen now, I already check output IR. Therefore, @mewmew you can check PR again. BTW, should we make a release for LLVM 10.0 supporting?

[mewmew]

Looks great.

[mewmew]

This won't happen now, I already check output IR. Therefore, @mewmew you can check PR again.

Merged :)

BTW, should we make a release for LLVM 10.0 supporting?

Definitely! I think we should prepare the last few things and then do a release of llir/llvm will LLVM 10.0 support.

I seems most larger concepts are now complete, the only thing remaining is e.g. new enum values?

Edit: I moved the comment below to #128 (comment) to keep track of it better.