Hi @ftynse ! I will update this diff accordingly. I'll figure out how to handle BlockAddresses first.

Addressed comments.

Addressed comments.

Hi, I was trying to implement some llvm intrinsics and wanted to understand the purpose of overloaded operands. The LLVM reference manual does not say that prefetch is overloaded.

@ftynse @rriddle please review this.

Addressed comments.

Updated with full context.

Hi @rriddle ! Thanks for the comments. I have updated the patch with more context now. Also, I have addressed all comments except one. Please let me know if you think I should change it to ArrayRef (I mentioned my reason in the inline comment).

I do not have write access to the repository. If would be great if someone could commit for me. Thanks.

Hi, can someone please provide a review for this patch.
Thanks.

Please review this.

I do not have write access to the repository. If would be great if someone could commit for me.
I have updated the patch.

I realized that I had forgotten to add function personality in our dialect. I will be adding it in another patch, once this gets accepted.
Apart from that, I have tried to address all the all inline comments. Please let me know if anything has to be changed. Thanks.

I have tried to address all the comments pointed out by @rriddle . I have also made the builder for llvm.InvokeOp. Please let me know if anything else has to be done.

I have tried to address almost all the comments.

Haven't looked at the code, let met know if you want me to. Please see design comments inline.

No, the code is not finished yet. I just wanted reviews on the design. Thanks.

Yes, I'm sorry for the confusion.

Thanks for the clarification. I will use arc in my future patches.

I would also add an error message in cases where constant conversion failed and made sure the callers of getConstantAsAttr are aware that it can return null (there may be x86_fp80 and fp128 values, that are not supported in MLIR at all). Fine for a separate commit.

Sure!

Also, @mehdi_amini I do not have permissions to commit. It would be great if you could commit this.

The order of instructions in the test for FP arithmetic wasn't right. Fixed it.

Thank you all for your comments. I will get back to this issue after doing the landingpad instruction.

Removed printOptionalAttrDict from printResumeOp as it is not in syntax.