Fix DataLayout::operator==()

[• rafael]

The patch is correct, but it should probably have a test. Without the patch llvm-link crashes when linking two files that differ only in the pointers in the datalayout, so it should be easy to add a test.

[maksqwe]

Can I add data to an existing test?

test/Linker/datalayout.ll

[maksqwe]

I added data layout pointers to "test/Linker/Inputs/datalayout-b.ll"

[• rafael]

LGTM.

Do you have commit access?

[maksqwe]

No.

[• rafael]

Committed as r206911.

lib/IR/DataLayout.cpp

              StackNaturalAlign == Other.StackNaturalAlign &&
              ManglingMode == Other.ManglingMode &&
              LegalIntWidths == Other.LegalIntWidths &&
-             Alignments == Other.Alignments && Pointers == Pointers;
+             Alignments == Other.Alignments && Pointers == Other.Pointers;
   assert(Ret == (getStringRepresentation() == Other.getStringRepresentation()));
   return Ret;
 }

test/Linker/Inputs/datalayout-b.ll

-target datalayout = "E"
+target datalayout = "e-p:16:16"