[llvm] Fix warnings while building llvm against emscripten for web assembly (PR #118262)
Anutosh Bhat via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 9 05:28:40 PST 2024
anutosh491 wrote:
cc @sbc100
Do these changes look reasonable to you now ? I think @whitequark approves.
Obviously we might find work arounds here and there but shouldn't we cater specific configs needed by specific systems ?
I mean I see different setting based on checks like `if(APPLE)`, `If(MSVC)` etc
But if that's not the case, whatever workaround is there should be well documented somewhere. I've not had problems building llvm but then I see too warnings to ignore, hence in favour of such a change.
https://github.com/llvm/llvm-project/pull/118262
More information about the llvm-commits
mailing list