LGTM

This changeset is causing this buildbot to fail : clang-ppc64le-linux-multistage

Condensed the test case.

Simplified the mir file.

In D78063#1995983, @lkail wrote:

The error would be something like this

 %4:g8rc = RLDICL killed %16:g8rc, 0, 32
 %26:g8rc_and_g8rc_nox0 = RLDIC %16:g8rc, 2, 30

*** Bad machine code: Using a killed virtual register ***
- function:    fn1
- basic block: %bb.1 if.then (0x1000a68f270)
- instruction: %26:g8rc_and_g8rc_nox0 = RLDIC %16:g8rc, 2, 30
- operand 1:   %16:g8rc
fatal error: error in backend: Found 1 machine code errors.

Providing a mir test case would be better.

Replaced the .ll test case with a .mir test case to be able to see the effect of removing SrcMI more clearly.

Removed the check for !SrcMI->hasImplicitDef() in emitRLDICWhenLoweringJumpTables before deleting SrcMI, since at this stage we do not expect Implicit defs by SrcMI

In D78063#1979659, @lkail wrote:

Could you pre-commit the test case as an NFC patch so that we can see the difference?

Added an offset to a pointer in one of the tests, for added coverage.

Added test case.

Addressed review comment. [NFC]

Integrated the test cases dcbfl.ll and dcbflp.ll in the previous differential to the previously existing test case dcbf.ll

Addressed review comments : Added a comment.

It seems it needs a big endian machine to fail as all the little endian machines seem to be unaffected. And also it will fail in stage 2 not stage 1 as shown in the link.

It is breaking buildbots other than the sanitizer one too.

This change set is breaking the buildbot :

This changeset is breaking this buildbot :

This changeset is breaking the buildbot clang-ppc64le-linux (http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/28006/steps/ninja%20check%201/logs/stdio)

This changeset is breaking this buildbot

This changeset is causing failures in multiple buildbots

This changeset is causing this failure in multiple buildbots

This changeset is causing a buildbot failure :

In D68411#1718116, @lei wrote:

Since these are just mnemonics for using dcbf, why not just add the tests to llvm/test/CodeGen/PowerPC/dcbf.ll instead of creating 2 small test files.

Your changeset is causing at least some of the stage2 failures in the buildbot http://lab.llvm.org:8011/builders/ppc64le-lld-multistage-test?numbuilds=300
http://lab.llvm.org:8011/builders/ppc64le-lld-multistage-test/builds/3951/steps/build-stage2-unified-tree/logs/stdio
Please upload a fix for it or pull back your changeset.
Thanks.

Your changeset is causing a failure in the buildbot http://lab.llvm.org:8011/builders/ppc64le-lld-multistage-test/builds/3845

It seems your changeset broke this buildbot . There are other prior failures but it seems your changeset introduced some new failures, listed below.
http://lab.llvm.org:8011/builders/clang-ppc64be-linux-lnt?numbuilds=100
http://lab.llvm.org:8011/builders/clang-ppc64be-linux-lnt/builds/27772

I think your changeset caused a failure in this buildbot.
http://lab.llvm.org:8011/builders/clang-ppc64le-linux-lnt/builds/18933
Failure : http://lab.llvm.org:8011/builders/clang-ppc64le-linux-lnt/builds/18933/steps/ninja%20check%201/logs/stdio

Regarding my last comment about the possibility of your changesets causing stage 2 failures, the failures are not from your changeset. The author of rL361356 confirmed they are from his changeset.

Hi @anil9 . Yes, you are right -- it's my change being causing the stage 2 failure. I'm working on fix. Should I revert it meanwhile?

Hello , your changeset is causing this buildbot to fail.

Your changeset might be causing failures on this buildbot http://lab.llvm.org:8011/builders/clang-ppc64be-linux-multistage?numbuilds=100
http://lab.llvm.org:8011/builders/clang-ppc64be-linux-multistage/builds/18248
Most of the stage 2 changes are still failing consistently, could you please, take a look.
Given the machine is going for a planned outage on Friday I may not be able to sort the exact cause so I am letting you earlier.
Thanks.

Your changeset might be causing failures on this buildbot http://lab.llvm.org:8011/builders/clang-ppc64be-linux-multistage?numbuilds=100
http://lab.llvm.org:8011/builders/clang-ppc64be-linux-multistage/builds/18248
Most of the stage 2 changes are still failing consistently, could you please, take a look.
Given the machine is going for a planned outage on Friday I may not be able to sort the exact cause so I am letting you earlier.
Thanks.

Hello, your patch seems to be causing a failure in the following buildbot.

I am Anil, the current buildbot maintainer. Your change set ( gen-dwarf-cpp.s, line 19) has broken this buildbot. http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage/builds/9819