agozillon wrote:

> Yeah, I didn't realize this was a part of a series. :)

Sorry, I should have made it clearer in the title! I was a little worried about 
title length as I broke the windows clonability of the llvm-project with a 
combination of bad commit names and SPR with the previous PR stack, so I 
avoided adding any index prefix like PR 1/4 etc. I think the morale of the 
story is that I really need to work a lot on my PR/Commit titles moving forward 
:-)

The full series are the following:


https://github.com/llvm/llvm-project/pull/82853 : the flang level changes 
required, also the top-level PR that should pass CI if all is well in the stack
https://github.com/llvm/llvm-project/pull/82852 : the MLIR -> LLVM-IR changes 
required, mainly a refactoring and generalisation of the previous member map 
work from the initial descriptor type mapping PR
https://github.com/llvm/llvm-project/pull/82851 : the current PR, MLIR changes 
required
https://github.com/llvm/llvm-project/pull/82850 : runtime tests the patch 
should enable, or perform more optimally/correctly corresponding to how Clang 
does things. Will pass CI by default as it doesn't run these tests 
unfortunately.


https://github.com/llvm/llvm-project/pull/82851
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to