[PATCH] D137295: WebAssembly: Remove MachineFunction reference from MFI

Thu Nov 3 00:01:15 PDT 2022

pmatos added inline comments.

================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyMachineFunctionInfo.h:34
 class WebAssemblyFunctionInfo final : public MachineFunctionInfo {
-  const MachineFunction *MF;
-
----------------
Back in https://github.com/llvm/llvm-project/commit/cc5a1b3dd9039d50f6b9caa679d60398f0cec65f you changed this from a reference to a pointer. Apparently this is not needed at all and you are removing it. I am happy with it. Looking through the code I get the feeling that this was an artifact from doing similar things as other backends. Some of which have ended up removing MF, while others still have it.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137295/new/

https://reviews.llvm.org/D137295