[llvm] [X86Backend][M68KBackend] Make Ctx in X86MCInstLower (M68KInstLower) the same as AsmPrinter.OutContext (PR #133352)
weiwei chen via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 4 19:44:14 PDT 2025
weiweichen wrote:
> LGTM. This avoids creating a distinction between the 2 different sources of MCContexts in LLVM. As long as this happens to work for your use case that seems fine, but it seems fragile and could be broken in some other way in the future.
Thank you! Will definitely keep an eye on this part and follow up on more solid design if this turns out to be an issue down the road 🙏.
https://github.com/llvm/llvm-project/pull/133352
More information about the llvm-commits
mailing list