Make test robust enough to not fail due to CFG changes and re-enable for ARM/AArch64.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM. Enabling the test on ARM/AArch64 is a definite improvement IMO. Just a nit about the placement of the declare
| test/CodeGen/Generic/MachineBranchProb.ll | ||
|---|---|---|
| 11 | Maybe move this declare above the comment, with an extra blank line? | |
| test/CodeGen/Generic/MachineBranchProb.ll | ||
|---|---|---|
| 30 | Maybe also add a comment that this is here to prevent the SimplifyCFG pass run on AArch64/ARM to mess things up | |
Comment Actions
I modified the test case per your suggestions before commiting. Thanks for taking a look.
Maybe move this declare above the comment, with an extra blank line?