[PATCH] D155573: [Clang] Fix the location of default init expressions

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

Should this come with a release note or is it correcting functionality added for this release?



================
Comment at: clang/test/SemaCXX/source_location.cpp:796
+static_assert(S(0).i == S{0}.i);
+static_assert(S(0).j == S{0}.i);
+}
----------------
Shouldn't this test fail because `i != j`?

Can you add a test that demonstrates we've corrected the source location information for diagnostics?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155573/new/

https://reviews.llvm.org/D155573