<div dir="ltr"><div dir="ltr">Hi Alex,</div><div dir="ltr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">
Would you recommend that I do what is done in unittest/ObjectYAML/MinidumpYAMLTest.cpp
 which just uses a string inside the source file, or use an Inputs 
directory? Both are not great, as you point out about using an Inputs 
directory</div></blockquote><div><br></div><div>I'd be inclined to go with the string in the source file approach. It keeps the whole test localised to one place, making it easier to understand what the test is actually testing. <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>
Also, I'm not sure that I can create an object without invoking yaml2obj
 itself, I can't seem to find a function to do this exposed by 
libObjectYAML.</div></blockquote><div><br></div><div>It looks to me like you'd need to move big chunks of code out of the yaml2obj tool into the ObjectYAML library. The Minidump version was added quite recently, and took the approach of doing all the layout in the library, whereas for most other formats, the work is done in the executable layer. I think it would be perfectly reasonable to do this code moving, and would then allow you to use the library in your test setup, similar to how minidump works.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>
If I do need to invoke yaml2obj, would you happen to know how to tell 
CMake to build yaml2obj first, and also where would I find that 
executable?</div></blockquote><div><br></div><div>I'm not a CMake expert by any means, but I think you would need to create a dependency on yaml2obj from the test target, then add a step in the test's CMakeLists.txt to run yaml2obj and generate your inputs. I'm still not convinced that it's the right way to go though.</div><div><br></div><div>James<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 25 Jun 2019 at 06:33, Alex Brachet-Mialot <<a href="mailto:alexbrachetmialot@gmail.com">alexbrachetmialot@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Thanks James!</div><div><br></div><div>Thanks for the help. I certainly see your point. <br></div><div><br></div><div>> In either case, if you wanted to change the test input, you'd need to rerun CMake again</div><div>This is unfortunate, I would have hoped I could have CMake create $(wildcard *.yaml) type of output in the make files. But it sounds like this doesn't exist on make alternatives, so this wont work.<br></div><div><br></div><div>There is actually a function to get the Inputs directory, <a href="http://llvm.org/doxygen/namespacellvm_1_1unittest.html" target="_blank">http://llvm.org/doxygen/namespacellvm_1_1unittest.html</a>. I think it is only used in one test, though. Would you recommend that I do what is done in unittest/ObjectYAML/MinidumpYAMLTest.cpp which just uses a string inside the source file, or use an Inputs directory? Both are not great, as you point out about using an Inputs directory. <br></div><div><br></div><div>Also, I'm not sure that I can create an object without invoking yaml2obj itself, I can't seem to find a function to do this exposed by libObjectYAML. If I do need to invoke yaml2obj, would you happen to know how to tell CMake to build yaml2obj first, and also where would I find that executable? Thanks so much, James.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 24, 2019 at 5:19 AM James Henderson <<a href="mailto:jh7370.2008@my.bristol.ac.uk" target="_blank">jh7370.2008@my.bristol.ac.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi Alex,<br></div><div><br></div><div>Not answering your question directly, but I'd highly recommend against using CMake to bake the test outputs at the cmake step for several different reasons:</div><div><br></div><div>1) If yaml2obj changes, you'd have to rerun cmake to see whether the new version would cause the tests to fail. This is not the default, so could easily lead to build-bot breakages.</div><div>2) It's not clear from your statement whether you're talking about lit tests or gtest unit tests. If the former, there's no real precedence for doing this. If the latter, you'd need to have your C++ code able to detect the build output directory somehow, which feels like it would make the code at least unclear, and at worst unusable.</div><div>3) In either case, if you wanted to change the test input, you'd need to rerun CMake again, rather than it just working. Runnning CMake is not cheap, especially on Windows for Visual Studio generators.</div><div><br></div><div>As Jordan has mentioned offline, and assuming you're talking about gtest, you're probably better off just calling the corresponding library functions to generate the object from YAML. Otherwise, just use yaml2obj directly.</div><div><br></div><div>If you want to explore your original suggestion further, and need help doing so, you probably need to give a bit more context as to what you've tried and what didn't work about it.</div><div><br></div><div>Regards,</div><div><br></div><div>James<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 22 Jun 2019 at 21:28, Alex Brachet-Mialot via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Hi,</div><div dir="auto"><br></div><div dir="auto">I want to write unit tests for a project I am working on. I need to create object files, preferably I want to have yaml files in the source tree and have cmake generate object files using yaml2obj. Does anyone know what I would put in the CMakeLists.txt to get this behavior?</div><div dir="auto"> </div><div dir="auto">For a concrete example if I didn’t explain clearly:</div><div dir="auto"><br></div><div dir="auto">In llvm/unittests/Object I would maybe have an Inputs directory, with a basic-obj.yaml. In llvm/unittests/Object I would want a test case which can open Inputs/basic-obj.o. </div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto">Alex</div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>
</blockquote></div>
</blockquote></div></div>