Revision https://reviews.llvm.org/D69384 added the option "-codegenprepare" to an invocation of opt, which then requires LLVM to have the expected target available for code generation. The ARM build bots do not compile this target as part of their build, causing this test to abort: "Trying to construct TargetPassConfig without a target machine. Scheduling a CodeGen pass without a target triple set?" Moving this test to the X86 subdirectory ensures that builds without the X86 backend skip this test.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
I apologize for the constant early changes. I forget how phabricator catches and marks things as being referenced, so I was trying to get it to work.
This is a patch fix for D69384, which has broken ARM build bots. I present this as a band-aid since this feature and its test seems to be rather generic.
| llvm/test/Transforms/HotColdSplit/X86/coldentrycount.ll | ||
|---|---|---|
| 4 | Shouldn't this be x86-registered-target? If I build the compiler with an x86-only backend, but not an apple triple as the default triple, I think this test gets marked as unsupported with your change instead of being run. | |
Comment Actions
Is there any reason not to just move this file into llvm/test/Transforms/HotColdSplit/X86 where the existing lit filters will handle it?
Comment Actions
Either of these options sounds good to me.
@RKSimon At this point I'd tend to agree that this test belongs in the X86 subdirectory. I was hoping the original developer would come in and find a way to make this test generic once again, since doing so reduces test coverage in general.
Comment Actions
@JamesNagurne We proposed a fix in https://reviews.llvm.org/D85215, instead of moving it into X86/, we made the test cases architecture-agnostic.