Always Import FieldDecls from Records
Always Import FieldDecls from Records

When we switched this path over to use lazy member loading, a
subsequent commit introduced a check that members were canonical. In
certain extremely twisty scenarios involving multiple modular frameworks
pulling in non-modular headers, this check can fail which results in us
silently dropping these members on the floor.

It's ultimately correct to only pull in the canonical members, but real-world
examples of "system" frameworks evading modularity diagnostics exist so this
is strictly a regression.

Drop the canonicity check for FieldDecls.

rdar://86740970

rudkx/swift,apple/swift,apple/swift,benlangmuir/swift,gregomni/swift,rudkx/swift,gregomni/swift,atrick/swift,atrick/swift,ahoppen/swift,gregomni/swift,gregomni/swift,benlangmuir/swift,rudkx/swift,atrick/swift,apple/swift,roambotics/swift,ahoppen/swift,JGiola/swift,glessard/swift,benlangmuir/swift,benlangmuir/swift,JGiola/swift,ahoppen/swift,apple/swift,gregomni/swift,glessard/swift,roambotics/swift,roambotics/swift,glessard/swift,JGiola/swift,atrick/swift,benlangmuir/swift,glessard/swift,rudkx/swift,JGiola/swift,rudkx/swift,ahoppen/swift,benlangmuir/swift,JGiola/swift,apple/swift,glessard/swift,atrick/swift,JGiola/swift,roambotics/swift,ahoppen/swift,roambotics/swift,ahoppen/swift,gregomni/swift,apple/swift,glessard/swift,atrick/swift,roambotics/swift,rudkx/swift