Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
openmp/runtime/src/dllexports | ||
---|---|---|
821–822 | I'm a little unsure if that is having the same semantics ? but I see %ifdef arch_32 on line 839 (and it is inside the ifdef) | |
1248–1250 | I've not touched these functions as this patch is mainly a fix (for the github issue: #56349) for I can try and make a separate patch later. |
openmp/runtime/src/dllexports | ||
---|---|---|
821–822 | I think this change is OK, or I fail to see any issue. Just for reference,
So having # Both x86 and x64 %ifdef IS_IA_ARCH ... Both x86 and x64 exports %ifdef arch_32 # Only x86 exports %endif ... %endif Seems fine. |
Should these be guarded?