Ericsson / clang

Cross Translation Unit analysis capability for Clang Static Analyzer. (Fork of official clang at http://llvm.org/git/clang)
http://clang.llvm.org/
Other
15 stars 10 forks source link

Use DeclarationName in structural eq #545

Open martong opened 5 years ago

martong commented 5 years ago

I realized that structural eq does not handle some cases properly.

[  FAILED  ] 4 tests, listed below:
[  FAILED  ] StructuralEquivalenceFunctionTest.NotEqualLiteralOperators
[  FAILED  ] StructuralEquivalenceCXXMethodTest.DifferentClasses
[  FAILED  ] StructuralEquivalenceCXXMethodTest.NotEqualConstructors
[  FAILED  ] StructuralEquivalenceCXXMethodTest.NotEqualDestructors
martong commented 5 years ago

Unfortunately this seems to introduce more structural eq errors. http://ci2.codechecker.gic.ericsson.se:8080/job/ctu_pipeline_clang7/44/

martong commented 5 years ago

Perhaps it is better to put this on hold until we fix some other ODR issues.