[PATCH] D45255: [CodeGen] Add an option to suppress output of llvm.ident

[Jonas Devlieghere via Phabricator via cfe-commits]

JDevlieghere added a comment.

I’m happy with the test, thanks!

In https://reviews.llvm.org/D45255#1072335, @aprantl wrote:

> I'm not sure. Compatibility with GCC and getting identical output for debugging purposes seem to be at odds here. I personally lean slightly towards stripping it from the debug info as well.


If it were the other way around I’d agree with you, but given that -Qn becomes the default, I think we should not strip it from the debug info.


https://reviews.llvm.org/D45255