[PATCH] D49297: expose debugify as an LLVM option in clang

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 17 16:54:37 PDT 2018


vsk added a comment.

In https://reviews.llvm.org/D49297#1165866, @gbedwell wrote:

>   445 CheckFunctionDebugify [Merge disjoint stack slots]: FAIL
>   727 CheckFunctionDebugify [CodeGen Prepare]: FAIL
>   
>
> I've had a background process running on my PC hitting some preprocessed game code files pretty hard for a couple of weeks now.  It's produced small bug reports for a number of passes, but the above failures don't get spotted with the clang piped into opt approach. [...] The big question for me, is does this increase the potential coverage?


By default opt doesn't run -codegenprepare in its -OX pipelines, but it does supporting testing it. See e.g https://reviews.llvm.org/D47282.

The -stack-coloring pass lives in a weird space: it's a machine function pass which runs after isel, but it modifies IR. I don't think debugify operates on a very useful level at this point.


Repository:
  rL LLVM

https://reviews.llvm.org/D49297





More information about the llvm-commits mailing list