[PATCH] D15883: Add ARM EHABI-related constants to unwind.h.

Timon Van Overveldt via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 5 08:08:47 PST 2016


timonvo added a comment.

In http://reviews.llvm.org/D15883#319445, @rengolin wrote:

> Tests?


Sure, I could add some. But is it common practice to test constants? The tests would end up being a duplication of the definition, I'm not sure how valuable that would be. I also couldn't find tests for any of the existing constants, at least not through a quick search under Clang's test/ directory.

Or were you thinking of some other type of test?


http://reviews.llvm.org/D15883





More information about the cfe-commits mailing list