shraiysh wrote:

> I'll see what I can do about adding an OpenMPIRBuilderTest.cpp to the patch.

I think it’s okay if it’s tested by clang tests (as Johannes mentioned). In 
case you decide to add tests for this, there is already an 
OpenMPIRBuilderTest.cpp in `llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp`. 
You can add one there if you’d like to.
cfe-commits mailing list

Reply via email to