[clang] [llvm] [WebAssembly] Change to 64-bit table indices on Wasm64 (PR #180649)

[Demetrius Kanios via llvm-commits]

================

----------------
QuantumSegfault wrote:

I actually don't know. I'd have to investigate. Surrounding context suggested that Subtarget could be nullable and had to be guarded against.

I don't know if Subtarget is attached to particular IR functions or not, where there is no subtarget outside functions (I know functions can declare their own target attrs (e.g. sign-ext, etc.)). Not sure if target triple is allowed to mismatch in IR modules vs functions. Cause if it can that gets weird. If not, then the safe choice is to just use `Ctx.getTargetTriple().isArch64Bit()` as the global authority.

https://github.com/llvm/llvm-project/pull/180649