Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/lib/MC/MCXCOFFStreamer.cpp | ||
---|---|---|
39 ↗ | (On Diff #279619) | Some thoughts on how to implement this for object generation: But if we want to break the existing practice (which could be a bigger change), we could consider using getQualNameSymbol as proxy of the MCSection. As it is right now, if we seeking for minimum work and keep things consistent, we could set the visibility using getXCOFFSection. And if later we think it's important enough to pursue using getQualNameSymbol as proxy of the MCSection, we could do a refactoring at that time? |
llvm/lib/MC/MCXCOFFStreamer.cpp | ||
---|---|---|
39 ↗ | (On Diff #279619) | there are XCOFF::StorageMappingClass MappingClass; XCOFF::SymbolType Type; XCOFF::StorageClass StorageClass; XCOFF::VisibilityType VisibilityType = XCOFF::SYM_V_UNSPECIFIED; in the MCSectionXCOFF class, we need get the value of StorageClass, Type,MappingClass before we invoke the getXCOFFSection every time. actually for the StorageClass, we can set the value of StorageClass when set the if we want to refactor StorageClass later, I do not think we need to consistent with StorageClass in this patch. we can do NFC after this patch for StorageClass. |
llvm/lib/MC/MCXCOFFStreamer.cpp | ||
---|---|---|
39 ↗ | (On Diff #279619) | I don't want the setting of MCSectionXCOFF to be different for different property. |
llvm/test/CodeGen/PowerPC/aix-xcoff-visibility.ll | ||
---|---|---|
76 | Generating visibility in XCOFF32 requires the auxiliary header. The test needs to check the auxiliary header. @DiggerLin, @jasonliu, fya. |
llvm/test/CodeGen/PowerPC/aix-xcoff-visibility.ll | ||
---|---|---|
76 |
|
nit: Address Lint comment.