[clang] [llvm] [Clang] Fix nomerge attribute not working with __builtin_trap(), __debugbreak(), __builtin_verbose_trap() (PR #101549)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 1 19:17:48 PDT 2024
dyung wrote:
> > > > Thanks for reporting. Those tests are failed due to not updated properly. I relanded this change with updated tests: [ae6dc64](https://github.com/llvm/llvm-project/commit/ae6dc64ec670891cb15049277e43133d4df7fb4b).
> > >
> > >
> > > Your updated change still seems to cause the test `CodeGen/attr-nomerge.cpp` to fail on AArch64 bots:
> > >
> > > * [lab.llvm.org/buildbot#/builders/17/builds/1677](https://lab.llvm.org/buildbot/#/builders/17/builds/1677)
> > > * [lab.llvm.org/buildbot#/builders/154/builds/2248](https://lab.llvm.org/buildbot/#/builders/154/builds/2248)
> > > * [lab.llvm.org/buildbot#/builders/190/builds/3065](https://lab.llvm.org/buildbot/#/builders/190/builds/3065)
> > >
> > > Can you either fix or revert if you need time to investigate?
> >
> >
> > Hopefully fixed by [c5f1395](https://github.com/llvm/llvm-project/commit/c5f1395f2f7f92015748069528d46ad89cecc9f1). If not, perhaps a fixed `-triple` should be used instead.
>
> Thanks for the fix!
The test still appears to be failing after the fix.
https://lab.llvm.org/buildbot/#/builders/190/builds/3070
https://github.com/llvm/llvm-project/pull/101549
More information about the cfe-commits
mailing list