[PATCH] D96393: [CodeGen] Add Machine Function Splitter and DebugInfo composite testcase

Snehasish Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 9 20:33:06 PST 2021


snehasish added a comment.

Can you elaborate on the motivation behind this test?

As the behaviour exists today, -split-machine-functions and -basic-block-sections are exclusive options and the former takes precedence. With D96392 <https://reviews.llvm.org/D96392> we are planning on switching the precedence but they still remain exclusive. Thus in this test, the RUN invocations on line 2,3 and 4 are using -split-machine-functions. Furthermore, the underlying mechanism driving both options are the same so testing the generation of debug information together is redundant. For more information take a look at the discussion in D90989 <https://reviews.llvm.org/D90989>.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96393/new/

https://reviews.llvm.org/D96393



More information about the llvm-commits mailing list